Date: Mon, 04 Nov 2002 16:22:23 -0500 From: "Dan Langille" <dan@langille.org> To: Mario Sergio Fujikawa Ferreira <lioux@FreeBSD.org> Cc: Kris Kennaway <kris@obsecurity.org>, Kris Kennaway <kris@FreeBSD.org>, cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: ports/games Makefile ports/games/tome Makefile distinfo pkg-descr pkg-plist ports/games/tome/files patch-aa Message-ID: <3DC69EBF.13616.67D8B872@localhost> In-Reply-To: <20021104211418.81286.qmail@exxodus.fedaykin.here> References: <3DC690A9.2293.67A1B22C@localhost>
next in thread | previous in thread | raw e-mail | index | archive | help
On 4 Nov 2002 at 19:13, Mario Sergio Fujikawa Ferreir wrote: > On Mon, Nov 04, 2002 at 03:21:55PM -0500, Dan Langille wrote: > > On 4 Nov 2002 at 12:18, Kris Kennaway wrote: > > > > > On Mon, Nov 04, 2002 at 02:07:10PM -0500, Dan Langille wrote: > > > > > > > > > Why is there no pkg-comment? > > > > > > > > > > The PORTCOMMENT variable in the makefile takes care of it. > > > ^^^^^^^^^^^ > > > > $ make -V COMMENT > > > > /usr/home/dan/ports/games/tome/pkg-comment > > > > > > > > That's not right is it? > > > > > > No :-) > > > > I've raised it on -ports, and FreshPorts has a fix to cater for > > PORTCOMMENT. I'm just running it in my test environment for a few > > more commits before putting it into production. > > What about the following patch? > > 1) PORTCOMMENT is always set, i.e. if there is a COMMENT > file, PORTCOMMENT is set with COMMENT file's content > > 2) Therefore, if there are both a COMMENT file AND a > PORTCOMMENT variable, COMMENT file takes precedence > > 3) PORTCOMMENT is now the standard to find out the comment > for a file since it will always be set > > 4) make read works again since it was not using the info > from PORTCOMMENT when there was no COMMENT file From a FreshPorts point of view, this patch is great. It simplifies things. > Caveat with this patch? Only two I can think of. > > $ cd ports/textproc/p5-xmltv > $ make describe > p5-xmltv-0.5.2|/usr/ports/textproc/p5-xmltv|/usr/local|"Set of programs to process TV (tvguide) listings in XML format"|/usr/ports/textproc/p5-xmltv/pkg-descr|lioux@FreeBSD.org|textproc perl5|/usr/ports/archivers/p5-Compress-Zlib /usr/ports/converters/p5-Unicode-String /usr/ports/devel/p5-Date- Manip /usr/ports/devel/p5-Memoize /usr/ports/textproc/p5-Lingua-Preferred /usr/ports/textproc/p5-XML-Simple /usr/ports/textproc/p5-XML-Twig /usr/ports/textproc/p5-XML-Writer /usr/ports/www/p5-CGI.pm /usr/ports/www/p5-HTML-Parser /usr/ports/www/p5-HTML-TableExtract /usr/ports/www/p5-libwww /usr/ports/x11-toolkits/p5-Tk-TableMatrix|/usr/ports/archivers/p5-Compress-Zlib /usr/ports/converters/p5-Unicode-String /usr/ports/devel/p5-Date-Manip /usr/ports/devel/p5-Memoize /usr/ports/textproc/p5-Lingua-Preferred /usr/ports/textproc/p5-XML-Simple /usr/ports/textproc/p5-XML-Twig /usr/ports/textproc/p5-XML-Writer /usr/ports/www/p5-CGI.pm /usr/ports/www/p5-HTML-Parser /usr/ports/www/p5-HTML-TableExtract /usr/ports/www/p5-libwww /usr/ports/x11-toolkits/p5-Tk-TableMatrix|http://xmltv.sourceforge.net/ > > There is a nagging "" in there. Unfortunaly the "" is > required due to the parentheses in the comment. Don't ask me why > but the port breaks when I try to 'make package'. Perhaps this point is related: Kris mentioned that he had to escape a ' contained within PORTCOMMENT otherwise an error would occur at install time. i.e. ToME is a roguelike dungeon exploration game based on JRR Tolkien\'s works > The second one, we can now use PORTCOMMENT in 'make readme'. > However, if there is neither PORTCOMMENT variable nor COMMENT file, > we get an empty extra line inside README.html That should never happen. It's like leaving out pkg-descr. -- Dan Langille 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?3DC69EBF.13616.67D8B872>