Removing openssl package

[andyleejordan]

Again, Prosody automatically includes openssl when installed.

So we shouldn't require it, apt does it for us. Package requirements in Puppet are really iffy, especially for packages that are shared dependencies (such as this). So long as apt pulls it in for us, let's not step on other modules.

If we do have to have it, use ensure_packages([package]).

[fpletz]

Yup, thanks! 👍 This was actually some cruft from the upstream module. There are some more cleanups pending.

[andyleejordan]

For sure, I've been meaning to fork and update that upstream for a long time now. Glad to do it together.

[fpletz]

Awesome! We were also thinking about forking for a long time. Let's do this!

[andyleejordan]

Puppetlab's apt module makes adding repositories so nice. Granted, I currently manage repos/PPAs separately from modules (feel that it's cleaner). But adding the repository conditionally would be fine.

[andyleejordan]

Just an update: all the updates I made work perfectly fine with the latest version (0.9.3-1~precise) of prosody as well.

[andyleejordan]

@fpletz Also, you can go to settings and check "issues" to allow issues on forks now, which would be great for further development :)

[fpletz]

Thanks for the report and the hint with the issues, just enabled them. Feel free to open tickets as you please. :) Need to go to sleep now, though, little bit drunk… 🍻

[andyleejordan]

Sounds good! Thanks man. Have a good night, it is quite late in Germany right now :)