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
thonny: init at 3.0.0b3 #45632
thonny: init at 3.0.0b3 #45632
Conversation
|
||
buildPythonApplication rec { | ||
pname = "thonny"; | ||
name = "${pname}-${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.
This line can be deleted.
@@ -0,0 +1,46 @@ | |||
{ stdenv, fetchFromBitbucket, python3 }: |
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.
Does the author of this package specify python3 only? If not, can you please make it generic in the python version? The specific version can be specified in top-level/all-packages.nix
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.
The homepage states that it 'comes with Python 3", I could not find any reference of support for Python 2 in the documentation or anywhere else.
src = fetchFromBitbucket { | ||
owner = "plas"; | ||
repo = pname; | ||
#rev = 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 remove comment.
sha256 = "1s3pp97r6p3j81idglnml4faxryk7saszxmv3gys1agdfj75qczr"; | ||
}; | ||
|
||
propagatedBuildInputs = with python3.pkgs; [ jedi pyserial tkinter |
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.
The formatting of this line is not like how these lists are typically formatted in nixpkgs.
@GrahamcOfBorg build thonny |
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: thonny Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: thonny Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: thonny Partial log (click to expand)
|
Motivation for this change
Good Python IDE for beginners.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)