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
pythonPackages.vxi11: init at 0.9 #38058
Conversation
pkgs/top-level/python-packages.nix
Outdated
@@ -19253,6 +19253,22 @@ EOF | |||
}; | |||
}; | |||
|
|||
vxi11 = buildPythonPackage rec { |
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.
Python packages should have a dedicated directory in pkgs/development/python-modules
and be included using callPackage
.
pkgs/top-level/python-packages.nix
Outdated
@@ -19253,6 +19253,22 @@ EOF | |||
}; | |||
}; | |||
|
|||
vxi11 = buildPythonPackage rec { | |||
name = "python-vxi11-0.9"; |
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 use pname
and version
instead.
pkgs/top-level/python-packages.nix
Outdated
vxi11 = buildPythonPackage rec { | ||
name = "python-vxi11-0.9"; | ||
|
||
src = pkgs.fetchurl { |
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
pkgs/top-level/python-packages.nix
Outdated
|
||
meta = { | ||
description = "VXI-11 driver for controlling instruments over Ethernet"; | ||
homepage = "https://github.com/python-ivi/python-vxi11"; |
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.
no quotes
Done. |
@GrahamcOfBorg build python2.pkgs.vxi11 python3.pkgs.vxi11 |
Success on x86_64-linux (full log) Attempted: python2.pkgs.vxi11, python3.pkgs.vxi11 Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: python2.pkgs.vxi11, python3.pkgs.vxi11 Partial log (click to expand)
|
Please specify an appropriate checkPhase, e.g.
or set |
Done. Thanks again for the reviews! |
@GrahamcOfBorg build python2.pkgs.vxi11 python3.pkgs.vxi11 |
Success on x86_64-linux (full log) Attempted: python2.pkgs.vxi11, python3.pkgs.vxi11 Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: python2.pkgs.vxi11, python3.pkgs.vxi11 Partial log (click to expand)
|
Motivation for this change
This is a useful package for interfacing with test and measurement equipment.
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)