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
openspin: init at 2018-10-02 #46753
openspin: init at 2018-10-02 #46753
Conversation
@GrahamcOfBorg build openspin |
Failure on x86_64-darwin (full log) Attempted: openspin Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: openspin Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: openspin Partial log (click to expand)
|
The darwin stdenv does not use gcc but clang. The upstream makefile hardcodes these https://github.com/parallaxinc/OpenSpin/blob/master/Makefile#L16 so a substitution is required. |
Using |
@GrahamcOfBorg build openspin |
Failure on x86_64-darwin (full log) Attempted: openspin Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: openspin Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: openspin Partial log (click to expand)
|
@GrahamcOfBorg build openspin |
Success on x86_64-linux (full log) Attempted: openspin Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: openspin Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: openspin Partial log (click to expand)
|
98bdf82
to
da56fc8
Compare
@GrahamcOfBorg build openspin |
Success on x86_64-linux (full log) Attempted: openspin Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: openspin Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: openspin Partial log (click to expand)
|
@GrahamcOfBorg build openspin |
Success on x86_64-darwin (full log) Attempted: openspin Partial log (click to expand)
|
Ok - it should now be good to go. Just need confirmation from @jtojnar regarding how I dealt with the upstream hard-coded compiler issue. |
@GrahamcOfBorg build openspin |
Success on x86_64-darwin (full log) Attempted: openspin Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: openspin Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: openspin Partial log (click to expand)
|
Could you open an issue upstream and add a comment with a link next to the |
Even better, I've written a PR for them. Going to switch to my repo and back to make sure it resolves the issue. Assuming it does (it should), I'll publish. Making WIP. |
@GrahamcOfBorg build openspin |
Success on x86_64-darwin (full log) Attempted: openspin Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: openspin Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: openspin Partial log (click to expand)
|
They accepted my PR - I'm updated to their latest commit. Thanks, Red |
@GrahamcOfBorg build openspin |
sha256 = "1knkbzdanb60cwp7mggymkhd0167lh2sb1c00d1vhw7s0s1rj96n"; | ||
}; | ||
|
||
installPhase = '' |
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.
Maybe you could also try add install target to upstream Makefile.
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.
I'm not comfortable in doing that. They already have their own CI integration that cross-compiles it to multiple platforms. Their CI system then pulls the artifacts out for packaging for end-users.
I don't feel comfortable suggesting that change upstream.
Success on x86_64-linux (full log) Attempted: openspin Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: openspin Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: openspin Partial log (click to expand)
|
Motivation for this change
OpenSpin is an open source compiler for the Parallax Propeller MCU. Additional package for those propeller developers who run on NixOS.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)