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
Unbreak paperspace #102424
Unbreak paperspace #102424
Conversation
Result of 1 package failed to build:
11 packages built:
|
1798ee3
to
209d186
Compare
Fixed the broken package and added suggested tests |
Result of 12 packages built:
|
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.
While the current changes work, I'd suggest to disable Python 2 builds instead of fixing them.
@@ -0,0 +1,29 @@ | |||
{ stdenv, fetchPypi, buildPythonPackage |
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.
{ stdenv, fetchPypi, buildPythonPackage | |
{ stdenv, fetchPypi, buildPythonPackage, isPy3k |
buildPythonPackage rec { | ||
pname = "click-help-colors"; | ||
version = "0.8"; | ||
|
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.
disabled = !isPy3k; | |
@@ -0,0 +1,26 @@ | |||
{ lib, stdenv, fetchPypi, buildPythonPackage |
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.
{ lib, stdenv, fetchPypi, buildPythonPackage | |
{ lib, stdenv, fetchPypi, buildPythonPackage, isPy3k |
buildPythonPackage rec { | ||
pname = "gradient_statsd"; | ||
version = "1.0.1"; | ||
|
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.
disabled = !isPy3k; | |
propagatedBuildInputs = [ requests datadog ] | ||
++ lib.optional python.isPy2 configparser; |
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.
propagatedBuildInputs = [ requests datadog ] | |
++ lib.optional python.isPy2 configparser; | |
propagatedBuildInputs = [ requests datadog ]; |
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.
LGTM
Result of nixpkgs-review pr 102424 1
12 packages built:
- python27Packages.click-help-colors
- python27Packages.gradient_statsd
- python37Packages.click-help-colors
- python37Packages.gradient_sdk
- python37Packages.gradient_statsd
- python37Packages.hyperopt
- python37Packages.paperspace
- python38Packages.click-help-colors
- python38Packages.gradient_sdk
- python38Packages.gradient_statsd
- python38Packages.hyperopt
- python38Packages.paperspace
@omasanori he already did the work to make those builds work. But you're right, python2 is probably not a worthwhile platform to support. |
@jonringer Sure, it makes sense to merge anyway. |
Result of 5 packages failed to build:
7 packages built:
Edit: false positive, they build... |
Motivation for this change
Add several python packages to be able to unbreak paperspace
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)