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

[WIP] Fixed apt::ppa path in Ubuntu Wily #586

Closed
wants to merge 1 commit into from

Conversation

cristiklein
Copy link

No clue why, but the filenames that apt-add-repository produces seems to have changed in Wily. This update does the trick for me, but might not be backwards compatible.

No clue why, but the filenames that apt-add-repository produces seems to have changed in Wily. This update does the trick for me, but might not be backwards compatible.
@daenney
Copy link

daenney commented Jan 26, 2016

Test tests tests.

@cristiklein
Copy link
Author

This branch is not ready for merger yet, rather I wanted to discuss how to properly fix the issue. The filename for PPAs sources in /etc/apt/sources.list.d/ has changed in Ubuntu 15.10:

pre-wily: $team-$ppa-$release.list`    (e.g., `fish-shell-release-2-wily.list`)
wily    : $team-$distro-$ppa-$release` (e.g., `fish-shell-ubuntu-release-2-wily.list`)

I think the change was introduced here. This obviously confuses puppetlabs-apt, which triggers add-apt-repository and apt-get update unnecessarily.

Could you advise as to how properly fix this? Should I simply mirror the code in Ubuntu software-properties conditioning it on the release version? Or should I grep the whole /etc/apt/sources.list.d/? What mechanism would be more backwards compatible and more robust to future changes?

@daenney
Copy link

daenney commented Jan 27, 2016

This branch is not ready for merger yet, rather I wanted to discuss how to properly fix the issue.

It would be helpful to mention that when you open a PR. As it stood all we got is a "here's a bug fix" that broke the associated tests. The WIP prefix only tells us so much.

Since the behaviour on Willy and forward includes the -distro- thing we should include that too to keep inline with the new behaviour and not confuse Puppet and the CLI tools. However, do note that you can't hardcode $distro to ubuntu as that will vary.

@imphil
Copy link

imphil commented Feb 4, 2016

Btw, this addresses the same issue as #582.

@daenney
Copy link

daenney commented Feb 4, 2016

Closed in favour of #582

@daenney daenney closed this Feb 4, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants