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
opentimestamps: 0.2.1 -> 0.3.0, opentimestamps-client: 0.5.1 -> 0.6.0 #39599
Conversation
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.
I'd prefer 2 seperate commits.
@@ -3,14 +3,14 @@ | |||
|
|||
buildPythonPackage rec { | |||
name = "opentimestamps-${version}"; |
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 specify pname
instead
@@ -3,14 +3,14 @@ | |||
|
|||
buildPythonPackage rec { | |||
name = "opentimestamps-${version}"; | |||
version = "0.2.1"; | |||
version = "0.3.0"; | |||
disabled = (!isPy3k); | |||
|
|||
src = 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.
If possible, please use fetchPypi
instead.
@@ -1,19 +1,19 @@ | |||
{ lib, buildPythonApplication, fetchFromGitHub, isPy3k | |||
, opentimestamps, GitPython, pysocks }: | |||
, opentimestamps, appdirs, GitPython, pysocks }: | |||
|
|||
buildPythonApplication rec { | |||
name = "opentimestamps-client-${version}"; |
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.
pname
|
||
buildPythonApplication rec { | ||
name = "opentimestamps-client-${version}"; | ||
version = "0.5.1"; | ||
version = "0.6.0"; | ||
disabled = (!isPy3k); | ||
|
||
src = 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.
fetchPypi
cca20f7
to
d2c3151
Compare
I'v split the commits. As mentioned in my introductory PR, we can't use pypi for this package. I've added some source comments to clarify this. By the way, here's a usage example to help reviewing:
|
You forgot to use |
d2c3151
to
72f21d3
Compare
Thanks for the hint. |
I'm not sure it's necessary. There's no huge advantage in having no Btw, I think the more important thing to do is moving all packages out of |
@GrahamcOfBorg build python2.pkgs.opentimestamps python3.pkgs.opentimestamps opentimestamps-client @erikarvstedt You should prepend |
Success on x86_64-linux (full log) Attempted: python3.pkgs.opentimestamps, opentimestamps-client The following builds were skipped because they don't evaluate on x86_64-linux: python2.pkgs.opentimestamps Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: python3.pkgs.opentimestamps, opentimestamps-client The following builds were skipped because they don't evaluate on aarch64-linux: python2.pkgs.opentimestamps Partial log (click to expand)
|
72f21d3
to
4e79f2f
Compare
Fixed. Thanks for your patience! |
build-use-sandbox
innix.conf
on non-NixOS)./result/bin/
)