search

akumria / puppet-postgresql

Manage and install Postgresql databases and users
37 stars 20 forks source link

Don't hardcode a provider for the postgresql server package #6

Closed mattr- closed 11 years ago

mattr- commented 11 years ago

Let Puppet determine the proper provider to use.

akumria commented 11 years ago

Hi Matt,

Thanks for the diff, but I'm confused how this patch helps you.

What problem are you having that this solves?

mattr- commented 11 years ago

It's part of another set of changes I have upcoming that allow this module to be used with non-Debian distorts.

On Sep 2, 2012, at 6:22 PM, Anand Kumria notifications@github.com wrote:

Hi Matt,

Thanks for the diff, but I'm confused how this patch helps you.

What problem are you having that this solves?

— Reply to this email directly or view it on GitHub.

mattr- commented 11 years ago

It doesn't solve a particular problem that I'm having. I did it in an attempt to remove the platform specific stuff from the module so that I could use it on a RedHat or CentOS system.

I have more changes in the works for that as well. Are you interested in supporting this module on multiple operating systems?

On Sep 2, 2012, at 6:22 PM, Anand Kumria notifications@github.com wrote:

Hi Matt,

Thanks for the diff, but I'm confused how this patch helps you.

What problem are you having that this solves?

— Reply to this email directly or view it on GitHubhttps://github.com/akumria/puppet-postgresql/pull/6#issuecomment-8227227.

akumria commented 11 years ago

Hi Matt,

Yes I am interested in having the module work on multiple platforms, but without seeing the rest of the changes you have pending it is hard to see how things all fit together.

Does this need to be merged before you can send a pull request for your additional changes?

Thanks, Anand

akumria commented 11 years ago

It has been 7 months since I've heard anything, or seen any further patches.

Closing.