Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Fix PostGIS install #310

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Fix PostGIS install #310

wants to merge 2 commits into from

Conversation

gclough
Copy link
Collaborator

@gclough gclough commented Mar 22, 2018

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

This was referenced Mar 23, 2018
Closed
Closed
Merged
Closed
Closed
@gclough
Copy link
Collaborator Author

gclough commented Mar 27, 2018

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...

@aoyawale aoyawale mentioned this pull request Mar 30, 2018
Closed
@aoyawale
Copy link
Contributor

looks like this PR also failing for me cause installing epel-release is not done before this is installed. I opened this issue #326 so we can make that change. I think we should fix that first.

@aoyawale aoyawale mentioned this pull request Mar 31, 2018
Merged
@gclough gclough force-pushed the fix_postgis_on_centos branch 2 times, most recently from 90fb6e0 to a261876 Compare April 3, 2018 15:57
@gclough gclough closed this Apr 3, 2018
@gclough gclough reopened this Apr 3, 2018
@gclough gclough force-pushed the fix_postgis_on_centos branch 2 times, most recently from 49afdfb to 59c1a4c Compare April 3, 2018 23:07
gclough referenced this pull request in gclough/postgresql Apr 3, 2018
Merge pull request ANXS#299 from jlozadad/jlozadad_handlers
7b75bec 
added the use of handlers
This was referenced Apr 4, 2018
Closed
Closed
@aoyawale aoyawale closed this Apr 23, 2018
@aoyawale aoyawale reopened this Apr 23, 2018
@aoyawale
Copy link
Contributor

aoyawale commented Apr 23, 2018 •
edited

need git rebase @gclough please go through all your PR's and git rebase. Can't do nothing until the conflicts are fixed.

@gclough gclough force-pushed the fix_postgis_on_centos branch 4 times, most recently from a683ea5 to 9aa5605 Compare April 25, 2018 09:42
Merge branch 'master' of https://github.com/ANXS/postgresql into fix_…
02f5592 
…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
@gclough
Copy link
Collaborator Author

gclough commented Apr 25, 2018

@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.

@aoyawale
Copy link
Contributor

besides git rebase what else is needed here ?

@github-actions GitHub Actions
Copy link

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!

@github-actions github-actions bot added the stale label Apr 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
stale
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

POSTGIS task fails
2 participants
@gclough @aoyawale