Skip to content

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 1 commit into from
Aug 12, 2024
Merged

Conversation

compnerd
Copy link
Member

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.

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.
@compnerd
Copy link
Member Author

@swift-ci please build toolchain Windows platform

@jakepetroules jakepetroules self-requested a review August 11, 2024 23:39
@jakepetroules
Copy link
Contributor

Thank you!

@compnerd
Copy link
Member Author

@swift-ci please test

@compnerd compnerd merged commit 5c00681 into swiftlang:main Aug 12, 2024
6 checks passed
@compnerd compnerd deleted the sqlite branch August 12, 2024 16:18
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
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants