Skip to content
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

Fix Issue 22956 - Bump LLD version #505

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Conversation

maxhaton
Copy link
Member

Or try to bump version at least

Or try to bump version at least
@dlang-bot
Copy link
Contributor

Thanks for your pull request, @maxhaton!

Bugzilla references

Auto-close Bugzilla Severity Description
22956 minor Bump LLD version for windows releases

@maxhaton
Copy link
Member Author

@kinke looks like we got bitten by a brownout

@MoonlightSentinel
Copy link
Contributor

#506

@maxhaton
Copy link
Member Author

OK so LLVM don't do releases as the script expects anymore.

https://github.com/llvm/llvm-project/releases/download/llvmorg-14.0.0/lld-14.0.0.src.tar.xz is the new place we need to look apparently.

@maxhaton
Copy link
Member Author

I guess we should move to using the .sigs instead

@adamdruppe
Copy link

so what's left here? rerunning the tests or changing the download url?

@maxhaton
Copy link
Member Author

maxhaton commented Apr 1, 2022

so what's left here? rerunning the tests or changing the download url?

We updated the runner, but old script doesn't play nice with the new LLVM cmake setup and this version of VS. I think I'm just going to replace it with the instructions from the llvm site. The only potential issue is the SEH thing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants