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
corrected postgresql support for gdal 3.0+ #80127
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.
Could you also rebase the pull request such that it forms clearly labelled atomic commits? For example:
- gdal: Clean up formatting
- gdal: Fix finding postgresql after 3.0 update
- gdal: Port to Python 3
"--with-expat=${expat.dev}" | ||
"--with-jpeg=${libjpeg.dev}" | ||
"--with-libtiff=${libtiff.dev}" # optional (without largetiff support) | ||
"--with-png=${libpng.dev}" # optional | ||
"--with-poppler=${poppler.dev}" # optional | ||
"--with-libz=${zlib.dev}" # optional | ||
"--with-pg=${postgresql}/bin/pg_config" | ||
"--with-pg=yes" # since gdal 3.0 doesn't use ${postgresql}/bin/pg_config |
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.
There was some weird spacing.
"--with-pg=yes" # since gdal 3.0 doesn't use ${postgresql}/bin/pg_config | |
"--with-pg=yes" # since gdal 3.0 doesn't use ${postgresql}/bin/pg_config |
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.
For my local development I am using nixfmt (https://github.com/serokell/nixfmt), but I didn't want to change the whole file. Reading all of your comments seems like it is inevitable. I will do the adjustments tomorrow.
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.
Took me a lot of time. I had to actually undo all the commits and make it a new as the plugin I was using in VSCode was adding strange comments. Now it should be OK and all commits make sense.
0c355e0
to
f012b14
Compare
thanks for your time!, so clear much ❤️ |
You are welcome. Thank you for dedicating time to this project. I really like it :-) |
Motivation for this change
Configure parameters for PostgreSQL support in GDAL has changes since 3.0, see OSGeo/gdal#1657.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)