Browse Source

reformatted apt::ppa according to recommended
style.

removed require apt in favor of marking a
dependency.

converted release define into a class since
the resources are singletons.

Dan Bode 13 years ago
parent
commit
18f614b
2 changed files with 5 additions and 6 deletions
  1. 2 3
      manifests/ppa.pp
  2. 3 3
      manifests/release.pp

+ 2 - 3
manifests/ppa.pp

@@ -1,9 +1,8 @@
 # ppa.pp
 
-define apt::ppa(
+define apt::ppa() {
 
-) {
-    require apt
+  Class['apt'] -> Apt::Ppa[$title]
 
     exec { "apt-update-${name}":
         command     => "/usr/bin/aptitude update",

+ 3 - 3
manifests/release.pp

@@ -1,15 +1,15 @@
 # release.pp
 
-define apt::release (
-
+class apt::release (
+  $release_id
 ) {
 
 		owner => root,
 		group => root,
 		mode => 644,
-		content => "APT::Default-Release \"${name}\";"
 	}
   include apt::params
 
   file { "${apt::params::root}/apt.conf.d/01release":
+    content => "APT::Default-Release \"${release_id}\";"
 }