From owner-cvs-all Thu Jan 4 8:26:16 2001 From owner-cvs-all@FreeBSD.ORG Thu Jan 4 08:26:13 2001 Return-Path: Delivered-To: cvs-all@freebsd.org Received: from gratis.grondar.za (grouter.grondar.za [196.7.18.65]) by hub.freebsd.org (Postfix) with ESMTP id 08A0D37B400; Thu, 4 Jan 2001 08:26:08 -0800 (PST) Received: from grondar.za (root@gratis.grondar.za [196.7.18.133]) by gratis.grondar.za (8.11.1/8.11.1) with ESMTP id f04GPcY06775; Thu, 4 Jan 2001 18:25:41 +0200 (SAST) (envelope-from mark@grondar.za) Message-Id: <200101041625.f04GPcY06775@gratis.grondar.za> To: Dag-Erling Smorgrav Cc: Paul Richards , cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/usr.sbin/pkg_install/update pkg_update.pl References: In-Reply-To: ; from Dag-Erling Smorgrav "04 Jan 2001 17:14:01 +0100." Date: Thu, 04 Jan 2001 18:25:35 +0200 From: Mark Murray Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk X-Loop: FreeBSD.ORG > 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