Date: Fri, 23 Mar 2001 13:37:10 -0500 From: Garrett Rooney <rooneg@electricjellyfish.net> To: Maxim Sobolev <sobomax@FreeBSD.org> Cc: ports@FreeBSD.org, jhk@FreeBSD.org, gad@FreeBSD.org Subject: Re: [patch] which package functionality for pkg_info Message-ID: <20010323133710.B26933@electricjellyfish.net> In-Reply-To: <3ABB9565.F5F0C22@FreeBSD.org>; from sobomax@FreeBSD.org on Fri, Mar 23, 2001 at 08:26:45PM %2B0200 References: <20010301145950.B90945@electricjellyfish.net> <200103012014.f21KESc59992@vic.sabbo.net> <20010322183141.B23846@electricjellyfish.net> <3ABB17E8.A5558BDD@FreeBSD.org> <3ABB9565.F5F0C22@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Fri, Mar 23, 2001 at 08:26:45PM +0200, Maxim Sobolev wrote: > Maxim Sobolev wrote: > > Thank you for your reminder. I have several other comments regarding your > > patchset, which I hopefully will submit to you later today. Please hold on. > > Ok with this message I'm attaching final version of the patch in question. > Following fixes/cleanups have been made: > - comments formatted to match style(9); > - popen(3) return value check added; > - cmp_path() function simplified; > - error messages slighly ajusted to follow usual format; > - includes reordered to be consistent across the source; > - etc. > > If you agree with this patch, please let me know and I'll commit it shortly. These all seem fine with me. I especially like the change to cmp_path, as that is quite a bit simpler than my version. I'll have to remember that one. -- garrett rooney Unix was not designed to stop you from rooneg@electricjellyfish.net doing stupid things, because that would http://electricjellyfish.net/ stop you from doing clever things. To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-ports" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20010323133710.B26933>