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
python3.8-hdlparse: Init at 1.0.4 #94961
Conversation
Signed-off-by: Elliott Villars <elliottvillars@gmail.com>
Signed-off-by: Elliott Villars <elliottvillars@gmail.com>
Signed-off-by: Elliott Villars <elliottvillars@gmail.com>
|
||
buildPythonPackage rec { | ||
pname = "hdlparse"; | ||
version = "1.0.4"; |
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.
version = "1.0.4"; | |
version = "1.0.4"; | |
{stdenv,buildPythonPackage, fetchPypi}: | ||
|
||
buildPythonPackage rec { | ||
pname = "hdlparse"; |
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.
two spaces is standard indention
pname = "hdlparse"; | |
pname = "hdlparse"; |
}; | ||
|
||
#This module does not contain any tests. | ||
doCheck = false; |
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.
doCheck = false; | |
doCheck = false; | |
sha256 = "fb6230ed1e7a04a8f82f8d3fb59791d0751ae35e5b8e58dbbf2cbcf100d0d0f2"; | ||
}; | ||
|
||
#This module does not contain any tests. |
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.
#This module does not contain any tests. | |
# This module does not contain any tests. |
@@ -0,0 +1,22 @@ | |||
{stdenv,buildPythonPackage, fetchPypi}: |
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,buildPythonPackage, fetchPypi}: | |
{ stdenv, buildPythonPackage, fetchPypi}: |
@@ -0,0 +1,22 @@ | |||
{stdenv,buildPythonPackage, fetchPypi}: |
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 isn't used
{stdenv,buildPythonPackage, fetchPypi}: | |
{lib,buildPythonPackage, fetchPypi}: |
|
||
#This module does not contain any tests. | ||
doCheck = false; | ||
meta = with stdenv.lib; { |
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.
meta = with stdenv.lib; { | |
meta = with lib; { |
#This module does not contain any tests. | ||
doCheck = false; | ||
meta = with stdenv.lib; { | ||
homepage = https://kevinpt.github.io/hdlparse/; |
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.
rfc 45
homepage = https://kevinpt.github.io/hdlparse/; | |
homepage = "https://kevinpt.github.io/hdlparse/"; |
doCheck = false; | ||
meta = with stdenv.lib; { | ||
homepage = https://kevinpt.github.io/hdlparse/; | ||
description = "Hdlparse is a simple package implementing a rudimentary parser for VHDL and Verilog."; |
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.
Already in context of the package, no period
description = "Hdlparse is a simple package implementing a rudimentary parser for VHDL and Verilog."; | |
description = "Rudimentary parser for VHDL and Verilog"; |
please squash all the commits, should be two: Please make this into a separate commit:
|
Motivation for this change
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)