From 1c17887e79e82aad54da47550e953591b56ef212 Mon Sep 17 00:00:00 2001 From: Eimhin Laverty Date: Mon, 10 Sep 2018 11:22:04 +0100 Subject: [PATCH] (FM-7348) - OS logic cleanup --- manifests/backports.pp | 27 +++++++++++++-------------- spec/classes/apt_spec.rb | 2 +- 2 files changed, 14 insertions(+), 15 deletions(-) diff --git a/manifests/backports.pp b/manifests/backports.pp index 62927cd..0fa6958 100644 --- a/manifests/backports.pp +++ b/manifests/backports.pp @@ -64,24 +64,23 @@ class apt::backports ( if $key { $_key = $key } - if ($facts['lsbdistid'] == 'Debian' or $facts['lsbdistid'] == 'Ubuntu') { - unless $location { - $_location = $::apt::backports['location'] - } - unless $release { - $_release = "${facts['lsbdistcodename']}-backports" - } - unless $repos { - $_repos = $::apt::backports['repos'] - } - unless $key { - $_key = $::apt::backports['key'] - } - } else { + if (!($facts['lsbdistid'] == 'Debian' or $facts['lsbdistid'] == 'Ubuntu')) { unless $location and $release and $repos and $key { fail(translate('If not on Debian or Ubuntu, you must explicitly pass location, release, repos, and key')) } } + unless $location { + $_location = $::apt::backports['location'] + } + unless $release { + $_release = "${facts['lsbdistcodename']}-backports" + } + unless $repos { + $_repos = $::apt::backports['repos'] + } + unless $key { + $_key = $::apt::backports['key'] + } if $pin =~ Hash { $_pin = $pin diff --git a/spec/classes/apt_spec.rb b/spec/classes/apt_spec.rb index 011286d..f93770e 100644 --- a/spec/classes/apt_spec.rb +++ b/spec/classes/apt_spec.rb @@ -35,7 +35,7 @@ preferences_d = { ensure: 'directory', describe 'apt' do let(:facts) do { - os: { family: 'Debian', name: 'Debian', release: { major: '7', full: '7.0' } }, + os: { family: 'Debian', name: 'Debian', release: { major: '8', full: '8.0' } }, lsbdistid: 'Debian', osfamily: 'Debian', lsbdistcodename: 'wheezy', -- 2.32.3