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

gnustep-base, gnustep-gui, gnustep-back: Build on clang64 #21884

Merged
merged 3 commits into from
Sep 15, 2024

Conversation

qmfrederik
Copy link
Contributor

@qmfrederik qmfrederik commented Sep 9, 2024

gnustep-base, gnustep-gui and gnustep-back were disabled on clang64 because of linker issues. Linking the libraries failed with this error:

ld.lld: error: libc++.dll.a(libc++.dll): .idata$4 should not refer to special section 0

This appears to be because the GNUstep build process defines -Wl,--whole-archive in target.make. Prefixing the reference to libc++ with -Wl,--no-whole-archive manages to work around this.

I'm not sure if this is a bug in libc++.dll.a; .idata$4 appears to be empty:

# objdump -s -j '.idata$4' /clang64/lib/libc++.dll.a  | grep idata -A 2
Contents of section .idata$4:
 0000 00000000 00000000                    ........

#21017 (comment) has some more details.

@qmfrederik qmfrederik changed the title gnustep-base: Build on clang64 gnustep-base, gnustep-gui: Build on clang64 Sep 9, 2024
@qmfrederik qmfrederik changed the title gnustep-base, gnustep-gui: Build on clang64 gnustep-base, gnustep-gui, gnustep-back, preferencepanes: Build on clang64 Sep 10, 2024
@qmfrederik qmfrederik changed the title gnustep-base, gnustep-gui, gnustep-back, preferencepanes: Build on clang64 gnustep-base, gnustep-gui, gnustep-back: Build on clang64 Sep 10, 2024
@Biswa96 Biswa96 merged commit 6153b7b into msys2:master Sep 15, 2024
8 checks passed
@qmfrederik qmfrederik deleted the gnustep-clang64 branch September 15, 2024 08:15
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