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
shell_cmds: init at 198 #21133
shell_cmds: init at 198 #21133
Conversation
@matthewbauer, thanks for your PR! By analyzing the history of the files in this pull request, we identified @copumpkin, @pikajude and @LnL7 to be potential reviewers. |
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.
looks good, apart from the osx version
@@ -40,6 +40,7 @@ let | |||
basic_cmds = "55"; | |||
adv_cmds = "163"; | |||
file_cmds = "264.1.1"; | |||
shell_cmds = "198"; |
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.
The version of 10.11.6 is shell_cmds-187, I think this one is from 10.12.1.
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.
good catch. should be fixed
--replace "FCBA169514A146D000AA698B /* PBXTargetDependency */," "" \ | ||
--replace "FCBA168B14A146D000AA698B /* PBXTargetDependency */," "" \ | ||
--replace "FCBA165914A146D000AA698B /* PBXTargetDependency */," "" \ | ||
--replace "FCBA168714A146D000AA698B /* PBXTargetDependency */," "" |
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.
Does this generally work between versions?
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.
Sadly, no.
2c91f16
to
83c7660
Compare
Wonderful, thanks! |
Also noticed that |
Motivation for this change
This adds Apple's shell_cmds which contains the following binaries:
These are disabled currently because they don't build:
Things done
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
./result/bin/
)fixes #11707