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

Revert "changed rpaf Configuration Directives: RPAF -> RPAF_" #1446

Merged
merged 1 commit into from
May 3, 2016

Conversation

antaflos
Copy link
Contributor

This reverts commit 8ca7a66.

These changes are completely incompatible with mod_rpaf as provided in
Ubuntu 12.04 and Debian Jessie and Wheezy, and break the Apache
configuration and consequently the Puppet agent run.

I suspect these changes were made to be compatible with one of the
mod_rpaf forks (https://github.com/gnif/mod_rpaf) but I don't see how a
"random" fork should have precedence over, and break modules distributed
on the official channels.

This reverts commit 8ca7a66.

These changes are completely incompatible with mod_rpaf as provided in
Ubuntu 12.04 and Debian Jessie and Wheezy, and break the Apache
configuration and consequently the Puppet agent run.

I suspect these changes were made to be compatible with one of the
mod_rpaf forks (https://github.com/gnif/mod_rpaf) but I don't see how a
"random" fork should have precedence over, and break modules distributed
on the official channels.
@antaflos
Copy link
Contributor Author

antaflos commented May 2, 2016

This is also problematic for FreeBSD.

@janvonde
Copy link

janvonde commented May 3, 2016

+1

@bmjen
Copy link
Contributor

bmjen commented May 3, 2016

Thanks for catching and fixing this @antaflos .

@bmjen bmjen merged commit 400f363 into puppetlabs:master May 3, 2016
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.

4 participants