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

Introduce support for Gentoo #79

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

NiklausHofer
Copy link
Contributor

Pull Request (PR) description

This PR introduces support for Gentoo. As packaged in net-im/prosody, Prosody uses some path which differ from the ones hardcoded in the module. This PR makes these paths configurable and it introduces the correct defaults for Gentoo.

Additionally, the community modules are packaged for Gentoo and the PR introduces support for that, too.

@NiklausHofer
Copy link
Contributor Author

Sorry, I don't understand how I need to fix the formatting. If anyone could help me with that, I would certainly appreciate it.

Also, this is probably not a very elegant solution in the first place, so if you have a better solution in mind, just let me know.

Copy link
Member

@smortex smortex left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi, and thank you for adding support for Gentoo!

I added a bunch of in-line notes for various issues and improvements.

Feel free to send them as separate commits and ask for help if you are not sure about how to proceed!

manifests/virtualhost.pp Outdated Show resolved Hide resolved
manifests/community_modules.pp Outdated Show resolved Hide resolved
manifests/config.pp Outdated Show resolved Hide resolved
data/common.yaml Outdated Show resolved Hide resolved
data/os/Gentoo.yaml Outdated Show resolved Hide resolved
manifests/virtualhost.pp Outdated Show resolved Hide resolved
manifests/community_modules.pp Outdated Show resolved Hide resolved
@NiklausHofer
Copy link
Contributor Author

Thank you for the extensive feedback. I'll have a closer look on Tuesday evening and update the pull request then.

@smortex smortex added enhancement New feature or request needs-tests labels Sep 19, 2021
Gentoo has the Proosdy community modules packaged, so there is no need to manually manage their installation through Mercurial / Git.
@NiklausHofer
Copy link
Contributor Author

OK, I have rebased on the masterr and implemented the changes you suggested.

Copy link
Member

@smortex smortex left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks 👍 ! A few more in-line notes added!

manifests/virtualhost.pp Show resolved Hide resolved
manifests/config.pp Show resolved Hide resolved
manifests/community_modules.pp Outdated Show resolved Hide resolved
@NiklausHofer
Copy link
Contributor Author

I have incorporated your feedback now.

I did not add the manage_package option though as I don't think it makes much sense here. If someone wants to have the OS package installed but not by Puppet or through Puppet but somewhere else in their code can just not use the community_package module since that is all it does really.

@@ -7,14 +7,27 @@
os_facts
end


Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This blank line makes rubocop unhappy 😉

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

Successfully merging this pull request may close these issues.

2 participants