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.coconut init at 1.4.0 #54952
Conversation
I'll note that when using pypi2nix coconut was able to run without issue. |
pkgs/top-level/python-packages.nix
Outdated
@@ -1259,6 +1261,9 @@ in { | |||
|
|||
pretend = callPackage ../development/python-modules/pretend { }; | |||
|
|||
prompt-toolkit = callPackage ../development/python-modules/prompt-toolkit {}; |
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 exists as prompt_toolkit
.
|
||
meta = with stdenv.lib; { | ||
homepage = "http://coconut-lang.org"; | ||
license = "Apache-2.0"; |
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.
license = "Apache-2.0"; | |
license = licenses.asl20; |
meta = with stdenv.lib; { | ||
homepage = "http://coconut-lang.org"; | ||
license = "Apache-2.0"; | ||
description = "Simple, elegant, Pythonic functional programming."; |
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.
homepage = "http://coconut-lang.org"; | ||
license = "Apache-2.0"; | ||
description = "Simple, elegant, Pythonic functional programming."; | ||
maintainers = with lib.maintainers; [ bbarker ]; |
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.
maintainers = with lib.maintainers; [ bbarker ]; | |
maintainers = with maintainers; [ bbarker ]; |
description = "Simple, elegant, Pythonic functional programming."; | ||
maintainers = with lib.maintainers; [ bbarker ]; | ||
}; | ||
} |
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 add a \n
pyparsing | ||
]; | ||
|
||
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; { |
pname = "coconut"; | ||
version = "1.4.0"; | ||
|
||
src = 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
, pyparsing | ||
}: | ||
|
||
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.
Isn't this an application rather than a library?
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 think it is both (two different ways to call it - as a python import or the command line).
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.
Alright. Then it can stay that way.
sha256 = "b192c71022702b60b4c8c8dac06321a4a82e21c237ba2bb71d2edebb7540b8a9"; | ||
}; | ||
|
||
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.
Why?
@@ -0,0 +1,32 @@ | |||
{ stdenv |
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 | |
{ lib |
pyparsing has been updated to 2.3.0 on master |
06aff3e
to
a08c3e5
Compare
@dotlambda Ah right. In this case, what is the recommended course of action - attempt a patch to see if it will work on 2.3.0? |
eeffce3
to
5e7dae6
Compare
Making an upstream PR and use that with |
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 commits should be squashed and the resulting commit's message should equal this PR's title.
Are there any updates on this pull request, please? |
Nothing happening so closing. |
Motivation for this change
Adding the coconut extension language for python.
Currently not surewhy this version check error occurs, as
nix-shell
seems to pull in the only version ofpyparsing
that might be satisfied (2.2.0):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)