From owner-cvs-all@FreeBSD.ORG  Fri Feb 27 10:50:54 2004
Return-Path: <owner-cvs-all@FreeBSD.ORG>
Delivered-To: cvs-all@freebsd.org
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP id 2EE2916A4D0
	for <cvs-all@FreeBSD.org>; Fri, 27 Feb 2004 10:50:54 -0800 (PST)
Received: from root.org (root.org [67.118.192.226])
	by mx1.FreeBSD.org (Postfix) with SMTP id CCD5943D46
	for <cvs-all@FreeBSD.org>; Fri, 27 Feb 2004 10:50:53 -0800 (PST)
	(envelope-from nate@root.org)
Received: (qmail 86271 invoked by uid 1000); 27 Feb 2004 18:50:54 -0000
Date: Fri, 27 Feb 2004 10:50:54 -0800 (PST)
From: Nate Lawson <nate@root.org>
To: Dag-Erling Smorgrav <des@FreeBSD.org>
In-Reply-To: <20040227175103.2B36916A4EA@hub.freebsd.org>
Message-ID: <20040227105019.O86020@root.org>
References: <20040227175103.2B36916A4EA@hub.freebsd.org>
MIME-Version: 1.0
Content-Type: TEXT/PLAIN; charset=US-ASCII
cc: cvs-src@FreeBSD.org
cc: src-committers@FreeBSD.org
cc: cvs-all@FreeBSD.org
Subject: Re: cvs commit: src/usr.bin/login Makefile
X-BeenThere: cvs-all@freebsd.org
X-Mailman-Version: 2.1.1
Precedence: list
List-Id: CVS commit messages for the entire tree <cvs-all.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/cvs-all>
List-Post: <mailto:cvs-all@freebsd.org>
List-Help: <mailto:cvs-all-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Fri, 27 Feb 2004 18:50:54 -0000

On Fri, 27 Feb 2004, Dag-Erling Smorgrav wrote:
>   Modified files:
>     usr.bin/login        Makefile
>   Log:
>   Re-add the setuid bit, conditional on NO_SETUID_LOGIN being undefined.
>
>   Revision  Changes    Path
>   1.45      +5 -0      src/usr.bin/login/Makefile
>
>
> Index: src/usr.bin/login/Makefile
> diff -u src/usr.bin/login/Makefile:1.44 src/usr.bin/login/Makefile:1.45
> --- src/usr.bin/login/Makefile:1.44	Fri Feb 27 00:39:16 2004
> +++ src/usr.bin/login/Makefile	Fri Feb 27 09:50:59 2004
> @@ -7,6 +7,11 @@
>  DPADD=	${LIBUTIL} ${LIBPAM}
>  LDADD=	-lutil ${MINUSLPAM}
>  MAN=	login.1 login.access.5
> +.ifdef NO_SETUID_LOGIN
> +BINOWN= root
> +BINMODE=4555
> +INSTALLFLAGS=-fschg
> +.endif
>  NEED_LIBNAMES= yes
>
>  .include <bsd.prog.mk>

I believe this should be ifndef for the proper behavior.

-Nate