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
gitAndTools.git-fast-export: 190107 -> 200213, fix #87227
Conversation
@GrahamcOfBorg build gitAndTools.git-fast-export |
fixing involves providing it with mercurial_4 (and implicitly python27 - current released fast-export doesn't support py3 yet) and adding git to the wrapped PATH. this shouldn't be a runtime-resolved dependency.
93c4127
to
794c4d7
Compare
794c4d7
to
2caafd5
Compare
$out/bin/hg-fast-export.sh -r ../repo-hg/ --hg-hash | ||
for s in "foo" "bar" "baz" ; do | ||
(${git}/bin/git show | grep $s > /dev/null) && echo $s found | ||
done |
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.
Do we need to popd
here?
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.
Sure, I'll add it for symmetry.
this creates and converts a simple hg repository - should stop us inadvertently breaking this package.
2caafd5
to
b91ad42
Compare
LGTM with the caveat that I've carefully read but have not actually run the code |
Good enough for me, seeing as I clearly was the first person to try using this in n months. |
👍 That's how I got roped into this last year, too (#60653)... |
Motivation for this change
Turns out this was broken, probably by a mercurial bump to a py3k version. Fixed & bumped (still no py3k support in a released version though). Added an
installCheckPhase
so this doesn't happen again.Would be good to get a working version of this out (maybe to stable?) what with the bitbucket mercurial shutdown approaching.
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)