-
Notifications
You must be signed in to change notification settings - Fork 581
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
Fix PostGIS install #310
Fix PostGIS install #310
Conversation
Oh no... @sebalix , that's just made a complete hash of it. The rebase seems to be taking in other changes and merging them into this PR. Back to the drawing board... |
looks like this PR also failing for me cause installing |
90fb6e0
to
a261876
Compare
49afdfb
to
59c1a4c
Compare
need git rebase @gclough please go through all your PR's and git rebase. Can't do nothing until the conflicts are fixed. |
a683ea5
to
9aa5605
Compare
daa2383
to
d233cb9
Compare
…postgis_on_centos Conflicts: defaults/main.yml handlers/main.yml tasks/extensions/contrib.yml tasks/extensions/postgis.yml tests/docker/group_vars/postgresql.yml tests/vars.yml vars/Debian.yml vars/RedHat.yml
@jlozadad , I've merged this to master, but it could definitely do with a review to ensure it's solid. It passes the automated tests, so that's a good sign. |
besides git rebase what else is needed here ? |
This pr has been marked 'stale' due to lack of recent activity. If there is no further activity, the issue will be closed in another 30 days. Thank you for your contribution! |
This pr has been closed due to inactivity. If you feel this is in error, please reopen the issue or file a new issue with the relevant details. |
Fixes #251, and gives sensible defaults (the latest version) of PostGIS for each different version of PostgreSQL
I've also added the extension to the travis tests, so it should get exercised regularly from now on