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

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

Issue 29447571: Fix illegal variable name for pythonpath in web module (Closed)
Patch Set: Created May 24, 2017, 5:06 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
@@ -12,7 +12,7 @@
include sitescripts
- $PYTHONPATH = 'PYTHONPATH=/opt/cms:/opt/sitescripts'
+ $pythonpath = 'PYTHONPATH=/opt/cms:/opt/sitescripts'
# Ensure there is at least one character in the respective strings;
# see https://codereview.adblockplus.org/29329028/#msg3
@@ -27,7 +27,7 @@
Cron {
environment => concat(hiera('cron::environment', []), [
- $PYTHONPATH,
+ $pythonpath,
]),
}
@@ -141,7 +141,7 @@
user => www,
subscribe => [Exec["fetch_repo"], Exec["fetch_cms"]],
refreshonly => true,
- environment => $PYTHONPATH,
+ environment => $pythonpath,
}
file {'/var/www':
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld