Skip to content

[windows] Switch ICU4C from SourceForge to GitHub. #28683

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

Conversation

drodriguez
Copy link
Contributor

Use GitHub download instead of SourceForge for ICU4C binary package, since SF fails from time to time and the build fails because of it.

Also remove the usage of -z, since Community CI deletes the previous file anyway, so no point in checking for a cached copy, when the only thing that would happen is an annoying message in the build logs.

@drodriguez
Copy link
Contributor Author

@swift-ci please clean test Windows platform

Use GitHub download instead of SourceForge for ICU4C binary package, since SF fails from time to time and the build fails because of it.

Also remove the usage of -z, since Community CI deletes the previous file anyway, so no point in checking for a cached copy, when the only thing that would happen is an annoying message in the build logs.
@drodriguez drodriguez force-pushed the windows-switch-to-github-for-icu branch from 3ca3008 to b368f7a Compare December 11, 2019 22:09
@drodriguez
Copy link
Contributor Author

drodriguez commented Dec 11, 2019

@swift-ci please test Windows platform

[Edit: it actually has started in https://ci-external.swift.org/view/Pull Request/job/swift-PR-windows/234 and it seems that ICU has been downloaded and unzipped correctly this time]

@drodriguez
Copy link
Contributor Author

@swift-ci please smoke test

@drodriguez drodriguez merged commit 49993c5 into swiftlang:master Dec 12, 2019
@drodriguez drodriguez deleted the windows-switch-to-github-for-icu branch December 12, 2019 01:14
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