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

Install software-properties-common for xenial #605

Merged
merged 1 commit into from
May 3, 2016

Conversation

imphil
Copy link

@imphil imphil commented Apr 29, 2016

Instead of adding every newer release of Ubuntu, let's assume that the
package name that is used today will be used in the future, until the
next change is necessary.

@daenney
Copy link

daenney commented Apr 29, 2016

@tphoney This works for me, I don't think it introduces any regressions but we should run the Beaker tests though.

@imphil
Copy link
Author

imphil commented Apr 29, 2016

I've now also included changes to the README to reflect the implementation and test fixes. The functionality remains unchanged, however.

@daenney
Copy link

daenney commented Apr 29, 2016

@imphil Please squash this into a single commit.

Instead of adding every newer release of Ubuntu, let's assume that the
package name that is used today will be used in the future, until the
next change is necessary.
@imphil
Copy link
Author

imphil commented May 2, 2016

@daenney done

@daenney
Copy link

daenney commented May 2, 2016

@bmjen What about you?

@tphoney
Copy link

tphoney commented May 3, 2016

LGTM Thanks for the work @imphil

@tphoney tphoney merged commit 39870ce into puppetlabs:master May 3, 2016
joshuaspence pushed a commit to joshuaspence/dotfiles that referenced this pull request May 15, 2016
This brings in the following changes which I have deemed to be useful:

  - [[puppetlabs/puppetlabs-apt#604 | #604: Fix version check on 16.04]]
  - [[puppetlabs/puppetlabs-apt#605 | #605: Install `software-properties-common` for `xenial`]]
@LukasAud LukasAud added the bugfix label Jun 6, 2023
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