| 14 Jan 2024 |
|  Kai joined the room. | 19:21:03 |
| 15 Jan 2024 |
 Ivan Mincik (imincik) | qgis and qgis-ltr are currently both broken in master. | 17:10:30 |
| 19 Jan 2024 |
|  @kidanger:kidanger.net joined the room. | 16:28:06 |
| @kidanger:kidanger.net | Hi all!
Small question, a bit related to the proj-data MR, about pyproj.
Because of this patch https://github.com/NixOS/nixpkgs/blob/master/pkgs/development/python-modules/pyproj/001.proj.patch, it is not possible to override the proj data dir through the env var PROJ_DATA. internal_datadir is checked first (well, after the user data), and with the patch the path is always present on the disk and hardcoded.
On way around that is to call pyproj.datadir.set_data_dir from python, but it is inconvenient to change the code because of nix, and also multiprocessing would make things very ugly, as it always does.
Do you think we could adjust the patch to change the search priorities? or should it be changed upstream? | 18:22:13 |
| Ivan Mincik (imincik) |
Do you think we could adjust the patch to change the search priorities?
Sure, we can change the patch. Do you have some suggestions ?
| 18:29:04 |
| Ivan Mincik (imincik) | *
Do you think we could adjust the patch to change the search priorities?
Sure, we can change the patch if it makes a sense. Do you have some suggestions ?
| 18:29:13 |
| @kidanger:kidanger.net | maybe this https://sebsauvage.net/paste/?c8d8b4d37cf621c1#DsbOMlBf2Y9FApjPiA9rXSSLOc1ntPEn7eJLwKSXVSU=
I've just swapped the conditions in datadir.py (and compared to nixpkgs' patch, I had to update the setup.py "patch context") | 18:39:19 |
| @kidanger:kidanger.net | I'll test this for the project I'm packaging, as soon as I find how to override the patch | 18:40:26 |
| Ivan Mincik (imincik) | You can try something like this
nix-shell -p 'proj.overrideAttrs (_: _: { patches = [(fetchpatch {url = "<PATCH-URL>"; hash = "";}) ]; })'
| 19:02:10 |
| Ivan Mincik (imincik) | Sorry, change proj to python3Packages.pyproj | 19:03:23 |
| @kidanger:kidanger.net | it's all good I have something working (tests in nix, with some srtm tiles and proj data fetched for the tests), after so many hours spent today :( | 19:03:39 |
| Ivan Mincik (imincik) | I am happy to review your PR once you have something working. | 19:04:45 |
| @kidanger:kidanger.net | I can PR the patch above yes; but I'm not 100% confident it fits inside the ecosystem, so experts eyes are welcome! And it would be nice to understand what it means for the proj-data PR too. | 19:06:45 |
