RESOLVED WONTFIX 21022
bugs.webkit.org should discourage patches larger than 20k (or some arbitrary size) 
https://bugs.webkit.org/show_bug.cgi?id=21022
Summary bugs.webkit.org should discourage patches larger than 20k (or some arbitrary ...
Eric Seidel (no email)
Reported 2008-09-23 04:50:44 PDT
bugs.webkit.org should discourage patches larger than 20k (or some arbitrary size) I think our tools could assist us here. I think it could be helpful towards education of contributers, and towards sanity of reviewers, if bugs.webkit.org would display some sort of warning page when you tried to upload a patch which was larger than 50k/20k or some arbitrary size. Especially when non-committers or first-time contributers upload 100k patches, its just a waste of reviewers time and their time, trying to fight it down to something that's more reviewable. I don't think technology is the final solution to that problem, but making it clear during the patch upload process that your patch exceeds the "sane" size limit, could be useful. Heck there are other things bugzilla (or other scripts/tools) could scan for, like lack of ChangeLog too... but that's the subject for another bug.
Attachments
Add attachment proposed patch, testcase, etc.
anitria
Comment 1 2012-10-04 00:55:59 PDT
(In reply to comment #0) > bugs.webkit.org should discourage patches larger than 20k (or some arbitrary size) > > I think our tools could assist us here. I think it could be helpful towards education of contributers, and towards sanity of reviewers, if bugs.webkit.org would display some sort of warning page when you tried to upload a patch which was larger than 50k/20k or some arbitrary size. Especially when non-committers or first-time contributers upload 100k patches, its just a waste of reviewers time and their time, trying to fight it down to something that's more reviewable. I don't think technology is the final solution to that problem, but making it clear during the patch upload process that your patch exceeds the "sane" size limit, could be useful. > > Heck there are other things bugzilla (or other scripts/tools) could scan for, like lack of ChangeLog too... but that's the subject for another bug.
Alexey Proskuryakov
Comment 2 2012-10-04 10:28:20 PDT
svn-create-patch has the warning. Eric, is this still something that needs to be looked into?
Note You need to log in before you can comment on or make changes to this bug.