[Bug 556346] Review Request: stage - A 2.5D multi-robot simulator

[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=556346

--- Comment #24 from Ralf Corsepius <rc040203@xxxxxxxxxx> 2010-03-13 00:24:15 EST ---
(In reply to comment #23)

> > And? Add appropriate Requires: where applicable.
> > 
> > In it's present shape this package's *.pc's are simply plain defective
> 
> - Then in other words, almost not applicable for this package
Rubbish ... All that needs to be done would be to replace those hardcoded libs
from stage.pc this package erroniously pulls in with corresponding "Requires:".


> > > > * The doc clearly are devel docs.
> > > > => IMO, this *-doc package should be abandoned and be merged into the package
> > > > it documents: "*-devel".    
> > > 
> > > - I don't think splitting such many document files (non man-3 files)
> > >   into -doc is bad idea.    
> > I disagree - The number one rule wrt. to docs in Fedora is to put docs into
> > those packages they are documenting. doxygen generated developer docs clearly
> > belong into *-devel. 
> 
> - So it is not a blocker.
Yes ... nevertheless this package is badly packaged, IMNSHO due to its reviewer
having done a poor job.

I will file bugs against this package as soon as it is available in bugzilla
and (provided FESCO's ongoing absurdidities) will cast negative bodji votes.

-- 
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.
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review

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