-
-
Notifications
You must be signed in to change notification settings - Fork 13.7k
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
remarshal: 0.3.0 -> 0.6.0 #21523
remarshal: 0.3.0 -> 0.6.0 #21523
Conversation
@peterhoeg, thanks for your PR! By analyzing the history of the files in this pull request, we identified @offlinehacker, @kamilchm, @pSub and @FRidh to be potential reviewers. |
@@ -5868,7 +5868,7 @@ in { | |||
sha256 = "1v9j9fmf8g911yg6k01xa2db6dx3wv73zkk7fncsj7vagjqgs20l"; | |||
}; | |||
|
|||
propagatedBuildInputs = with self; [ self.six ]; | |||
propagatedBuildInputs = with self; [ six ]; |
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.
Please remove this unrelated change as it can cause trouble in the future with cherry-picking.
checkPhase = "${python.interpreter} test/test.py"; | ||
|
||
# fetchgit used to ensure test submodule is available | ||
src = pkgs.fetchgit { |
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.
pkgs.fetchFromGitHub
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.
fetchFromGitHub
retrieves the tarball. We need a proper git checkout in order to get the submodule.
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.
ah, its a git submodule. OK, then it makes sense :)
Motivation for this change
remarshal: 0.3.0 -> 0.6.0
upstream rewrote the application in python.
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/
)