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

Jussi Lehtola <jussi.leht...@iki.fi> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-review?              |fedora-review+

--- Comment #8 from Jussi Lehtola <jussi.leht...@iki.fi> 2010-07-25 15:25:14 
EDT ---
I'm sorry if my pedantic style has in any way offended you.

The main thing holding back the review was the source url. The "we are
upstream" exception was discussed in FESCo and was removed.

http://meetbot.fedoraproject.org/fedora-meeting/2009-08-07/fedora-meeting.2009-08-07-17.00.log.html#l-87

IIRC the reasoning was that "we are upstream" is not a valid excuse, as in the
spirit of free software the code should be made easily availables to other
people as well. And, if I'm not mistaken, fedorahosted.org was set up after
discussion to provide a means for the easy distribution of the sources.

Every one of the former "we are upstream" packages (including Anaconda,
http://git.fedoraproject.org/git/anaconda.git that you mentioned in comment #7)
that I have looked at in the recent past have become available on
fedorahosted.org. The project list is rather lengthy already.

**

Had a maintainer had a look at this review when I originally made it, more than
a year ago, the source url discussion would not have taken place as the
guidelines changed later on.

It's not in my interests to bug people unwarrantedly, so this package is 

APPROVED

although I still think the source should be made available at fedorahosted.org.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
_______________________________________________
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to