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 32-bit winegcc call on Linux when using LLD #5878

Merged
merged 3 commits into from
Jan 10, 2021

Conversation

rdrpenguin04
Copy link
Contributor

No description provided.

@LmmsBot
Copy link

LmmsBot commented Jan 9, 2021

🤖 Hey, I'm @LmmsBot from github.com/lmms/bot and I made downloads for this pull request, click me to make them magically appear! 🎩

Linux

Windows

macOS

🤖
{"platform_name_to_artifacts": {"Linux": [{"artifact": {"title": {"title": "(AppImage)", "platform_name": "Linux"}, "link": {"link": "https://12081-15778896-gh.circle-artifacts.com/0/lmms-1.3.0-alpha.1.64%2Bg57587bb-linux-x86_64.AppImage"}}, "build_link": "https://circleci.com/gh/LMMS/lmms/12081?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link"}], "Windows": [{"artifact": {"title": {"title": "32-bit", "platform_name": "Windows"}, "link": {"link": "https://12084-15778896-gh.circle-artifacts.com/0/lmms-1.3.0-alpha.1.64%2Bg57587bb1f-mingw-win32.exe"}}, "build_link": "https://circleci.com/gh/LMMS/lmms/12084?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link"}, {"artifact": {"title": {"title": "64-bit", "platform_name": "Windows"}, "link": {"link": "https://12085-15778896-gh.circle-artifacts.com/0/lmms-1.3.0-alpha.1.64%2Bg57587bb1f-mingw-win64.exe"}}, "build_link": "https://circleci.com/gh/LMMS/lmms/12085?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link"}], "macOS": [{"artifact": {"title": {"title": "", "platform_name": "macOS"}, "link": {"link": "https://12083-15778896-gh.circle-artifacts.com/0/lmms-1.3.0-alpha.1.64%2Bg57587bb1f-mac10.14.dmg"}}, "build_link": "https://circleci.com/gh/LMMS/lmms/12083?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link"}]}, "commit_sha": "caf3ec5cce77bfd66dacc3b7670b655c7d0b94c7"}

@PhysSong
Copy link
Member

Possibly related: #5024
See also: llvm/llvm-project@0a7412f

@rdrpenguin04
Copy link
Contributor Author

That other issue could be related; since lld blocks text relocations by default, it was a hard error for me.

The commit you referenced seems to be implementing the -z notext flag, not setting it as the default. I run main branch LLVM, so if it was made the default, I would have it by now.

This commit simply adds the flag so that RemotePluginVST32 will build and work properly, and, as I have stated, I have tested it on a 32-bit VST that I have.

@PhysSong
Copy link
Member

LGTM. Please revert the last commit, BTW. The CONTRIBUTORS file is a fallback for automatically generated file, so it should probably be regenerated properly if it needs updating.

@rdrpenguin04
Copy link
Contributor Author

K, will do.

@PhysSong PhysSong merged commit ca17641 into LMMS:master Jan 10, 2021
@rdrpenguin04 rdrpenguin04 deleted the winegcc-m32-lld-fix branch January 10, 2021 12:38
zynskeywolf pushed a commit to zynskeywolf/lmms that referenced this pull request Jan 12, 2021
IanCaio pushed a commit to IanCaio/lmms that referenced this pull request Mar 28, 2021
devnexen pushed a commit to devnexen/lmms that referenced this pull request Apr 10, 2021
sdasda7777 pushed a commit to sdasda7777/lmms that referenced this pull request Jun 28, 2022
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