<div class="gmail_quote">2010/1/19 Frédéric Buclin <span dir="ltr"><<a href="mailto:lpsolit@gmail.com">lpsolit@gmail.com</a>></span><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Because nobody would review it. Also, you don't need to do a CVS diff. A<br>
local diff would work too.<br>
<div class="im"><br></div></blockquote><div>Actually meant that it would be nice if the patch file was created on submission. Rather than client side. So the review process would be no different.<br>Not saying that doing the diff is that hard, more that sometimes the 5 minutes it takes is 5 minutes someone might not have at a given moment in time. The result being that a perfectly good fix or enhancement gets relgated to the back burner and forgotten even though the work is already done. <br>
 </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">Because Bugzilla would have no idea what the file is. When you attach<br>
plain text, the name of the file is not included. Bugzilla cannot guess.<br>
<div class="im"><br></div></blockquote><div>True, I'm not saying as of this second it would work, more that if the appropriate tags were added the base files it would work, eg tag with path/name and tag with version. <br>
</div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div class="im">
<br>
</div>Well, that's inaccurate as Bugzilla lets you view the whole file from a<br>
patch with only a few lines of context. That's how I always review<br>
patches. No need for the whole file.<br>
<br></blockquote><div>Ok figured that out: Click 'View attachment as Diff' then click 'File' in "Context:
    (Patch / 
    File)" so right in front of me :-)</div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>
LpSolit<br></blockquote></div>