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

MODULES-2990: Gentoo - fix module includes in portage::makeconf #1337

Merged
merged 1 commit into from
Jan 20, 2016

Conversation

derdanne
Copy link
Contributor

https://tickets.puppetlabs.com/browse/MODULES-2990

update module handling for gentoo when used apache 2.4
add module authnz_external as external package in gentoo

add module authnz_external as external package in gentoo
@derdanne derdanne changed the title Gentoo - fix module includes in portage::makeconf MODULES-2990: Gentoo - fix module includes in portage::makeconf Jan 19, 2016
@jonnytdevops
Copy link
Contributor

Hi There

Thanks for this PR.

Unfortunately, Gentoo is not a supported or compatible operating system for this module. To bring a module up to a "compatible" level for a particular OS, it would need a reference in the Readme, have some level of testing, as well as be included in the metadata. Also, all other features would need to work with the OS, or at least the "main" features.

All of these things would need to be done before we can merge this PR.

Thanks

@derdanne
Copy link
Contributor Author

i added the support last year, and it is in the README (#957) ;)

I see, it is missing in metadata.json - maybe we should add it there too.

jonnytdevops added a commit that referenced this pull request Jan 20, 2016
MODULES-2990: Gentoo - fix module includes in portage::makeconf
@jonnytdevops jonnytdevops merged commit 95694e6 into puppetlabs:master Jan 20, 2016
@jonnytdevops
Copy link
Contributor

Hi There

Happy to merge this given you are maintaining the unofficial gentoo support. Apologies that I didn't notice this. Perhaps an update to the Readme to state the unofficial support for gentoo would be cool :)

Thanks

JT

@derdanne
Copy link
Contributor Author

Hi,

In the README is written:

This module also provides functions for other distributions and operating systems, such as FreeBSD, Gentoo, and Amazon Linux, but is not formally tested on them and are subject to regressions.

I think this should be sufficient, what do you think?

@jonnytdevops
Copy link
Contributor

@derdanne I missed that too! I was looking in the limitations section which I don't think makes a reference to Gentoo?

Thanks

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.

3 participants