-
Notifications
You must be signed in to change notification settings - Fork 43
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
chore(nix): refactor, fix library packages #1080
base: master
Are you sure you want to change the base?
Conversation
Jenkins BuildsClick to see older builds (7)
|
026364e
to
1c5684d
Compare
For some reason when i try to build
It works just fine within |
""") | ||
} } | ||
} | ||
stage('Check') { | ||
steps { | ||
sh 'ldd ./result/bin/c' | ||
sh 'ldd ./result/bin/libgowaku.a' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See prev comment.
@jakubgs is it possible to run
I'm thinking this |
Or maybe doing something like this works?
|
Thank you for opening this pull request! We require pull request titles and commits to follow the Conventional Commits specification and it looks like your PR needs to be adjusted. Details:
|
cfba0b9
to
c19ac82
Compare
I have tried applying I have also tried |
Yes, I can add this in the build derivation: preBuild = ''
go env GOGCCFLAGS
''; And I can then get:
So it seems ot me like it is there, and yet it still fails with:
|
Bit of a cleanup to make it more readable and also fix building of libraries. Moving the actual build to `default.nix` makes `flake.nix` more readable. Signed-off-by: Jakub Sokołowski <[email protected]>
``` /nix/store/mzrw2zphz9xx9s3qq8x5zpf18isp0jga-go-1.20.3/share/go/pkg/tool/linux_amd64/link: running gcc failed: exit status 1 /nix/store/ybw485608d7f1yv1v071j2052q64mvla-binutils-2.40/bin/ld: /build/go-link-2631521468/go.o: relocation R_X86_64_TPOFF32 against `runtime.tlsg' can not be used when making a shared object; recompile with -fPIC /nix/store/ybw485608d7f1yv1v071j2052q64mvla-binutils-2.40/bin/ld: failed to set dynamic section sizes: bad value collect2: error: ld returned 1 exit status ``` Signed-off-by: Jakub Sokołowski <[email protected]>
Bit of a cleanup to make it more readable and also fix building of libraries.
Moving the actual build to
default.nix
makesflake.nix
more readable.