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

[deps/llvm] don't use hardcode LLVM_SHARED_LIB_NAME (#45908) #48547

Closed

Conversation

stahta01
Copy link
Contributor

@stahta01 stahta01 commented Feb 6, 2023

(cherry picked from commit eb72c2a)

I needed this commit when working on trying to get MSys2 MINGW64 to build LLVM; I am working on updating the MSys2 Mingw package for Julia to 1.8.3 or maybe a different version.

I still have to test the commit works; so, might be a day or so before I change from Draft status.
This is the first git cherry-pick commit I have done; so, I want to be extra careful.

Edit: After testing, I found the need to hand edit two files to see if the build issue I am having locally would be fixed by it.
I am still running the tests.

Tim S.

@inkydragon
Copy link
Sponsor Member

the base branch here needs to be backports-release-1.8 instead of release-1.8.
#46987

Hand edited changes to deps/llvm.version and deps/Versions.make
Moved LLVM_VER from Versions.make to llvm.version by Tim S.

(cherry picked from commit eb72c2a)
@stahta01
Copy link
Contributor Author

stahta01 commented Feb 6, 2023

the base branch here needs to be backports-release-1.8 instead of release-1.8.
#46987

Okay, I did not see that branch; can I change or do I need to close and create an new PR?
Edit: I did not see it because it does not exist!
Edit2: Created issue #48554

Tim S.

@stahta01
Copy link
Contributor Author

stahta01 commented Feb 6, 2023

I can not figure out how to change the branch targeted by PR; so, closing this one.

@stahta01 stahta01 closed this Feb 6, 2023
@giordano
Copy link
Contributor

giordano commented Feb 6, 2023

Click on the "Edit" button in the top-right corner, then the target branch names becomes a dropdown menu. You can still reopen this PR and change it.

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