Merged
Conversation
Mic92
changed the title
GrahamcOfBorg
added
the
6.topic: golang
Member
Author
|
|
GrahamcOfBorg
added
10.rebuild-darwin: 101-500
Member
Author
|
It works! |
Mic92
commented
Nov 20, 2018
Member
Author
|
buildGoPackage still needs a fix to not put all binaries into a subdirectory: |
Member
Author
|
The cross-compiled go compiler suffers from the same problem: @Ericson2314 I am pretty sure there is an easy fix to distinguish between a go compiler |
|
Success on aarch64-linux (full log) Attempted: go_1_11 Partial log (click to expand)
|
|
Success on x86_64-linux (full log) Attempted: go_1_11 Partial log (click to expand)
|
Member
Author
|
|
Mic92
commented
Nov 20, 2018
Member
Author
There was a problem hiding this comment.
I wonder if we should remove the prefix already in the buildPhase in case someone breaks it by overriding installPhase?
|
Success on x86_64-darwin (full log) Attempted: go_1_11 Partial log (click to expand)
|
|
Success on aarch64-linux (full log) Attempted: go_1_11 Partial log (click to expand)
|
|
Success on x86_64-darwin (full log) Attempted: go_1_11 Partial log (click to expand)
|
|
Success on x86_64-linux (full log) Attempted: go_1_11 Partial log (click to expand)
|
Member
Author
|
Also support C bindings like a charm! |
Member
Author
|
@GrahamcOfBorg build pkgsCross.aarch64-multiplatform.direnv |
Mic92
commented
Nov 20, 2018
|
Unexpected error: command failed with exit code 1 on x86_64-linux (full log) Attempted: pkgsCross.aarch64-multiplatform.direnv Partial log (click to expand)
|
|
Success on aarch64-linux (full log) Attempted: pkgsCross.aarch64-multiplatform.direnv Partial log (click to expand)
|
|
Failure on aarch64-linux (full log) Attempted: go_1_11 Partial log (click to expand)
|
|
Timed out, unknown build status on x86_64-darwin (full log) Attempted: go_1_11 Partial log (click to expand)
|
|
Success on aarch64-linux (full log) Attempted: go_1_11 Partial log (click to expand)
|
|
Success on x86_64-linux (full log) Attempted: go_1_11 Partial log (click to expand)
|
Member
Author
|
@GrahamcOfBorg build direnv |
Member
Author
|
@GrahamcOfBorg build pkgsCross.aarch64-multiplatform.direnv |
|
Success on aarch64-linux (full log) Attempted: direnv Partial log (click to expand)
|
|
Success on aarch64-linux (full log) Attempted: pkgsCross.aarch64-multiplatform.direnv Partial log (click to expand)
|
|
Success on aarch64-linux (full log) Attempted: go_1_11 Partial log (click to expand)
|
|
Success on x86_64-linux (full log) Attempted: pkgsCross.aarch64-multiplatform.direnv Partial log (click to expand)
|
|
Success on x86_64-linux (full log) Attempted: go_1_11 Partial log (click to expand)
|
|
Timed out, unknown build status on x86_64-darwin (full log) Attempted: go_1_11 Partial log (click to expand)
|
|
Timed out, unknown build status on x86_64-darwin (full log) Attempted: direnv Partial log (click to expand)
|
Mic92
commented
Nov 22, 2018
Member
Author
There was a problem hiding this comment.
@dtzWill is cross compilation for pkgsCross.musl64 something we aim for?
The go compiler would break in this case because ${GOHOSTOS}_${GOHOSTARCH} is equal to
${GOOS}_${GOARCH}, which means it would delete the go binaries in that case.
Member
There was a problem hiding this comment.
You could probably make make the condition GOOS != GOHOSTOS etc. since go doesn’t use a libc it’s kind of irrelevant here… but it should still work in the sense that you have a working compiler.
Member
Author
There was a problem hiding this comment.
It does link against glibc for things like gethostbyname via nss modules.
|
Timed out, unknown build status on x86_64-darwin (full log) Attempted: go_1_11 Partial log (click to expand)
|
|
Success on x86_64-linux (full log) Attempted: direnv Partial log (click to expand)
|
|
Failure on x86_64-darwin (full log) Attempted: pkgsCross.aarch64-multiplatform.direnv Partial log (click to expand)
|
10 tasks
|
Success on x86_64-linux (full log) Attempted: go_1_11 Partial log (click to expand)
|
|
Timed out, unknown build status on x86_64-darwin (full log) Attempted: go_1_11 Partial log (click to expand)
|
|
Success on aarch64-linux (full log) Attempted: go_1_11 Partial log (click to expand)
|
Mic92
commented
Nov 29, 2018
|
|
||
| # {CC,CXX}_FOR_TARGET must be only set for cross compilation case as go expect those | ||
| # to be different from CC/CXX | ||
| CC_FOR_TARGET = if (stdenv.hostPlatform != stdenv.targetPlatform) then |
Member
There was a problem hiding this comment.
That comment was pending and I forgot to submit it.
Member
There was a problem hiding this comment.
Putting my thing another way, you can do either way (manual var or depsBuildBuild) the same way for !(build == host == target)
Ericson2314
reviewed
Nov 30, 2018
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Motivation for this change
Mentally prepare myself to cross-compile rust.
Things that work:
Things done
sandboxinnix.confon non-NixOS)nix-shell -p nox --run "nox-review wip"./result/bin/)nix path-info -Sbefore and after)