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

(GH-73) Switch to puppetlabs/inifile from cprice/inifile #74

Conversation

kbarber
Copy link
Contributor

@kbarber kbarber commented Jul 31, 2013

cprice/inifile is deprecated, this patch changes the module to use inifile's
new home puppetlabs/inifile.

Signed-off-by: Ken Barber ken@bob.sh

cprice/inifile is deprecated, this patch changes the module to use inifile's
new home puppetlabs/inifile.

Signed-off-by: Ken Barber <ken@bob.sh>
@kbarber-jenkins-bot
Copy link

Merged build triggered. (Status: PENDING, Details: null)

@kbarber-jenkins-bot
Copy link

Merged build started. (Status: PENDING, Details: http://box.bob.sh:8080/job/puppetlabs-puppetdb/152/)

@kbarber-jenkins-bot
Copy link

Merged build finished. (Status: FAILURE, Details: http://box.bob.sh:8080/job/puppetlabs-puppetdb/152/)

@kbarber
Copy link
Contributor Author

kbarber commented Aug 1, 2013

retest this please

@kbarber-jenkins-bot
Copy link

Merged build triggered. (Status: PENDING, Details: null)

@kbarber-jenkins-bot
Copy link

Merged build started. (Status: PENDING, Details: http://box.bob.sh:8080/job/puppetlabs-puppetdb/154/)

@kbarber-jenkins-bot
Copy link

Merged build finished. (Status: SUCCESS, Details: http://box.bob.sh:8080/job/puppetlabs-puppetdb/154/)

@bjoernhaeuser
Copy link

I would love to see this happening :)

@kbarber kbarber mentioned this pull request Sep 5, 2013
@blkperl
Copy link

blkperl commented Sep 28, 2013

👍

@robinbowes
Copy link
Contributor

Are you planning to push a release to the forge with this change in any time soon?

kbarber added a commit that referenced this pull request Oct 4, 2013
…e-deprecated

(GH-73) Switch to puppetlabs/inifile from cprice/inifile
@kbarber kbarber merged commit 73c1d65 into puppetlabs:master Oct 4, 2013
@kbarber
Copy link
Contributor Author

kbarber commented Oct 4, 2013

I have another major change in the pipeline to switch to postgresql 3.0.0 and wanted to hold off for that before release, since this request isn't handled well by the module tool it becomes a major release on its own. No harm in merging this in now though.

@robinbowes
Copy link
Contributor

Are you saying you won't push another major release before the pg3 changes?

@kbarber
Copy link
Contributor Author

kbarber commented Oct 4, 2013

@robinbowes yeah, otherwise it would probably be 2 major releases that way. Is the absence of this change causing a major hassle?

@robinbowes
Copy link
Contributor

Well, I'm using stephenrjohnson/puppet and it uses puppetlabs/inifile and also puppetdb which uses cprice404/inifile. The cprice404 version is the one that gets installed so all subsequent puppet runs get all the deprecation warnings. I currently work round that by manually removing modues/inifile and installing puppetlabs/inifile. It would be nice to not have to. :)

@kbarber
Copy link
Contributor Author

kbarber commented Oct 4, 2013

@robinbowes alright, I'll cut a major release just for this change.

@kbarber
Copy link
Contributor Author

kbarber commented Oct 4, 2013

@robinbowes pr #85 covers the release notes. I'll push to get that merged in and released ASAP.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants