Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 12 Feb 2015 21:56:47 -0700
From:      Ian Lepore <ian@freebsd.org>
To:        Baptiste Daroussin <bapt@FreeBSD.org>
Cc:        svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org
Subject:   Re: svn commit: r278642 - in head/usr.bin: chpass passwd
Message-ID:  <1423803407.80968.126.camel@freebsd.org>
In-Reply-To: <201502122308.t1CN8Sg0062438@svn.freebsd.org>
References:  <201502122308.t1CN8Sg0062438@svn.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, 2015-02-12 at 23:08 +0000, Baptiste Daroussin wrote:
> Author: bapt
> Date: Thu Feb 12 23:08:27 2015
> New Revision: 278642
> URL: https://svnweb.freebsd.org/changeset/base/278642
> 
> Log:
>   Use PRECIOUSPROG instead of custom code to handle schg
>   
>   This allows to preserve schg when installed with -DNO_ROOT
>   
>   MFC after:	1 week
> 
> Modified:
>   head/usr.bin/chpass/Makefile
>   head/usr.bin/passwd/Makefile
> 
> Modified: head/usr.bin/chpass/Makefile
> ==============================================================================
> --- head/usr.bin/chpass/Makefile	Thu Feb 12 23:00:31 2015	(r278641)
> +++ head/usr.bin/chpass/Makefile	Thu Feb 12 23:08:27 2015	(r278642)
> @@ -9,6 +9,7 @@ PROG=	chpass
>  SRCS=	chpass.c edit.c field.c pw_scan.c table.c util.c
>  BINOWN=	root
>  BINMODE=4555
> +PRECIOUSPROG=
>  .if ${MK_NIS} != "no"
>  CFLAGS+= -DYP
>  .endif
> @@ -34,16 +35,4 @@ MLINKS=	chpass.1 chfn.1 chpass.1 chsh.1
>  MLINKS+= chpass.1 ypchpass.1 chpass.1 ypchfn.1 chpass.1 ypchsh.1
>  .endif
>  
> -beforeinstall:
> -.for i in chpass chfn chsh ypchpass ypchfn ypchsh
> -.if exists(${DESTDIR}${BINDIR}/$i)
> -	-chflags noschg ${DESTDIR}${BINDIR}/$i
> -.endif
> -.endfor
> -

It looks like the removal of this beforeinstall target is causing
failure to re-install over an existing copy of the program.

-- Ian





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