X-Git-Url: https://review.fuel-infra.org/gitweb?a=blobdiff_plain;f=manifests%2Fsource.pp;h=2386c0a5f3b2eae05fdff5d1864f1e78cc0023a9;hb=refs%2Fheads%2Frelease-prep;hp=5fde580c07b875b58c3ed19ad1b86f5c275db71f;hpb=3d3620ccd767ca8c4055f1c84a658c79de31fcb3;p=puppet-modules%2Fpuppetlabs-apt.git diff --git a/manifests/source.pp b/manifests/source.pp index 5fde580..2386c0a 100644 --- a/manifests/source.pp +++ b/manifests/source.pp @@ -80,7 +80,7 @@ define apt::source ( Boolean $notify_update = true, Boolean $check_valid_until = true, ) { - include ::apt + include apt $_before = Apt::Setting["list-${title}"] @@ -98,7 +98,7 @@ define apt::source ( if ! $location { fail('cannot create a source entry without specifying a location') } - elsif ($::apt::proxy['https_acng']) and ($location =~ /(?i:^https:\/\/)/) { + elsif ($apt::proxy['https_acng']) and ($location =~ /(?i:^https:\/\/)/) { $_location = regsubst($location, 'https://','http://HTTPS///') } else { @@ -107,14 +107,14 @@ define apt::source ( # Newer oses, do not need the package for HTTPS transport. $_transport_https_releases = ['9'] if (fact('os.release.major') in $_transport_https_releases) and $_location =~ /(?i:^https:\/\/)/ { - ensure_packages('apt-transport-https') + stdlib::ensure_packages('apt-transport-https') Package['apt-transport-https'] -> Class['apt::update'] } } else { $_location = undef } - $includes = merge($::apt::include_defaults, $include) + $includes = $apt::include_defaults + $include if $key and $keyring { fail('parameters key and keyring are mutualy exclusive') @@ -125,7 +125,7 @@ define apt::source ( unless $key['id'] { fail('key hash must contain at least an id entry') } - $_key = merge($::apt::source_key_defaults, $key) + $_key = $apt::source_key_defaults + $key } else { $_key = { 'id' => assert_type(String[1], $key) } } @@ -140,20 +140,21 @@ define apt::source ( } $sourcelist = epp('apt/source.list.epp', { - 'comment' => $comment, - 'includes' => $includes, - 'options' => delete_undef_values( { - 'arch' => $architecture, - 'trusted' => $allow_unsigned ? { true => 'yes', false => undef }, - 'allow-insecure' => $allow_insecure ? { true => 'yes', false => undef }, - 'signed-by' => $keyring, - 'check-valid-until' => $check_valid_until? { true => undef, false => 'false' }, - }, - ), - 'location' => $_location, - 'release' => $_release, - 'repos' => $repos, - }) + 'comment' => $comment, + 'includes' => $includes, + 'options' => delete_undef_values({ + 'arch' => $_architecture, + 'trusted' => $allow_unsigned ? { true => 'yes', false => undef }, + 'allow-insecure' => $allow_insecure ? { true => 'yes', false => undef }, + 'signed-by' => $keyring, + 'check-valid-until' => $check_valid_until? { true => undef, false => 'false' }, + }, + ), + 'location' => $_location, + 'release' => $_release, + 'repos' => $repos, + } + ) apt::setting { "list-${name}": ensure => $ensure, @@ -163,7 +164,7 @@ define apt::source ( if $pin { if $pin =~ Hash { - $_pin = merge($pin, { 'ensure' => $ensure, 'before' => $_before }) + $_pin = $pin + { 'ensure' => $ensure, 'before' => $_before } } elsif ($pin =~ Numeric or $pin =~ String) { $url_split = split($location, '[:\/]+') $host = $url_split[1]