Skip to content

Prepare for Linux build and CI #14

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 34 commits into from
Oct 2, 2024
Merged

Conversation

ktoso
Copy link
Collaborator

@ktoso ktoso commented Sep 30, 2024

This prepares for Linux CI.

  • Include docker files for local Linux development
  • Pull OpenJDK 23, as 22 is not supported anymore since this month, so we should stick to "supported" versions I think (we can debate this later)
  • Disable the nm hack on Linux, the entire approach was going to be removed anyway so no reason to implement any of the Dylib... stuff parity on other platforms -- we'll just remove it all and replace by depending on Option to emit mangled names for public symbols into the .swiftinterface #76600 in snapshot Swift
  • Pick changes from [CI] Add support for GitHub Actions #10
    • will polish that up to use the supported versions and commands

@ktoso ktoso marked this pull request as draft September 30, 2024 12:25
@ktoso ktoso marked this pull request as ready for review September 30, 2024 12:54
@ktoso ktoso marked this pull request as draft September 30, 2024 12:54
@ktoso ktoso force-pushed the wip-linux-biuld branch 3 times, most recently from 0491d87 to fe282a9 Compare September 30, 2024 15:00
@ktoso
Copy link
Collaborator Author

ktoso commented Sep 30, 2024

Still a little bit of work remaining, but we're getting there :)

@ktoso ktoso force-pushed the wip-linux-biuld branch 3 times, most recently from df8fcb5 to c095f59 Compare October 2, 2024 12:19
@ktoso ktoso marked this pull request as ready for review October 2, 2024 15:01
@lokesh-tr
Copy link
Contributor

Almost ready to merge? 👀🤞🏻

@ktoso
Copy link
Collaborator Author

ktoso commented Oct 2, 2024

Okey this is a good first step: we have the swift tests passing and we can work on getting the java ones passing next.

all the other soundness things also ok, except the format task has some issues.

Overall we're now waiting for a new nightly toolchain and then we should merge #27

Meanwhile we can continue to rely on the swift tests for the time being, this is already much better than no CI after all 😉

@lokesh-tr
Copy link
Contributor

@ktoso That's great!

@ktoso
Copy link
Collaborator Author

ktoso commented Oct 2, 2024

This has had a lot of churn so I'll squash it all as "initial ci support" basically

@ktoso ktoso merged commit 080c2ae into swiftlang:main Oct 2, 2024
8 of 9 checks passed
@ktoso ktoso deleted the wip-linux-biuld branch October 2, 2024 15:06
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.

3 participants