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

Use the correct package names on Debian #88

Merged
merged 1 commit into from May 4, 2017
Merged

Conversation

ekohl
Copy link
Member

@ekohl ekohl commented Apr 29, 2017

This also allows using jessie-backports. Replaces #76.

@ekohl
Copy link
Member Author

ekohl commented Apr 30, 2017

I also used the following apt config to pull the right packages in. Not sure it belongs in this module, maybe just in the README?

include ::apt
include ::apt::backports
apt::pin { 'jessie-backports-letsencrypt':
  release  => 'jessie-backports',
  packages => prefix(['acme', 'cryptography', 'openssl', 'psutil', 'setuptools', 'pyasn1', 'pkg-resources'], 'python-'),
  priority => 700,
}

@oranenj
Copy link
Contributor

oranenj commented Apr 30, 2017

@ekohl: Can you add that to the README? (with syntax fixed, please :))

What's the 'prefix-' for though?

@ekohl
Copy link
Member Author

ekohl commented Apr 30, 2017

The prefix() comes from stdlib. The actual package names are python-acme, python-cryptography but this resulted in a shorted line so puppet-lint is happy. I can go with the explicit version if that would be preferred.

@oranenj
Copy link
Contributor

oranenj commented Apr 30, 2017

@ekohl: In your code snippet you were adding a 'prefix-' prefix to the packages instead of 'python-' though :)

Using prefix() in the example is fine.

@ekohl
Copy link
Member Author

ekohl commented Apr 30, 2017

@oranenj you're totally right. I guess late night hacking results in bad code sometimes :)

This also allows using jessie-backports
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants