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.

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

Install Postgis extension with yum #259

Closed
wants to merge 2 commits into from
Closed

Conversation

a-surfer
Copy link

  1. Just moved the original Debian-based Postgis dependencies from defaults/main.yml to the actual apt task in the tasks/extensions/postgis.yml file
  2. Needed to make a terse variable for the postgis version in defaults/main.yml
  3. Added a yum task in the tasks/extensions/postgis.yml file to install postgis with the right version

anderes and others added 2 commits August 25, 2017 11:03
Make config directory permissions only for owner: 700
Copy link

@erdenezul erdenezul left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you could just delete lines instead of commenting those.

@taybin
Copy link

taybin commented Feb 15, 2018

I'd love to see this merged, but agree about removing the lines instead of commenting them out.

Copy link

@erdenezul erdenezul left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

could you please remove commented lines

@gclough
Copy link
Collaborator

gclough commented Mar 23, 2018

I didn't see this @aal-ssara, @taybin, and @erdenezul ... so I wrote #310 instead. I think it goes a bit further than this one:

  1. Has lookup table for PostgreSQL/PostGIS compatibility, and installs the latest compatible version
  2. Moves the package names out to the vars/Blah.yml files, so it's easier to read
  3. Travis does tests, and installs PostGIS during the CI validation

@aoyawale
Copy link
Contributor

closing this PR since we have not received a response and there is a more later PR for this.

@aoyawale aoyawale closed this Mar 30, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants