[Bug 554243] Review Request: moovida - Media Center

bugzilla at redhat.com bugzilla at redhat.com
Mon Mar 8 07:36:06 UTC 2010


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

--- Comment #10 from Alex Lancaster <alexl at users.sourceforge.net> 2010-03-08 02:36:01 EST ---
(In reply to comment #9)
> (In reply to comment #8)
> > (In reply to comment #7)
> > > I can review them if Matthias can't.    
> > 
> > Hi Peter, thanks for the offer, but are you a sponsor?  Since this is Graeme's
> > first package, he has to be reviewed by a sponsor, otherwise I would have
> > reviewed this myself long ago.  That was why Matthias volunteered (since he is
> > a sponsor).    
> 
> The reviewer and sponsor are generally different people. The sponsor helps
> Graeme through the process that the reviewer completes. I would ask the list
> for help.

That's not my understanding:

"The Reviewer can be any Fedora account holder, who is a member of the packager
group. There is one exception: If it is the first package of a Contributor, the
Reviewer must be a Sponsor."

from:

https://fedoraproject.org/wiki/Package_Review_Process#Reviewer

but if others are OK with it, I'm fine with the sponsor doing the final
sign-off, as I'm keen to get this package updated ASAP.

> Also as mentioned by Jason each package will need to be split out into a
> separate review as each package needs to go through the following guidelines
> https://fedoraproject.org/wiki/Packaging:ReviewGuidelines and then have a CVS
> request which will confuse things as to what package needs what fixed. See bug
> 553717 as an example.

I understand, but I think that because of the package interdependencies that it
makes sense to review this as a single package.  How about opening up separate
bugs for each tarball, but doing the main review/discussion (at least
initially) in one bug to allow cross-package issues to be discussed in one
plac?.  Then the final review checklist can be posted to the relevant bug and
the CVS request done in the separate bug.

> Finally if its just a rename you can do a rename request for the package which
> is quicker. See 517300 for an example. Graeme will still need a sponsor for
> that.    

I think a full review is in order here because of the time since the last
update.

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