[Bug 512217] Review Request: geany-plugins - A bundle of plugins for Geany

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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=512217





--- Comment #8 from Jonathan Underwood <jonathan.underwood@xxxxxxxxx>  2009-07-17 12:51:01 EDT ---
(In reply to comment #6)
> (In reply to comment #5)
> > Also, please try to use %{name} rather than "geany-plugins" in the file list.  
> 
> Why? The guidelines only demand that macro usage is consistent. This means not
> to mix e. g. %{buildroot} and $RPM_BUILD_ROOT. The rest is up to the
> maintainer.

Because at present there is an *inconsistent* mixture of %{name} and
"geany-plugins" used in the specfile. Consistency is the key here, I agree.

> Last but not least I suggest to split this package into different subpackages.  

Yes - Dominic and I discussed this. The present barrier to doing that is that
the po files are not split up for each plugin. Once that's resolved it would be
possible to split them up. I think Dominic has a scheme to do that from the
unified po files, but I'll let him respond on that :).

-- 
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.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]