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
ydiff: init at 1.1 #45323
ydiff: init at 1.1 #45323
Conversation
There is no homepage (in meta) ? |
Added the GitHub page. With fetchFromGitHub that would have been automatic, but of course that doesn't hold. |
@GrahamcOfBorg build ydiff |
Success on x86_64-linux (full log) Attempted: ydiff Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: ydiff Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: ydiff Partial log (click to expand)
|
Are you sure the tests get run? |
The tests weren't being run, and for completeness sake I've added them. |
checkInputs = [ coverage coreutils git subversion which ]; | ||
|
||
checkPhase = lib.optional (!isPy3k) "make test" ++ | ||
lib.optional (isPy3k) "make test3"; |
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.
You can just use
checkPhase = if isPy3k then ''
make test3
'' else ''
make test
''
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.
Now using checkTarget, as suggested.
@GrahamcOfBorg build ydiff |
Failure on aarch64-linux (full log) Attempted: ydiff Partial log (click to expand)
|
Failure on x86_64-linux (full log) Attempted: ydiff Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: ydiff Partial log (click to expand)
|
f379c61
to
82d80c7
Compare
propagatedBuildInputs = [ less patchutils ]; | ||
checkInputs = [ coverage coreutils git subversion which ]; | ||
|
||
checkPhase = if isPy3k then ''make test3'' else ''make test''; |
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.
Sorry to nitpick, but if it's in a single line, then the ''
quotes aren't needed:
{
checkPhase = if isPy3k then "make test3" else "make test";
}
:D
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.
Indeed, so let's shave off those two extra characters...
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.
BTW: There is something rewarding with a tool that you can use on the commit itself.
ydiff -l -s
@GrahamcOfBorg build ydiff |
Failure on aarch64-linux (full log) Attempted: ydiff Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: ydiff Partial log (click to expand)
|
Failure on x86_64-linux (full log) Attempted: ydiff Partial log (click to expand)
|
86aef4f
to
e71cff0
Compare
Can't you test this locally? |
Annoyingly, it builds on my system - I only push when it does that. But then there is different behaviour on the CI I'm trying to fix. Apologies! |
Try enabling the sandbox for your builds |
propagatedBuildInputs = [ less patchutils ]; | ||
checkInputs = [ coverage coreutils git subversion which ]; | ||
|
||
checkPhase = if isPy3k then "make test3" else "make test"; |
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.
You can use checkTarget instead of checkPhase 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.
@matthewbauer: thanks, didn't know about that. Did that.
Motivation for this change
Handy tool for working with diffs.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)