[Bug 393041] Review Request: libzip - C library for reading, creating, and modifying zip archives

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

Summary: Review Request: libzip - C library for reading, creating, and modifying zip archives


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





------- Additional Comments From pertusus@xxxxxxx  2007-11-23 13:32 EST -------
Although I don't disagree with hacks working around upstream issues
I think that we should never ever sacrifice quality for deadlines.

If the reason for using this hack is deadlines, this is a very bad
idea, in my opinion. It adds the possibility for 'blitz' reviews of 
dubious quality and I have seen too much reviews like that, we should 
really be especially watchful and not let anything go that way. For a 
recent example, you can see how the ltsp related reviews begun, 
we should not go down that road. And for merge reviews there is also
a lot of similar issues (although some come more from ignorance than
something else).

Here it is much less extreme since the review was quite correct,
however if the hack is there because of the time it is wrong.

-- 
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, or are watching someone who is.

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