Tim Starling has submitted this change and it was merged.

Change subject: Fix stupid mistake (scap should run rsync, not echo it)
......................................................................


Fix stupid mistake (scap should run rsync, not echo it)

I introduced the echo to test change I702777b4a and neglected to remove it
before committing.

Change-Id: Id71f48272bc1f06466437bf3df46a474424b81c6
---
M files/scap/scap-1skins
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Tim Starling: Verified; Looks good to me, approved
  jenkins-bot: Verified



diff --git a/files/scap/scap-1skins b/files/scap/scap-1skins
index b4ffe8f..a72bfb6 100755
--- a/files/scap/scap-1skins
+++ b/files/scap/scap-1skins
@@ -10,7 +10,7 @@
 
 for mwVerNum in ${mwVersionNums[@]}; do
        sudo -u mwdeploy sh -c "
-       echo rsync -a --no-perms 
${MW_RSYNC_HOST}::common/php-${mwVerNum}/skins/ \
+       rsync -a --no-perms ${MW_RSYNC_HOST}::common/php-${mwVerNum}/skins/ \
        ${MW_COMMON}/php-${mwVerNum}/skins && \
        echo ok || \
        echo failed

-- 
To view, visit https://gerrit.wikimedia.org/r/72062
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Id71f48272bc1f06466437bf3df46a474424b81c6
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Ori.livneh <o...@wikimedia.org>
Gerrit-Reviewer: Tim Starling <tstarl...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to