From owner-freebsd-audit Mon Aug 20 5:55: 3 2001 Delivered-To: freebsd-audit@freebsd.org Received: from ipcard.iptcom.net (ipcard.iptcom.net [212.9.224.5]) by hub.freebsd.org (Postfix) with ESMTP id 08B2F37B42B; Mon, 20 Aug 2001 05:48:04 -0700 (PDT) (envelope-from max@vega.com) Received: from vega.vega.com (dialup13-58.iptelecom.net.ua [212.9.229.58]) by ipcard.iptcom.net (8.9.3/8.9.3) with ESMTP id PAA35603; Mon, 20 Aug 2001 15:46:54 +0300 (EEST) (envelope-from max@vega.com) Received: (from max@localhost) by vega.vega.com (8.11.4/8.11.3) id f7KCkNg15792; Mon, 20 Aug 2001 15:46:23 +0300 (EEST) (envelope-from sobomax@FreeBSD.org) From: Maxim Sobolev Message-Id: <200108201246.f7KCkNg15792@vega.vega.com> Subject: Re: adding -P option to pkg_delete(1) To: knu@iDaemons.org (Akinori MUSHA) Date: Mon, 20 Aug 2001 15:46:23 +0300 (EEST) Cc: sobomax@FreeBSD.ORG (Maxim Sobolev), audit@FreeBSD.ORG, green@FreeBSD.ORG (Brian F. Feldman), mike@FreeBSD.ORG (Mike Barcroft), ports@FreeBSD.ORG In-Reply-To: from "Akinori MUSHA" at Aug 20, 2001 09:26:47 PM X-Mailer: ELM [version 2.5 PL5] MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit Sender: owner-freebsd-audit@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG > > At Mon, 20 Aug 2001 14:39:24 +0300, > sobomax wrote: > > > At Mon, 20 Aug 2001 13:33:03 +0300, > > > sobomax wrote: > > > > Please hold on. What is the purpose of this feature? I can't recall > > > > any discussion regarding the topic. > > > > > > It's okay, but why don't you just try to read the patch in which the > > > purpose of the feature is *explicitly* written? > > > > Well, perhaps you misread my question. I meant "what the code in question > > is expected to be used for?", not "what the code in question does?". Also > > I don't differentiate the two, or three. All your questions above > sounds pointless to me. What are you implying actually? Ok, let me put it differently - I just do not see any reasonable use for the new option. What the user would gain by removing all package content except of shared libraries? We couldn't reasonably gurantee that package will work after such partial removal, so I suppose that we are just asking ourselves a trouble in the form on increased number of PRs "hey, I pkg_delete'd -P libfoo-1.0, and bar-2.0 stopped working." Also such partial removal will screw our dependency system - LIB_DEPENDS will detect shared library, but at the same time all headers of the package will be missed. > > idea to hardcode regex (two of them actually) into an utility doesn't > > looks very solid to me, instead I would like to see it be specificable > > (or at least overrideable) from the command line, i.e.: > > > > # pkg_delete -P ^.\*foo.bar foo-0.0 > > (don't delete files which end with `foo.bar'). > > Sounds interesting, but I'm afraid it would be superfluous, I mean you > might be doing YAGNI[*], since I don't see why we need this extension > as much as we just need to preserve shared libraries. > > I suggest you explain the expected use of it, that is, what for > example you want to preserve other than old shared libraries. Just a bit of generalisation, forget it if you don't like it. -Maxim To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-audit" in the body of the message