[Bug 497756] Review Request: lpg - LALR Parser Generator

bugzilla at redhat.com bugzilla at redhat.com
Mon Apr 27 10:34:37 UTC 2009


Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=497756





--- Comment #3 from Igor Jurišković <juriskovic.igor at gmail.com>  2009-04-27 06:34:35 EDT ---
(In reply to comment #2)
> (In reply to comment #1)
> > Hi Mat,
> > 
> > first of all this is not official review. I'm in need of sponsor.
> > 
> > URL:       http://lpg.sourceforge.net/
> > It is much better to use macro. If the package name changes you will not need
> > to change the url too.
> > URL:       http://%{name}.sourceforge.net/
> But on the other hand
> - names of the projects very rarely change
> - using macros in the URL / SOURCE tags block usage of certain automated
> verification tools. And even simple copy / paste does not work any more.
> So I beg to differ, but using macros here is not at all much better.  

You're right. I used macro in my review request and nobody complained. I guess
both ways are right. You could wait for advanced packager to tell you what is
better.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.




More information about the package-review mailing list