Skip to content

pijul: 0.4.4 -> 0.6.0 #26527

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

Merged
merged 1 commit into from
Jun 12, 2017
Merged

pijul: 0.4.4 -> 0.6.0 #26527

merged 1 commit into from
Jun 12, 2017

Conversation

FlorentBecker
Copy link
Contributor

Motivation for this change

Update pijul to the latest version

Things done
  • Tested using sandboxing
    (nix.useSandbox on NixOS,
    or option build-use-sandbox in nix.conf
    on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • Linux
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

Sorry, something went wrong.

tar -xf ${name}.tar.gz --transform=s,^,${name}/,
tar -cf ${name}.tar.gz ${name}
rm -rf ${name}
'';
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did you try just sourceRoot = "."; instead of repacking?

I do agree that it would be nice if the upstream tarball was what you obtain by repacking, though.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

0.7 should have a "nice" tarball, so it's only a temporary measure.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well, sourceRoot would be a simpler temporary measure, in my opinion.


buildInputs = [ perl ]++ stdenv.lib.optionals stdenv.isDarwin
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool. Now it becomes really lighter then git.

@7c6f434c 7c6f434c merged commit 36728c6 into NixOS:master Jun 12, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants