You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ponymail.apache.org by se...@apache.org on 2019/09/05 11:12:03 UTC

[incubator-ponymail] branch master updated: Ensure sort order is stable

This is an automated email from the ASF dual-hosted git repository.

sebb pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-ponymail.git


The following commit(s) were added to refs/heads/master by this push:
     new fb176b5  Ensure sort order is stable
fb176b5 is described below

commit fb176b5d9d34bd70ab1612c168903c63ef275b57
Author: Sebb <se...@apache.org>
AuthorDate: Thu Sep 5 12:11:53 2019 +0100

    Ensure sort order is stable
---
 site/js/dev/combine.sh | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/site/js/dev/combine.sh b/site/js/dev/combine.sh
index 775d6c2..567d38d 100755
--- a/site/js/dev/combine.sh
+++ b/site/js/dev/combine.sh
@@ -17,7 +17,9 @@ echo '/*
 */
 // THIS IS AN AUTOMATICALLY COMBINED FILE. PLEASE EDIT dev/*.js!!
 ' > ../ponymail.js
-for f in `ls *.js`; do
+# Warning: ls/sort order depends on the locale; this can affect the order
+# of non-alphanumerics such as '.' and '_'. So force the use of 'C' locale
+for f in `LC_ALL=C ls *.js`; do
     printf "\n\n/******************************************\n Fetched from dev/${f}\n******************************************/\n\n" >> ../ponymail.js
     sed -e '/^\/\*/,/\*\//d' ${f} >> ../ponymail.js
 done