Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: modules/sitescripts/manifests/init.pp

Issue 5814178002501632: Issue 1495 - Introduce $sitescriptsini_content parameter (Closed)
Patch Set: Created Oct. 23, 2014, 9:33 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: modules/sitescripts/manifests/init.pp
===================================================================
--- a/modules/sitescripts/manifests/init.pp
+++ b/modules/sitescripts/manifests/init.pp
@@ -1,5 +1,6 @@
class sitescripts (
- $sitescriptsini_source = undef
+ $sitescriptsini_source = '',
+ $sitescriptsini_content = '',
){
concat {'/etc/sitescripts.ini':
@@ -8,15 +9,22 @@
group => root,
}
- define configfragment($source = $title)
+ define configfragment($content = '', $source = '')
{
- concat::fragment {$source:
- target => '/etc/sitescripts.ini',
- source => $source
+ concat::fragment {"/etc/sitescripts.ini#$title":
+ target => '/etc/sitescripts.ini',
+ content => $content,
+ source => "$source;$content" ? {
+ ';' => $title,
+ default => $source,
+ }
}
}
- configfragment {$sitescriptsini_source: }
+ configfragment {'/etc/sitescripts.ini':
+ content => $sitescriptsini_content,
+ source => $sitescriptsini_source,
+ }
exec { "fetch_sitescripts":
command => "hg clone https://hg.adblockplus.org/sitescripts /opt/sitescripts",
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld