]> gitweb.fluxo.info Git - puppet-apt.git/commitdiff
README update: clarify the site-apt preferences options
authorMicah Anderson <micah@riseup.net>
Tue, 7 Dec 2010 18:39:54 +0000 (13:39 -0500)
committerMicah Anderson <micah@riseup.net>
Tue, 7 Dec 2010 18:39:54 +0000 (13:39 -0500)
The README described a few things that were not true relating to the
apt/preferences file.

First of all it said you could ship a 'file', but preferences.pp very clearly
uses the 'content => $custom_preferences' parameter, which will not take file
sources, only templates.

Secondly, it seemed to imply that you could just drop the custom preferences
into your site-apt and it would work. But you actually need to set the
$custom_preferences to indicate the content source.

Lastly, it said that you could specify a host-specific file in the site-apt
module, but there is no facility for this (nor can you use files).

Perhaps this is where this module is going eventually, once we have a
preferences.d possibility? Until then, it makes more sense to have it reflect
the current situation.

README

diff --git a/README b/README
index 8118d3cb0259c3c6b44c754d980ad4c5ddfffe43..97a398b75cb5beccdd8e9d1b9ea98a359894f4ee 100644 (file)
--- a/README
+++ b/README
@@ -75,15 +75,9 @@ installation will not accidentally pull in packages from those suites
 unless you explicitly specify the version number. This file will be
 complemented with all of the preferences_snippet calls (see below).
 
-If the default preferences template doesn't suit your needs, you can
-create a file named 'preferences' in a site-apt module's files
-directory. You can also create a host-specific file:
-
-  site-apt
-   - files/
-     - server.domain.com/
-       - preferences
-       preferences
+If the default preferences template doesn't suit your needs, you can create a
+template located in your site-apt module, and set $custom_preferences with the
+location (eg. $custom_preferences = "puppet:///modules/site-apt/preferences")
 
 Setting this variable to false before including this class will force the
 apt/preferences file to be absent: