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
parallel: 20200922 -> 20210122 #111499
parallel: 20200922 -> 20210122 #111499
Conversation
5ba3bb0
to
9b0fa99
Compare
Result of 2 packages built:
|
Since https://git.savannah.gnu.org/cgit/parallel.git/commit/src/parallel?id=ec2026334368e667a2567f1535b5c2d807ddb224 the patch is no longer needed. Thanks @bcc32! |
Squash commits? |
This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch). Result of 1 package marked as broken and skipped:
8 packages built:
|
This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch). Result of 1 package marked as broken and skipped:
1 package failed to build and are new build failure:
2 packages built:
|
@tomberek I was assuming the squash would happen at merge-time. I can also squash eagerly if you would prefer (the extra commit was created using the GitHub accept-suggestion interface). |
Result of 1 package marked as broken and skipped:
8 packages built:
|
By default, I tend to squash commits myself to trim down on the "commit noise". I know some projects allow a squash and merge and I think I've seen some discussion about this for Nixpkgs. I don't have merge permissions, so I'm not sure what is currently enabled. |
It is always nicer to squash them because you can easily forget to squash them.
We have everything enabled. |
2d79377
to
984d339
Compare
Got it, rebased and squashed. |
984d339
to
b9b2db1
Compare
b9b2db1
to
dc90d13
Compare
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.
Updated PR to add coreutils to path in the wrapper.
Thanks! |
This was my first ever merge into Nixpkgs! 🎉 |
Motivation for this change
Updating GNU parallel to the latest version.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)Tested that the patch is no longer necessary, as
parallel -X -j2 --tagstring '{#}' echo ::: $(seq 1 100000)
correctly splits the argument list into several jobs as needed to satisfy the maximum command line length.