From owner-cvs-all  Tue Sep  3 15:43:25 2002
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 4EBC137B400; Tue,  3 Sep 2002 15:43:22 -0700 (PDT)
Received: from espresso.q9media.com (espresso.q9media.com [65.39.129.122])
	by mx1.FreeBSD.org (Postfix) with ESMTP
	id E9FCF43E42; Tue,  3 Sep 2002 15:43:21 -0700 (PDT)
	(envelope-from mike@espresso.q9media.com)
Received: by espresso.q9media.com (Postfix, from userid 1002)
	id 02DAF9E59; Tue,  3 Sep 2002 18:37:16 -0400 (EDT)
Date: Tue, 3 Sep 2002 18:37:16 -0400
From: Mike Barcroft <mike@FreeBSD.org>
To: Juli Mallett <jmallett@FreeBSD.org>
Cc: Tony Finch <fanf@FreeBSD.org>, cvs-committers@FreeBSD.org,
	cvs-all@FreeBSD.org
Subject: Re: cvs commit: src/usr.bin/uudecode uudecode.c src/usr.bin/uuencode uuencode.1
Message-ID: <20020903183716.O94253@espresso.q9media.com>
References: <200209031937.g83JbNxO079200@freefall.freebsd.org> <20020903132218.D56616@FreeBSD.org>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20020903132218.D56616@FreeBSD.org>; from jmallett@FreeBSD.org on Tue, Sep 03, 2002 at 01:22:18PM -0700
Organization: The FreeBSD Project
Sender: owner-cvs-all@FreeBSD.ORG
Precedence: bulk
List-ID: <cvs-all.FreeBSD.ORG>
List-Archive: <http://docs.freebsd.org/mail/> (Web Archive)
List-Help: <mailto:majordomo@FreeBSD.ORG?subject=help> (List Instructions)
List-Subscribe: <mailto:majordomo@FreeBSD.ORG?subject=subscribe%20cvs-all>
List-Unsubscribe: <mailto:majordomo@FreeBSD.ORG?subject=unsubscribe%20cvs-all>
X-Loop: FreeBSD.ORG

Juli Mallett <jmallett@FreeBSD.org> writes:
> * De: Tony Finch <fanf@FreeBSD.org> [ Data: 2002-09-03 ]
> 	[ Subjecte: cvs commit: src/usr.bin/uudecode uudecode.c src/usr.bin/uuencode uuencode.1 ]
> > fanf        2002/09/03 12:37:23 PDT
> > 
> >   Modified files:
> >     usr.bin/uudecode     uudecode.c 
> >     usr.bin/uuencode     uuencode.1 
> >   Log:
> >   Don't try (and fail) to fchmod /dev/stdout. Revert the manual page
> >   to suggest the portable alternative to -p.
> 
> I think doing this based on fstat telling us that something is not a regular
> file (if it exists) is the right thing to do.

That's a good point, since a user could output to /dev/stderr and
encounter the problem.

Best regards,
Mike Barcroft

To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message