Date: Thu, 7 Dec 2006 13:31:03 -0500 From: Wesley Shields <wxs@atarininja.org> To: ports@freebsd.org, Doug Barton <dougb@FreeBSD.org> Subject: Re: distfile belongs to? Message-ID: <20061207183103.GA73597@atarininja.org> In-Reply-To: <20061206014049.GA43656@atarininja.org> References: <45749998.3070308@FreeBSD.org> <20061204232125.GA42307@atarininja.org> <790a9fff0612050838s66c655fapfde80d4038f64ca2@mail.gmail.com> <20061205172803.GA51892@atarininja.org> <4575C568.70905@FreeBSD.org> <20061205194146.GC52966@atarininja.org> <20061205203120.GA53894@atarininja.org> <4575EBA8.3020601@FreeBSD.org> <BDBB52B0-3FB2-4C82-A507-4785183B3C1F@FreeBSD.org> <20061206014049.GA43656@atarininja.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, Dec 05, 2006 at 08:40:49PM -0500, Wesley Shields wrote: > On Tue, Dec 05, 2006 at 05:08:45PM -0800, Ade Lovett wrote: > > > > On Dec 05, 2006, at 13:59 , Doug Barton wrote: > > > > >Wesley Shields wrote: > > >>>@comment DISTFILE:$filename:$size:$md5:$sha256 > > > > > >I think that's good, the other information might come in handy down > > >the road. One thing you might want to consider is to put the md5 sum > > >at the end, since at some point down the road (maybe years from now, > > >but still ...) we're likely to drop md5 altogether. > > > > Better still would be: > > > > @comment DISTFILE:<filename>[':' <key>=<value>]* > > > > eg: DISTFILE:foo:SIZE=1234:MD5=...:SHA256=... > > > > Relying on the specific position of an item in a delimited list > > almost always comes back and bites in painful spots later on down the > > way. Minimal extra parsing required by consumers, and bits and > > pieces can be added/removed almost at will. > > > > -aDe > > I agree. I'll modify Doug's latest patch to add this. I've reworked the two patches to take all of the suggestions in. Part 1 (the pkg_info piece): - Silently ignores the -F flag when distfile information is not recorded in +CONTENTS. - Prints the information out in a human readable format (still respects -q). Part 2 (the bsd.port.mk piece) is the patch posted by Doug with Ade's suggestions. The entire thing no longer requires a change to pkg_create in order to function. Anymore thoughts/comments/ideas before I send-pr these? http://www.atarininja.org/~wxs/patches/package-distinfo-part1.diff http://www.atarininja.org/~wxs/patches/package-distinfo-part2.diff -- WXS
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20061207183103.GA73597>