X-Git-Url: https://git.adam-barratt.org.uk/?a=blobdiff_plain;f=modules%2Fbuildd%2Fmanifests%2Finit.pp;h=fc3e21d99966bd480a34e54d1c033f78fa350e7b;hb=1c7a4178f4687fd07825eac48ef8d2829a67719b;hp=ab6b05a2c8231d77543d38eadb2c1b58b75d1400;hpb=5f7e9b2a528a64883641f38558c028ae4b1967f0;p=mirror%2Fdsa-puppet.git

diff --git a/modules/buildd/manifests/init.pp b/modules/buildd/manifests/init.pp
index ab6b05a2c..fc3e21d99 100644
--- a/modules/buildd/manifests/init.pp
+++ b/modules/buildd/manifests/init.pp
@@ -2,18 +2,26 @@ class buildd {
 
 	# sigh, sort this mess out, kids
 	if $::lsbdistcodename in [lenny,squeeze] {
-		package { 'schroot': ensure => installed }
+		package { 'schroot':
+			ensure => installed,
+		}
 	} else {
-		package { 'schroot': ensure => held }
+		// package { 'schroot': ensure => held }
 	}
 
+	package { 'apt-transport-https':
+		ensure => installed,
+	}
 	package { [
 			'sbuild',
-			'apt-transport-https',
 			'debootstrap',
 			'dupload'
 		]:
-			ensure => installed
+		ensure => installed,
+		require => [
+			File['/etc/apt/sources.list.d/db.debian.org.list'],
+			Exec['apt-get update']
+		]
 	}
 
 	site::linux_module { 'dm_snapshot': }
@@ -23,12 +31,19 @@ class buildd {
 		ensure => absent,
 	}
 	site::aptrepo { 'buildd.debian.org':
-		content => template('buildd/etc/apt/sources.list.d/buildd.list.erb'),
-		key     => 'puppet:///modules/buildd/buildd.debian.org.asc',
+		template => 'buildd/etc/apt/sources.list.d/buildd.list.erb',
+		key      => 'puppet:///modules/buildd/buildd.debian.org.asc',
+		require  => Package['apt-transport-https'],
 	}
 
-	file { '/etc/apt/preferences.d/buildd':
-		ensure  => absent
+	# "bad" extension
+	file { '/etc/apt/preferences.d/buildd.debian.org':
+		ensure => absent,
+	}
+	file { '/etc/apt/preferences.d/buildd.debian.org':
+		content => template('buildd/etc/apt/preferences.d/buildd'),
+		before  => File["/etc/apt/sources.list.d/buildd.debian.org.list"],
+		;
 	}
 	file { '/etc/schroot/mount-defaults':
 		content => template('buildd/etc/schroot/mount-defaults.erb'),