svn commit: r328934 - in head: . bin/sh

Rodney W. Grimes freebsd at pdx.rh.CN85.dnsmgr.net
Tue Feb 6 19:25:15 UTC 2018


> Author: arichardson
> Date: Tue Feb  6 15:41:35 2018
> New Revision: 328934
> URL: https://svnweb.freebsd.org/changeset/base/328934
> 
> Log:
>   Don't hardcode /usr/bin as the path for mktemp in build tools
>   
>   It won't work e.g. when crossbuilding from Ubuntu Linux as mktemp is in
>   /bin there.
>   
>   Reviewed By:	bdrewery
>   Approved By:	jhb (mentor)
>   Differential Revision: https://reviews.freebsd.org/D13937

Would it be better to create the variable MKTEMP to point at
either /bin/mktemp or /usr/bin/mktemp dependent on platform,
there are reasons we use full paths in Makefiles, mostly to
stop /usr/local/bin/foo contimaton, which I believe this
change now opens up, though very slight as I dont know of
a third party mktemp binary.


> Modified:
>   head/Makefile.inc1
>   head/bin/sh/mkbuiltins
>   head/bin/sh/mktokens
> 
> Modified: head/Makefile.inc1
> ==============================================================================
> --- head/Makefile.inc1	Tue Feb  6 15:41:26 2018	(r328933)
> +++ head/Makefile.inc1	Tue Feb  6 15:41:35 2018	(r328934)
> @@ -463,7 +463,7 @@ TMPPATH=	${STRICTTMPPATH}:${PATH}
>  # when in the middle of installing over this system.
>  #
>  .if make(distributeworld) || make(installworld) || make(stageworld)
> -INSTALLTMP!=	/usr/bin/mktemp -d -u -t install
> +INSTALLTMP!=	mktemp -d -u -t install
>  .endif
>  
>  .if make(stagekernel) || make(distributekernel)
> 
> Modified: head/bin/sh/mkbuiltins
> ==============================================================================
> --- head/bin/sh/mkbuiltins	Tue Feb  6 15:41:26 2018	(r328933)
> +++ head/bin/sh/mkbuiltins	Tue Feb  6 15:41:35 2018	(r328934)
> @@ -34,7 +34,7 @@
>  #	@(#)mkbuiltins	8.2 (Berkeley) 5/4/95
>  # $FreeBSD$
>  
> -temp=`/usr/bin/mktemp -t ka`
> +temp=`mktemp -t ka`
>  havehist=1
>  if [ "X$1" = "X-h" ]; then
>  	havehist=0
> 
> Modified: head/bin/sh/mktokens
> ==============================================================================
> --- head/bin/sh/mktokens	Tue Feb  6 15:41:26 2018	(r328933)
> +++ head/bin/sh/mktokens	Tue Feb  6 15:41:35 2018	(r328934)
> @@ -38,7 +38,7 @@
>  # token marks the end of a list.  The third column is the name to print in
>  # error messages.
>  
> -temp=`/usr/bin/mktemp -t ka`
> +temp=`mktemp -t ka`
>  cat > $temp <<\!
>  TEOF	1	end of file
>  TNL	0	newline
> 
> 

-- 
Rod Grimes                                                 rgrimes at freebsd.org


More information about the svn-src-head mailing list