-
-
Notifications
You must be signed in to change notification settings - Fork 14k
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
go: add mips64 support #161165
go: add mips64 support #161165
Conversation
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.
Commit message should be go: add mips64 support
https://github.com/NixOS/nixpkgs/blob/master/CONTRIBUTING.md#submitting-changes
Commit message is still wrong. |
sorry, missed that. fixed now. |
Does this depend on some of your other PRs to work? |
No. It isn't terribly useful without the rest of the support for mips64. However unlike all the other patches in the series this one can be merged ahead of the one that creates stuff like |
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.
Let's wait until it works and is useful before merging so we aren't adding cruft.
@ofborg eval |
This PR translates nix's system name for mips64el into golang's system names, so golang can be built targeting mips64el.
Motivation for this change
Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)nixos/doc/manual/md-to-db.sh
to update generated release notes