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
Update bpython to 0.17 #30922
Update bpython to 0.17 #30922
Conversation
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.
platforms is set by buildPythonPackage
.
|
||
propagatedBuildInputs = [ wcwidth ]; | ||
|
||
buildInputs = [ pytest pytestrunner ]; |
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.
checkInputs
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.
done
sha256 = "13fyyx06645ikvmj9zmkixr12kzk1c3a3f9s9i2rvaczjycn82lz"; | ||
}; | ||
|
||
buildInputs = [ mock ]; |
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.
checkInputs
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.
done
name = "${pname}-${version}"; | ||
|
||
# 0.17 is still missing on PyPI, https://github.com/bpython/bpython/issues/706 | ||
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
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.
0.17 is not released to PyPI yet, thats why fetchUrl
is used. Issue is linked above.
|
||
propagatedBuildInputs = [ blessings wcwidth pyte ]; | ||
|
||
buildInputs = [ nose mock pytest ]; |
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.
checkInputs
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.
done
homepage = "https://bpython-interpreter.org/"; | ||
license = licenses.mit; | ||
maintainers = with maintainers; [ flokli ]; | ||
platforms = platforms.all; |
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.
platforms needs to be removed.
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.
done
homepage = https://pypi.python.org/pypi/curtsies; | ||
license = licenses.mit; | ||
maintainers = with maintainers; [ flokli ]; | ||
platforms = platforms.all; |
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.
platforms needs to be removed.
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.
done
homepage = https://github.com/selectel/pyte; | ||
license = licenses.lgpl3; | ||
maintainers = with maintainers; [ flokli ]; | ||
platforms = platforms.all; |
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.
platforms needs to be removed.
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.
done
136ebd2
to
a1761f0
Compare
@FRidh Thanks for your review. I addressed all the problems mentioned. |
@GrahamcOfBorg python3.pkgs.bpython |
Motivation for this change
As mentioned by @adisbladis,
pythonPackages
is a mess and needs to be cleaned up.I bumped the
bpython
package to 0.17, cleaned it up, and addedcurtsies
andpyte
, which are both dependencies ofbpython
(at least now)Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)