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
geojson: init at 2.3.0 #40762
geojson: init at 2.3.0 #40762
Conversation
buildPythonPackage rec { | ||
pname = "geojson"; | ||
version = "2.3.0"; | ||
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.
The name
attribute is added by buildPython*
and should therefore be removed.
sha256 = "06ihcb8839zzgk5jcv18kc6nqld4hhj3nk4f3drzcr8n8893v1y8"; | ||
}; | ||
|
||
# test_unicode_properties fails for Python 3 |
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.
Could you post the full error.
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.
Here you are:
======================================================================
ERROR: test_unicode_properties (test_features.FeaturesTest)
----------------------------------------------------------------------
Traceback (most recent call last):
File "/build/geojson-2.3.0/tests/test_features.py", line 41, in test_unicode_properties
obj = geojson.load(file_)
File "/build/geojson-2.3.0/geojson/codec.py", line 43, in load
**kwargs)
File "/nix/store/96wn2gz3mwi71gwcrvpfg39bsymd7gqx-python3-3.6.5/lib/python3.6/json/__init__.py", line 296, in load
return loads(fp.read(),
File "/nix/store/96wn2gz3mwi71gwcrvpfg39bsymd7gqx-python3-3.6.5/lib/python3.6/encodings/ascii.py", line 26, in decode
return codecs.ascii_decode(input, self.errors)[0]
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 28: ordinal not in range(128)
----------------------------------------------------------------------
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 is a common issue described in the Nixpkgs manual:
Unicode issues can typically be fixed by including glibcLocales in
buildInputscheckInputs and exporting LC_ALL=en_US.utf-8.
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 didn't knew about this one. Thanks for your help!
95251f2
to
8cc805c
Compare
8cc805c
to
2db356f
Compare
@GrahamcOfBorg build python.pkgs.geojson python3.pkgs.geojson |
Success on x86_64-darwin (full log) Attempted: python.pkgs.geojson, python3.pkgs.geojson Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: python.pkgs.geojson, python3.pkgs.geojson Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: python.pkgs.geojson, python3.pkgs.geojson Partial log (click to expand)
|
Motivation for this change
Adds the geojson Python-module to nixpkgs
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)