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

Unified Diff: modules/web/manifests/server.pp

Issue 29438574: Noissue - Improve CMS repository clone command (Closed)
Patch Set: For comment 2 Created May 16, 2017, 7:18 p.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/web/manifests/server.pp
===================================================================
--- a/modules/web/manifests/server.pp
+++ b/modules/web/manifests/server.pp
@@ -99,8 +99,14 @@
managehome => true,
}
+ $fetch_cms_cmd = [
+ 'hg', 'clone',
+ 'https://hg.adblockplus.org/cms/',
+ '/opt/cms',
+ ]
+
exec {"fetch_cms":
- command => "hg clone https://hg.adblockplus.org/cms/ /opt/cms",
+ command => shellquote($fetch_cms_cmd),
path => ["/usr/bin/", "/bin/"],
require => Package['mercurial'],
timeout => 0,
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld