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

Replace deprecated apt::source parameters #675

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dpash
Copy link

@dpash dpash commented Apr 28, 2017

A number of apt:source parameters were deprecated, and have now been
removed. This commit migrates to replacements

  • key_source is replaced with passing a hash to the key parameter.
  • include_src is replaced with passing a hash to the include parameter.
  • required_packages has been replaced with explicitly installing them,
    and adding a dependency between the package and the apt::source.

I'm not sure if the explicit package installation might cause an issue
with other modules doing the same thing.

This will resolve #607

A number of apt:source parameters were deprecated, and have now been
removed. This commit migrates to replacements

* key_source is replaced with passing a hash to the key parameter.
* include_src is replaced with passing a hash to the include parameter.
* required_packages has been replaced with explicitly installing them,
and adding a dependency between the package and the apt::source.

I'm not sure if the explicit package installation might cause an issue
with other modules doing the same thing.
@dpash
Copy link
Author

dpash commented Apr 28, 2017

The replacement for required_packages comes from

https://tickets.puppetlabs.com/browse/MODULES-3323

@cdenneen
Copy link

@garethr any chance to get this merged?

@cdenneen
Copy link

cdenneen commented Jun 1, 2017

@dpash Can you rebase to get tests to pass?
@garethr metadata.json doesn't specify older puppetlabs-apt so this fails with changes merged in few weeks back by @hunner

@ColinHebert
Copy link
Contributor

What's the status of this rebase? cc @dpash

@bittner
Copy link

bittner commented Aug 10, 2017

Any chance to get this rebased, @dpash, to get the tests to pass?

@garethr Can we get this issue fixed, the changes merged and a new release issued? Please! That would be tremendously helpful.

@Jeoffreybauvin
Copy link

Any news about this PR ?

@bittner
Copy link

bittner commented Sep 19, 2017

@Jeoffreybauvin You have to tweet to Puppetlabs (see the reaction on my tweet), this may get noticed. Otherwise there's little chance for getting attraction from the maintainers. Or try to open a support request at Puppetlabs?

cegeka-jenkins pushed a commit to cegeka/puppet-docker that referenced this pull request Apr 17, 2024
(maint) don't skip testing "with download location" on most OS's
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.

Apt::Source and Apt::Key deprecation warnings in garethr-docker v5.2.0
5 participants