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

Fix postgis2 installs, add release_version helper to main cookbook #136

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mage2k
Copy link
Contributor

@mage2k mage2k commented Oct 30, 2013

This is for DATA-287!

Emerge postgis manually

This avoids getting the binaries that may have been against the wrong Postgres
version. Still need to emerge gdal via binary.

some minor formatting cleanup

adding lib file to main cookbook with release_version helper function

use different gdal ebuild for pre/post stable-v4

…okbook

Emerge postgis manually

This avoids getting the binaries that may have been against the wrong Postgres
version.  Still need to emerge gdal via binary.

some minor formatting cleanup

adding lib file to main cookbook with release_version helper function

use different gdal ebuild for pre/post stable-v4
@dennisjbell
Copy link
Contributor

Eric,

Can you change release_version to distro, and use gentoo2009 vs gentoo12dot11 or similar -- the stable-v2/stable-v4 monikers are going to be phased out in the new year.

-Dennis

@mage2k
Copy link
Contributor Author

mage2k commented Dec 19, 2013

Dennis,

So, this?

class Chef
  class Recipe
    def distro
      release = File.read('/etc/engineyard/release')
      return "gentoo12dot11" if release.to_i > 2009
      "gentoo2009"
    end
  end
end

@dennisjbell
Copy link
Contributor

Erik,

That looks perfect, with the corresponding changes in the calls in cookbooks/postgresql9_extensions/recipes/ext_postgis2_install.rb to go along with it, of course.

Thanks,
-Dennis

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.

2 participants