Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 27 Jan 2017 07:25:48 -0800 (PST)
From:      "Rodney W. Grimes" <freebsd-rwg@pdx.rh.CN85.dnsmgr.net>
To:        Julian Elischer <julian@freebsd.org>
Cc:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-11@freebsd.org
Subject:   Re: svn commit: r312871 - stable/11/share/zoneinfo
Message-ID:  <201701271525.v0RFPmVq025332@pdx.rh.CN85.dnsmgr.net>
In-Reply-To: <201701270911.v0R9BiQr000147@repo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
> Author: julian
> Date: Fri Jan 27 09:11:44 2017
> New Revision: 312871
> URL: https://svnweb.freebsd.org/changeset/base/312871
> 
> Log:
>   MFH: r308671
>   
>   When you select make OLDTIMEZONES=1 then you need a few added directories
>   to be made or the command fails
>   
>   Sponsored by:	panzura
>   
>   MFH: r310426
>   
>   If you are going to be run individually to make a new timezone set
>   then ensure the destination directories exist.
>   Especially if you define OLDTIMEZONES because the mtree pass
>   doesn't do it for you.

Perhaps it is time to start pre-processing mtree files so this
would no longer be the case?

>   Sponsored by:	Panzura
> 
> Modified:
>   stable/11/share/zoneinfo/Makefile
> Directory Properties:
>   stable/11/   (props changed)
> 
> Modified: stable/11/share/zoneinfo/Makefile
> ==============================================================================
> --- stable/11/share/zoneinfo/Makefile	Fri Jan 27 09:07:11 2017	(r312870)
> +++ stable/11/share/zoneinfo/Makefile	Fri Jan 27 09:11:44 2017	(r312871)
> @@ -67,6 +67,10 @@ TZBUILDSUBDIRS=	\
>  		Pacific \
>  		SystemV
>  
> +.if defined(OLDTIMEZONES)
> +TZBUILDSUBDIRS+= US Mexico Chile Canada Brazil
> +.endif
> +
>  .if !defined(_SKIP_BUILD)
>  all: zoneinfo
>  .endif
> @@ -81,6 +85,8 @@ zoneinfo: yearistype ${TDATA}
>  
>  beforeinstall: install-zoneinfo
>  install-zoneinfo:
> +	mkdir -p ${DESTDIR}/usr/share/zoneinfo
> +	cd ${DESTDIR}/usr/share/zoneinfo;  mkdir -p ${TZBUILDSUBDIRS}

This has the failure mode that your current uid and umask, etc are
used in creating the directories.  Please add appropriate chown/chmod
commands to reflect what mtree would of done had it done this.

>  	cd ${TZBUILDDIR} && \
>  	    find -s * -type f -print -exec ${INSTALL} ${TAG_ARGS} \
>  	    -o ${BINOWN} -g ${BINGRP} -m ${NOBINMODE} \
> _______________________________________________
> svn-src-stable-11@freebsd.org mailing list
> https://lists.freebsd.org/mailman/listinfo/svn-src-stable-11
> To unsubscribe, send any mail to "svn-src-stable-11-unsubscribe@freebsd.org"
> 

-- 
Rod Grimes                                                 rgrimes@freebsd.org



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