Conversation
|
@ofborg build geos python3Packages.shapely |
|
I am running nixpkgs-review now ... |
|
nixpkgs-review is still running ... Building tensorflow. |
|
After many hours, nixpkgs-review crashed. I am not sure why. From logs, I see following packages having problems with GEOS 3.12.1:
|
|
Error in |
|
I will take a look at |
|
Errors in
But there are other errors as well: which are starting to look like ones that are not worth patching in I am wondering if this is the point at which either
Feedback appreciated... Rough patch for |
Result of 11 packages marked as broken and skipped:
20 packages failed to build:
216 packages built:
|
|
I would suggest adding a new derivation for GEOS 3.11 (much like 3.9 for other legacy uses: https://github.com/NixOS/nixpkgs/blob/master/pkgs/development/libraries/geos/3.9.nix) and shifting pygeos to use that as a dependency (and perhaps note either in top-level or the 3.11 derivation that this is being maintained solely for pygeos). That way pygeos can keep living for users who can't migrate to shapely, but we don't have to hold up the version bump for everyone else. I called that 3.9 version |
@nialov , thank you very much for this investigation. As I said in previous comment, we can keep GEOS 3.11 for some time. After that we deprecate pygeos. |
Great thanks for psycopg fix. I'll add it to our psycopg package once it is ready. |
Keep old version of package due to requirements of following packages: * kikit * pygeos
Keep old version of package due to requirements of following packages: * kikit
|
@ofborg build kikit python3Packages.pygeos |
|
Thanks for the responses, keeping |
|
|
Use maintained version of shapely 1.8 instead of locally packaged version.
|
@ofborg build kikit python3Packages.pygeos |
Result of 5 packages marked as broken and skipped:
8 packages failed to build:
215 packages built:
|
|
@NixOS/geospatial , PR finally looks good now. Please review. Thanks. |
Result of 5 packages marked as broken and skipped:
212 packages built:
|
|
@willcohen , thanks for review. Anybody else interested in review or we can merge now ? |
|
@sikmir , if you are OK with this PR, would you please merge ? Thank you. |
|
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
Description of changes
Update GEOS to the latest version.
List of changes:
This PR was successfully tested with current Shapely version (2.0.2).
This PR also contains following changes:
Things done
nix.conf? (See Nix manual)sandbox = relaxedsandbox = truenix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/)