Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 6 Feb 2018 23:12:17 +0000 (UTC)
From:      Warner Losh <imp@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r328958 - head/share/zoneinfo
Message-ID:  <201802062312.w16NCHTO070030@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: imp
Date: Tue Feb  6 23:12:16 2018
New Revision: 328958
URL: https://svnweb.freebsd.org/changeset/base/328958

Log:
  Avoid find -s, use find | sort instead.
  
  find -s was introduced to make the metalog more
  deterministic. However, find -s is not portable. find | sort is
  portable and accomplishes the same goals, even if it isn't
  pedantically the same. TZS is the same before / after the change so
  any fussy differences between the two are moot and there won't be
  METALOG churn across this change.
  
  Differential Revision: https://reviews.freebsd.org/D14231

Modified:
  head/share/zoneinfo/Makefile

Modified: head/share/zoneinfo/Makefile
==============================================================================
--- head/share/zoneinfo/Makefile	Tue Feb  6 22:38:19 2018	(r328957)
+++ head/share/zoneinfo/Makefile	Tue Feb  6 23:12:16 2018	(r328958)
@@ -95,8 +95,13 @@ zoneinfo: yearistype ${TDATA}
 	zic -D -d ${TZBUILDDIR} -p ${POSIXRULES} -m ${NOBINMODE} \
 	    ${LEAPFILE} -y ${.OBJDIR}/yearistype ${TZFILES}
 
+#
+# Sort TZS to ensure they are the same every build. find -s might
+# be a shorter way to express this, but it's non-portable. Any
+# differences between the two don't matter for this purpose.
+#
 .if make(*install*)
-TZS!= cd ${TZBUILDDIR} && find -s * -type f
+TZS!= cd ${TZBUILDDIR} && find * -type f | env LC_ALL=C sort
 .endif
 
 beforeinstall: install-zoneinfo



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201802062312.w16NCHTO070030>