Fix for Issue #653 - Wrong ci-repo URL for 'master' #863
Conversation
|
Thanks. I was about to merge this, but I think this needs fixing for Debian too. |
bastelfreak
added
needs-rebase
bug
| @@ -2,10 +2,17 @@ | |||
|
|
|||
| if $::collectd::ci_package_repo { | |||
|
|
|||
| # Fix for issue #653 | |||
| if $::collectd::ci_package_repo != 'master' { | |||
There was a problem hiding this comment.
please don't add :: directly after $
|
Dear @sgi-tschaller, thanks for the PR! This is pccibot, your friendly Vox Pupuli GitHub Bot. I noticed that your pull request contains merge conflict. Can you please rebase? You can find my sourcecode at voxpupuli/vox-pupuli-tasks |
|
Dear @sgi-tschaller, thanks for the PR! This is pccibot, your friendly Vox Pupuli GitHub Bot. I noticed that your pull request contains merge conflict. Can you please rebase? You can find my sourcecode at voxpupuli/vox-pupuli-tasks |
|
Hi, I'm going to close this PR due to inactivity. Please reopen it if you're still interested. |
Many thanks for the puppet-collectd module. It is magnificent, and I hope that you find this PR worthy. :)
Pull Request (PR) description
when the ci_package_repo is set to 'master' an incorrect 'collectd-' was prepended. This patch changes that.
This Pull Request (PR) fixes the following issues
Fixes #653