summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMicah Anderson <micah@riseup.net>2010-12-07 13:18:41 -0500
committerMicah Anderson <micah@riseup.net>2010-12-07 13:18:41 -0500
commite15283b781f35779a75415725cd315104879aa67 (patch)
treeb40a8779915a1b77b826181ca80de6886b23cba3
parentbb03c32b4589400bd03d76645db6d8bb869424ed (diff)
format standardization
-rw-r--r--manifests/apt_conf.pp6
-rw-r--r--manifests/cron/base.pp3
-rw-r--r--manifests/dselect.pp1
-rw-r--r--manifests/init.pp33
-rw-r--r--manifests/preferences.pp4
-rw-r--r--manifests/preferences/absent.pp3
-rw-r--r--manifests/preferences_snippet.pp5
-rw-r--r--manifests/preseeded_package.pp2
-rw-r--r--manifests/sources_list.pp5
-rw-r--r--manifests/unattended_upgrades.pp3
-rw-r--r--manifests/upgrade_package.pp1
11 files changed, 35 insertions, 31 deletions
diff --git a/manifests/apt_conf.pp b/manifests/apt_conf.pp
index 62e4377..a18a9bd 100644
--- a/manifests/apt_conf.pp
+++ b/manifests/apt_conf.pp
@@ -1,11 +1,13 @@
define apt::apt_conf(
$ensure = 'present',
$source = '',
- $content = undef
-){
+ $content = undef )
+{
+
if $source == '' and $content == undef {
fail("One of \$source or \$content must be specified for apt_conf ${name}")
}
+
if $source != '' and $content != undef {
fail("Only one of \$source or \$content must specified for apt_conf ${name}")
}
diff --git a/manifests/cron/base.pp b/manifests/cron/base.pp
index 2cfbefb..de55772 100644
--- a/manifests/cron/base.pp
+++ b/manifests/cron/base.pp
@@ -1,3 +1,4 @@
class apt::cron::base {
- package { cron-apt: ensure => installed }
+
+ package { cron-apt: ensure => installed }
}
diff --git a/manifests/dselect.pp b/manifests/dselect.pp
index 52c66cb..45a5ddc 100644
--- a/manifests/dselect.pp
+++ b/manifests/dselect.pp
@@ -1,4 +1,5 @@
class apt::dselect {
+
# suppress annoying help texts of dselect
append_if_no_such_line { dselect_expert:
file => "/etc/dpkg/dselect.cfg",
diff --git a/manifests/init.pp b/manifests/init.pp
index e7eac33..4ca2b2e 100644
--- a/manifests/init.pp
+++ b/manifests/init.pp
@@ -88,23 +88,23 @@ class apt {
}
apt_conf { "02show_upgraded":
- source => ["puppet:///modules/site-apt/${fqdn}/02show_upgraded",
- "puppet:///modules/site-apt/02show_upgraded",
- "puppet:///modules/apt/02show_upgraded"]
+ source => [ "puppet:///modules/site-apt/${fqdn}/02show_upgraded",
+ "puppet:///modules/site-apt/02show_upgraded",
+ "puppet:///modules/apt/02show_upgraded" ]
}
if ( $virtual == "vserver" ) {
apt_conf { "03clean_vserver":
- source => ["puppet:///modules/site-apt/${fqdn}/03clean_vserver",
- "puppet:///modules/site-apt/03clean_vserver",
- "puppet:///modules/apt/03clean_vserver"]
+ source => [ "puppet:///modules/site-apt/${fqdn}/03clean_vserver",
+ "puppet:///modules/site-apt/03clean_vserver",
+ "puppet:///modules/apt/03clean_vserver" ]
}
}
else {
apt_conf { "03clean":
- source => ["puppet:///modules/site-apt/${fqdn}/03clean",
- "puppet:///modules/site-apt/03clean",
- "puppet:///modules/apt/03clean"]
+ source => [ "puppet:///modules/site-apt/${fqdn}/03clean",
+ "puppet:///modules/site-apt/03clean",
+ "puppet:///modules/apt/03clean" ]
}
}
@@ -121,13 +121,9 @@ class apt {
# backward compatibility: upgrade from previous versions of this module.
file {
- ["/etc/apt/apt.conf.d/from_puppet",
- "/etc/apt/apt.conf.d/99from_puppet"
- ]:
+ [ "/etc/apt/apt.conf.d/from_puppet", "/etc/apt/apt.conf.d/99from_puppet" ]:
ensure => 'absent',
- require => [ Apt_conf['02show_upgraded'],
- Apt_conf['03clean'],
- ],
+ require => [ Apt_conf['02show_upgraded'], Apt_conf['03clean'] ];
}
# watch .d directories and ensure they are present
@@ -143,13 +139,12 @@ class apt {
'refresh_apt':
command => '/usr/bin/apt-get update && sleep 1',
refreshonly => true,
- subscribe => [ File['/etc/apt/apt.conf.d'],
- Config_file['/etc/apt/sources.list'] ];
+ subscribe => [ File['/etc/apt/apt.conf.d'], Config_file['/etc/apt/sources.list'] ];
+
'update_apt':
command => '/usr/bin/apt-get update && /usr/bin/apt-get autoclean',
refreshonly => true,
- require => [ File['/etc/apt/apt.conf.d',
- '/etc/apt/preferences'],
+ require => [ File['/etc/apt/apt.conf.d', '/etc/apt/preferences' ],
Config_file['/etc/apt/sources.list'] ],
loglevel => info,
# Another Semaphor for all packages to reference
diff --git a/manifests/preferences.pp b/manifests/preferences.pp
index 4acfdaf..4c93542 100644
--- a/manifests/preferences.pp
+++ b/manifests/preferences.pp
@@ -3,7 +3,7 @@ class apt::preferences {
include common::moduledir
$apt_preferences_dir = "${common::moduledir::module_dir_path}/apt/preferences"
module_dir{'apt/preferences': }
- file{"${apt_preferences_dir}_header":
+ file { "${apt_preferences_dir}_header":
content => $custom_preferences ? {
'' => $operatingsystem ? {
'debian' => template("apt/${operatingsystem}/preferences_${codename}.erb"),
@@ -13,7 +13,7 @@ class apt::preferences {
},
}
- concatenated_file{'/etc/apt/preferences':
+ concatenated_file { '/etc/apt/preferences':
dir => $apt_preferences_dir,
header => "${apt_preferences_dir}_header",
# use Concatenated_file[apt_config] to reference a completed configuration
diff --git a/manifests/preferences/absent.pp b/manifests/preferences/absent.pp
index d8b2e10..0e96119 100644
--- a/manifests/preferences/absent.pp
+++ b/manifests/preferences/absent.pp
@@ -1,5 +1,6 @@
class apt::preferences::absent {
- file {'/etc/apt/preferences':
+
+ file { '/etc/apt/preferences':
ensure => absent,
}
}
diff --git a/manifests/preferences_snippet.pp b/manifests/preferences_snippet.pp
index fd2956d..4b8e575 100644
--- a/manifests/preferences_snippet.pp
+++ b/manifests/preferences_snippet.pp
@@ -2,8 +2,9 @@ define apt::preferences_snippet(
$ensure = 'present',
$source = '',
$release,
- $priority
-){
+ $priority )
+{
+
if $custom_preferences == false {
fail("Trying to define a preferences_snippet with \$custom_preferences set to false.")
}
diff --git a/manifests/preseeded_package.pp b/manifests/preseeded_package.pp
index 76c432e..8f3bcbd 100644
--- a/manifests/preseeded_package.pp
+++ b/manifests/preseeded_package.pp
@@ -5,7 +5,7 @@ define apt::preseeded_package ($content = "", $ensure = "installed") {
default => $content
}
- file{ $seedfile:
+ file { $seedfile:
content => $real_content,
mode => 0600, owner => root, group => root,
}
diff --git a/manifests/sources_list.pp b/manifests/sources_list.pp
index bbf887a..ff8f21a 100644
--- a/manifests/sources_list.pp
+++ b/manifests/sources_list.pp
@@ -1,8 +1,9 @@
define apt::sources_list (
$ensure = 'present',
$source = '',
- $content = undef
-) {
+ $content = undef )
+{
+
if $source == '' and $content == undef {
fail("One of \$source or \$content must be specified for apt_sources_snippet ${name}")
}
diff --git a/manifests/unattended_upgrades.pp b/manifests/unattended_upgrades.pp
index f52448d..e2992f8 100644
--- a/manifests/unattended_upgrades.pp
+++ b/manifests/unattended_upgrades.pp
@@ -1,5 +1,6 @@
class apt::unattended_upgrades {
- package{'unattended-upgrades':
+
+ package { 'unattended-upgrades':
ensure => present,
require => undef,
}
diff --git a/manifests/upgrade_package.pp b/manifests/upgrade_package.pp
index 41d5d52..99e9c2a 100644
--- a/manifests/upgrade_package.pp
+++ b/manifests/upgrade_package.pp
@@ -1,4 +1,5 @@
define apt::upgrade_package ($version = "") {
+
case $version {
'': {
exec { "aptitude -y install $name":