-
Notifications
You must be signed in to change notification settings - Fork 10.5k
utils: migrate away from fetched SQLite #75823
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
With the introduction of swift-toolchain-sqlite, we have a version of SQLite that we can use to build the toolchain. Prefer this over the fetched amalgamation which simplifies the logic for the toolchain and ensures that we are able to track this as a regular component.
@swift-ci please build toolchain Windows platform |
Thank you! |
jakepetroules
approved these changes
Aug 11, 2024
@swift-ci please test |
compnerd
added a commit
to thebrowsercompany/swift-build
that referenced
this pull request
Aug 12, 2024
Update to use the new sqlite mirror for the swift toolchain. This ensures that we keep in sync with the official toolchain builds.
compnerd
added a commit
to compnerd/swift-build
that referenced
this pull request
Aug 12, 2024
Update to use the new sqlite mirror for the swift toolchain. This ensures that we keep in sync with the official toolchain builds.
compnerd
added a commit
to thebrowsercompany/swift-build
that referenced
this pull request
Aug 12, 2024
Update to use the new sqlite mirror for the swift toolchain. This ensures that we keep in sync with the official toolchain builds.
compnerd
added a commit
to compnerd/swift-build
that referenced
this pull request
Aug 12, 2024
Update to use the new sqlite mirror for the swift toolchain. This ensures that we keep in sync with the official toolchain builds.
hyp
pushed a commit
to thebrowsercompany/swift-build
that referenced
this pull request
Aug 27, 2024
Update to use the new sqlite mirror for the swift toolchain. This ensures that we keep in sync with the official toolchain builds. (cherry picked from commit e626872)
hyp
pushed a commit
to hyp/swift-build
that referenced
this pull request
Aug 29, 2024
Update to use the new sqlite mirror for the swift toolchain. This ensures that we keep in sync with the official toolchain builds. (cherry picked from commit e626872)
hyp
added a commit
to compnerd/swift-build
that referenced
this pull request
Oct 7, 2024
* Update default.xml Remove ICU from the checkout. This has been superseded by swift-foundation-icu. (cherry picked from commit 5183f32) * Update default.xml Add swift-toolchain-sqlite to the toolchain manifest. (cherry picked from commit 4dc9f92) * GHA: timestamp the caches Enable the timestamp in the caches to make sure that the correct cache is restored. This should hopefully help increase the cache hit rate. (cherry picked from commit aada331) * repo: update the organisations for migrated repositories Update the organisation for the repositories which have migrated from apple to swiftlang. (cherry picked from commit 73d1501) * GHA: update workflow for swiftlang/swift#75823 Update to use the new sqlite mirror for the swift toolchain. This ensures that we keep in sync with the official toolchain builds. (cherry picked from commit e626872) * GHA: remove ICU build after the re-core transition Remove the ICU dependency as it has been removed from the repo manifest. This now requires that we use the re-cored Foundation. (cherry picked from commit a513566) * Update default.xml (cherry picked from commit 9f72dc7) * [android] checkout unpinned installer script The pin will no longer be needed since we're switching to swift-foundation (cherry picked from commit b49b337) * [GHA workflow] only create a latest release when running on main branch version (cherry picked from commit ea62eb9) * GHA: update curl version CURL was updated to 8.9.1 upstream. Update the build rules to reflect that. (cherry picked from commit 1b5a7bb) * GHA: split out standard library phase Split out the standard library build phase to allow us to build the macros for the correct hosts. This is required to allow us to build the macros for distribution and for the build of the SDK. Co-authored-by: Alex Lorenz <[email protected]> (cherry picked from commit 1093ac8) * [sqlite] Remove Android builds These are not consumed by any other step. (cherry picked from commit 1177dfa) * GHA: introduce a new macros build phase Introduce a new macros build phase for the Foundation (and eventually Testing) macros. This depends on the previously introduced stdlib phase as the macros use the standard library. We build the macros for the compiler hosts, reusing them to build the SDK content. Wire this into the packaging step for the build tools as well. Co-authored-by: Alex Lorenz <[email protected]> (cherry picked from commit 400fce1) * [Mac build] Rename builder to a generic name (#797) `windows_build_runner` is too Windows-specific. This renames the variable to `default_build_runner`, to make space for future Darwin builds. The variable is kept to its original name in the `context` step, since this step will eventually also output Darwin variables. (cherry picked from commit e0239e1) * [Mac build] Move build to sub-workflow This is a preliminary step to eventually add the Mac build. The Mac build will be using a separate job with its own matrix. Co-authored-by: Saleem Abdulrasool <[email protected]> (cherry picked from commit 0cc87ce) * GHA: enable HCS on Windows toolchains Now that the just built clang should no longer perform HCS on scoped EH functions, we should be able to enable this optimization globally. (cherry picked from commit d558028) * Make runners configurable via workflow input (cherry picked from commit a75c14e) * GHA: tweak some names (NFC) Clean up the job names to be consistent throughout. (cherry picked from commit ecbfb22) * GHA: adjustments for the SDK phase Adjust the paths for the SDK phase, unify the swift-corelibs-foundation checkout. Co-authored-by: Alex Lorenz <[email protected]> (cherry picked from commit 876ec4a) * GHA: update repositories for swiftlang migration Update the repositories which have migrated to the swiftlang organisation. (cherry picked from commit 2395959) * Properly set SQLite version This was broken in #796 (cherry picked from commit 8c02c26) * GHA: repair the Windows runtime/SDK packaging We now need to download the standard library and the SDK to get the same content that we used to previously have in just the SDK. Adjust that accordingly. (cherry picked from commit b91b97e) * GHA: wire up Foundation macros to swift-corelibs-foundation We are already building the Foundation macros but did not pass that along to the Foundation build. This is now required and failure to pass it along results in a build failure. Wire it up so that we can build and avoid another build of the macros. (cherry picked from commit 0463388) * GHA: setup builds for swift-testing Introduce a build of swift-testing in the SDK as that is now distributed as part of the SDKs upstream. This allows us to have parity with the official builds. (cherry picked from commit 034d28f) * Add the workspace hash to ccache key (#811) Depending on the runner configuration, `github.workspace` might be set up differently, resulting in issues when restoring previous builds from sccache. In order to work around this issue, this adds a hash of `github.workspace` to the sccache key to uniquely identify the build between different runner types. (cherry picked from commit 0579aa8) * Use runners from vars if available (#812) This was removed in #801 to bring the file in line with downstream. However, it is better to rely on the vars environment when invoking the main workflow from dispatch, while offering an override when invoking the workflow as a sub-workflow. (cherry picked from commit 67bb06c) * GHA: correct swift-testing macro provider The plugin name is `TestingMacros.dll` not `SwiftTesting.dll`. Correct the spelling for the build. (cherry picked from commit 4ca63bb) * GHA: specify Swift compile target, C++ compiler, install location Add missing flags for swift-testing configuration. This is primarily the C++ compiler, target triple, and the install location. (cherry picked from commit 9210338) * GHA: remove swift-syntax dependency for SDK This is no longer needed as we pre-build the macros and no longer need to match up the swift-syntax build from the toolchain. (cherry picked from commit e6a8682) * GHA: fork swift-testing for Android fixes Pull the future main branch for Android fixes. (cherry picked from commit 8336d10) * GHA: restructure devtools build Setup `SDKROOT` environment variable rather than inlining that at each site. This greatly simplifies the CMake invocation. Additionally, extract prebuilt content into `BinaryCache` rather than `BuildRoot` which allows us to easily isolate the built artifacts for upload. (cherry picked from commit 105a9e4) * GHA: add a workaround for old Swift driver workaround the issue with missing new swift driver when building swift testing for android by disabling CMP0157. (cherry picked from commit 0b18205) * GHA: download additional images when packaging Pull in the Android stdlib when building Android installer MSI. (cherry picked from commit 3f2b15d) * Fix devtools stage (#815) * build with explicit -sdk passed into Swift * The SDKROOT environment variable isn't enough when using the old driver * checkout latest swift testing * pass c++ flags and c++ compiler for the markdown build * download devtools host sdk after the target libs are setup correctly * fix cmark path * debugging tools: download stdlib as well (cherry picked from commit 6ba7e5f) --------- Co-authored-by: Saleem Abdulrasool <[email protected]> Co-authored-by: Fabrice de Gans <[email protected]> Co-authored-by: Fabrice de Gans <[email protected]> Co-authored-by: Saleem Abdulrasool <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
With the introduction of swift-toolchain-sqlite, we have a version of SQLite that we can use to build the toolchain. Prefer this over the fetched amalgamation which simplifies the logic for the toolchain and ensures that we are able to track this as a regular component.