Code Review
/
puppet-modules
/
puppetlabs-apt.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
review
|
tree
raw
|
inline
| side by side
source: Support complex pin, like key does.
[puppet-modules/puppetlabs-apt.git]
/
manifests
/
ppa.pp
diff --git
a/manifests/ppa.pp
b/manifests/ppa.pp
index 4a08dce6fd413134590962f8ad11091b171f5958..5d4c8903d5c6d9968c64e255f63cc61466554326 100644
(file)
--- a/
manifests/ppa.pp
+++ b/
manifests/ppa.pp
@@
-1,17
+1,17
@@
# ppa.pp
define apt::ppa(
$ensure = 'present',
# ppa.pp
define apt::ppa(
$ensure = 'present',
- $release = $::lsbdistcodename,
$options = $::apt::ppa_options,
$options = $::apt::ppa_options,
+ $release = $::apt::xfacts['lsbdistcodename'],
$package_name = $::apt::ppa_package,
$package_manage = false,
) {
$package_name = $::apt::ppa_package,
$package_manage = false,
) {
-
if !
$release {
+
unless
$release {
fail('lsbdistcodename fact not available: release parameter required')
}
fail('lsbdistcodename fact not available: release parameter required')
}
- if $::
operatingsystem != 'U
buntu' {
- fail('apt::ppa is currently supported on Ubuntu only.')
+ if $::
apt::distid != 'u
buntu' {
+ fail('apt::ppa is currently supported on Ubuntu
and LinuxMint
only.')
}
$filename_without_slashes = regsubst($name, '/', '-', 'G')
}
$filename_without_slashes = regsubst($name, '/', '-', 'G')
@@
-60,9
+60,4
@@
define apt::ppa(
notify => Exec['apt_update'],
}
}
notify => Exec['apt_update'],
}
}
-
- # Need anchor to provide containment for dependencies.
- anchor { "apt::ppa::${name}":
- require => Class['apt::update'],
- }
}
}