Date: Thu, 04 Jan 2001 18:25:35 +0200 From: Mark Murray <mark@grondar.za> To: Dag-Erling Smorgrav <des@ofug.org> Cc: Paul Richards <paul@FreeBSD.org>, cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/usr.sbin/pkg_install/update pkg_update.pl Message-ID: <200101041625.f04GPcY06775@gratis.grondar.za> In-Reply-To: <xzp3dezl2gm.fsf@flood.ping.uio.no> ; from Dag-Erling Smorgrav <des@ofug.org> "04 Jan 2001 17:14:01 %2B0100." References: <xzp3dezl2gm.fsf@flood.ping.uio.no>
next in thread | previous in thread | raw e-mail | index | archive | help
> This is a crutch, not a cure. The fundamental problem here is that > open() mixes information about the type of operation to perform with > the name of the file on which to perform that operation - which is > very poor API design. My advice is to use sysopen() consistently > except for these two cases: open(PIPE, "-|") and open(PIPE, "|-"). I think we'd better agree to disagree :-). In cases where it really counts, things like stat(3) and so on are really required. Plus, any significant perl program (as opposed to simple script) needs to be warning, strict and taint- clean anyway, so clean data is for free by the time you get to an open. M -- Mark Murray Warning: this .sig is umop ap!sdn To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200101041625.f04GPcY06775>