Re: svn commit: r301944 - head/share/zoneinfo

2016-06-16 Thread Stefan Esser
Am 16.06.2016 um 05:00 schrieb Glen Barber:
> Author: gjb
> Date: Thu Jun 16 03:00:10 2016
> New Revision: 301944
> URL: https://svnweb.freebsd.org/changeset/base/301944
> 
> Log:
>   Fix zoneinfo file packaging.
>   
>   This change fixes 468 of 488 zoneinfo file packaging issues,
>   the rest still to be investigated.
>   
>   Approved by:re (blanket, pkgbase)
>   Sponsored by:   The FreeBSD Foundation
> 
> Modified:
>   head/share/zoneinfo/Makefile
> 
> Modified: head/share/zoneinfo/Makefile
> ==
> --- head/share/zoneinfo/Makefile  Thu Jun 16 02:48:18 2016
> (r301943)
> +++ head/share/zoneinfo/Makefile  Thu Jun 16 03:00:10 2016
> (r301944)
> @@ -82,10 +82,10 @@ zoneinfo: yearistype ${TDATA}
>  beforeinstall: install-zoneinfo
>  install-zoneinfo:
>   cd ${TZBUILDDIR} && \
> - find -s * -type f -print -exec ${INSTALL} \
> + find -s * -type f -print -exec ${INSTALL} -T ${TAGS} \
>   -o ${BINOWN} -g ${BINGRP} -m ${NOBINMODE} \
>   \{} ${DESTDIR}/usr/share/zoneinfo/\{} \;
> - ${INSTALL} -o ${BINOWN} -g ${BINGRP} -m ${NOBINMODE} \
> + ${INSTALL} -T ${TAGS} -o ${BINOWN} -g ${BINGRP} -m ${NOBINMODE} \
>   ${CONTRIBDIR}/zone.tab ${DESTDIR}/usr/share/zoneinfo/

This change breaks installworld for me.

${TAGS} is empty and makes ${INSTALL} fail:

===> share/zoneinfo (install)
cd /usr/obj/usr/svn/base/head/share/zoneinfo/builddir &&  find -s *
-type f -print -exec install -T   -o root -g wheel -m 444  \{}
//usr/share/zoneinfo/\{} \;
Africa/Abidjan
install: target `//usr/share/zoneinfo/Africa/Abidjan' is not a directory
usage: install [-bCcpSsUv] [-f flags] [-g group] [-m mode] [-o owner]
   [-M log] [-D dest] [-h hash] [-T tags]
   [-B suffix] [-l linkflags] [-N dbdir]
   file1 file2
   install [-bCcpSsUv] [-f flags] [-g group] [-m mode] [-o owner]
   [-M log] [-D dest] [-h hash] [-T tags]
   [-B suffix] [-l linkflags] [-N dbdir]
   file1 ... fileN directory
   install -dU [-vU] [-g group] [-m mode] [-N dbdir] [-o owner]
   [-M log] [-D dest] [-h hash] [-T tags]
   directory ...
...

A simple work-around might be double quotes around ${TAGS} which makes
${INSTALL} happy - it accepts the empty string as parameter for -T
without complaints.

Regards, STefan
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r301944 - head/share/zoneinfo

2016-06-15 Thread Glen Barber
Author: gjb
Date: Thu Jun 16 03:00:10 2016
New Revision: 301944
URL: https://svnweb.freebsd.org/changeset/base/301944

Log:
  Fix zoneinfo file packaging.
  
  This change fixes 468 of 488 zoneinfo file packaging issues,
  the rest still to be investigated.
  
  Approved by:  re (blanket, pkgbase)
  Sponsored by: The FreeBSD Foundation

Modified:
  head/share/zoneinfo/Makefile

Modified: head/share/zoneinfo/Makefile
==
--- head/share/zoneinfo/MakefileThu Jun 16 02:48:18 2016
(r301943)
+++ head/share/zoneinfo/MakefileThu Jun 16 03:00:10 2016
(r301944)
@@ -82,10 +82,10 @@ zoneinfo: yearistype ${TDATA}
 beforeinstall: install-zoneinfo
 install-zoneinfo:
cd ${TZBUILDDIR} && \
-   find -s * -type f -print -exec ${INSTALL} \
+   find -s * -type f -print -exec ${INSTALL} -T ${TAGS} \
-o ${BINOWN} -g ${BINGRP} -m ${NOBINMODE} \
\{} ${DESTDIR}/usr/share/zoneinfo/\{} \;
-   ${INSTALL} -o ${BINOWN} -g ${BINGRP} -m ${NOBINMODE} \
+   ${INSTALL} -T ${TAGS} -o ${BINOWN} -g ${BINGRP} -m ${NOBINMODE} \
${CONTRIBDIR}/zone.tab ${DESTDIR}/usr/share/zoneinfo/
 
 afterinstall:
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"