Skip to content

utils: support building android SDKs on Windows#72014

Merged
hyp merged 12 commits into
swiftlang:mainfrom
compnerd:android
Jun 7, 2024
Merged

utils: support building android SDKs on Windows#72014
hyp merged 12 commits into
swiftlang:mainfrom
compnerd:android

Conversation

@compnerd

@compnerd compnerd commented Mar 1, 2024

Copy link
Copy Markdown
Member

This adds the ability to build the Android SDKs as part of the Windows build.

This adds the ability to build the Android SDKs as part of the Windows
build.
Comment thread utils/build.ps1 Outdated
Comment thread utils/build.ps1 Outdated
Comment thread utils/build.ps1 Outdated
Comment thread utils/build.ps1 Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did the trick I suggested with Isolate-EnvVars and Invoke-VsDevShell not work?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I need to try to implement them. Diff welcome of course :)

Comment thread utils/build.ps1 Outdated
Comment thread utils/build.ps1 Outdated
Comment thread utils/build.ps1 Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The $LibraryRoot\icu-69.1\usr\lib\$Platform\$ShortArch prefix is worth extracting before this block to keep it looking line just a set of assignments

Comment thread utils/build.ps1 Outdated
Comment thread utils/build.ps1 Outdated
Comment thread utils/build.ps1 Outdated
@compnerd compnerd force-pushed the android branch 2 times, most recently from c22f85f to 41533ef Compare March 1, 2024 23:33
@hyp hyp force-pushed the android branch 2 times, most recently from e0c89db to bb92d76 Compare April 29, 2024 22:45
@hyp

hyp commented Apr 29, 2024

Copy link
Copy Markdown
Contributor

adding latest changes today

@hyp

hyp commented Apr 30, 2024

Copy link
Copy Markdown
Contributor

@compnerd @tristanlabelle made some updates. could you take another look please? also updated foundation - swiftlang/swift-corelibs-foundation#4889

Comment thread utils/build.ps1 Outdated
Comment thread utils/build.ps1 Outdated
Comment thread utils/build.ps1 Outdated
Comment thread utils/build.ps1 Outdated
@hyp

hyp commented May 14, 2024

Copy link
Copy Markdown
Contributor

@compnerd addressed your review comments

@hyp

hyp commented May 30, 2024

Copy link
Copy Markdown
Contributor

@swift-ci please tst

@hyp

hyp commented May 30, 2024

Copy link
Copy Markdown
Contributor

@swift-ci please test

@hyp

hyp commented May 30, 2024

Copy link
Copy Markdown
Contributor

@swift-ci please build toolchain

@hyp hyp marked this pull request as ready for review May 30, 2024 19:18
@hyp hyp requested review from a team and ktoso as code owners May 30, 2024 19:18
@hyp hyp requested a review from hjyamauchi May 30, 2024 19:18
@hyp

hyp commented May 31, 2024

Copy link
Copy Markdown
Contributor

@swift-ci please test windows platform

1 similar comment
@hyp

hyp commented Jun 3, 2024

Copy link
Copy Markdown
Contributor

@swift-ci please test windows platform

Comment thread utils/build.ps1
Comment thread utils/build.ps1
Comment thread utils/build.ps1
@hyp

hyp commented Jun 6, 2024

Copy link
Copy Markdown
Contributor

@swift-ci please test windows platform

@hyp

hyp commented Jun 6, 2024

Copy link
Copy Markdown
Contributor

@swift-ci please test windows platform

@hyp

hyp commented Jun 6, 2024

Copy link
Copy Markdown
Contributor

@swift-ci please build windows toolchain

@hyp

hyp commented Jun 6, 2024

Copy link
Copy Markdown
Contributor

@swift-ci please test

@hyp

hyp commented Jun 6, 2024

Copy link
Copy Markdown
Contributor

@swift-ci please build toolchain

@hyp

hyp commented Jun 6, 2024

Copy link
Copy Markdown
Contributor

@swift-ci please test

@hyp

hyp commented Jun 6, 2024

Copy link
Copy Markdown
Contributor

@swift-ci please test source compatibility

@hyp

hyp commented Jun 6, 2024

Copy link
Copy Markdown
Contributor

This is ready!

Comment thread utils/build.ps1
Comment thread utils/build.ps1
@hyp

hyp commented Jun 6, 2024

Copy link
Copy Markdown
Contributor

@swift-ci please test

@hyp

hyp commented Jun 6, 2024

Copy link
Copy Markdown
Contributor

@swift-ci please build toolchain

Comment thread utils/build.ps1 Outdated

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wait, this is the host ... I was wrong. I'm confused, why are we not passing in the Arch for the host arch sdk? Could we not cross-compile to more than one arch?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

because there's only one host arch?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah right, this is the host SDK for the toolchain, not an arbitrary host SDK. I think that generalising this to Get-SwiftSDK($Arch) might be nice as it currently makes me think that it is getting the Swift SDK for a particular host (which is true, it's just that the particular host happens to be the host that we are building the toolchain for). But, lets leave that for a follow up, I'd like to get this in.

@hyp

hyp commented Jun 6, 2024

Copy link
Copy Markdown
Contributor

@swift-ci please test

@hyp

hyp commented Jun 6, 2024

Copy link
Copy Markdown
Contributor

@swift-ci please build toolchain

@compnerd

compnerd commented Jun 7, 2024

Copy link
Copy Markdown
Member Author

@swift-ci please test source compatibility

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.

4 participants