Tim Starling has submitted this change and it was merged.
Change subject: Preserve timestamps when copying l10n cdb files
......................................................................
Preserve timestamps when copying l10n cdb files
The fix for bug 27320 will have better performance if it can skip
updating languages that haven't actually changed since the last run.
LocalisationUpdate already avoids needlessly updating the cdb files, but
we need to tell l10nupdate-1 to preserve timestamps when copying them.
Change-Id: I72a1557f9c18b845c952dd2e2697d92e8eb71d93
---
M files/misc/l10nupdate/l10nupdate-1
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Tim Starling: Verified; Looks good to me, approved
diff --git a/files/misc/l10nupdate/l10nupdate-1
b/files/misc/l10nupdate/l10nupdate-1
index 8291f5e..22a8d2d 100755
--- a/files/misc/l10nupdate/l10nupdate-1
+++ b/files/misc/l10nupdate/l10nupdate-1
@@ -73,7 +73,7 @@
--quiet
--outdir=/var/lib/l10nupdate/cache-"$mwVerNum"
# Copy the LC files
echo "Completed. Copying LC files to /home"
- cp --force /var/lib/l10nupdate/cache-"$mwVerNum"/l10n_cache-*
/home/wikipedia/common/php-"$mwVerNum"/cache/l10n
+ cp --preserve=timestamps --force
/var/lib/l10nupdate/cache-"$mwVerNum"/l10n_cache-*
/home/wikipedia/common/php-"$mwVerNum"/cache/l10n
echo "Syncing to Apaches"
$BINDIR/sync-l10nupdate-1 "$mwVerNum"
# clearMessageBlobs.php sometimes causes some sort of database
stampede
--
To view, visit https://gerrit.wikimedia.org/r/58910
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I72a1557f9c18b845c952dd2e2697d92e8eb71d93
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Anomie <[email protected]>
Gerrit-Reviewer: Tim Starling <[email protected]>
Gerrit-Reviewer: jenkins-bot
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
