On 7/25/2010 8:35 PM, "Martin v. Löwis" wrote:
Am 26.07.2010 02:24, schrieb Terry Reedy:
To review a patch on the tracker, I have to read and try to make sense
of the raw diff file. Sometimes that is easy, sometimes not.
*After* a patch is applied, I can click the revxxxx link and then the
'text changed' link and see a nice, colored, side-by-side web-pace view
created by ViewVC. Is there any way a 'view' button could be added,
along with the current edit and remove buttons, to produce the same web
page and make it easier to review patches *before* commitment?
You should be using Rietveld for that.
There is no mention of Rietveld in the tracker docs that I could fine.
Did I miss something?
That fact that one needs to go to http://codereview.appspot.com/
rather than the rietveld page was not immediately obvious from 'use
Rietveld'.
> That may not be so easy - you'll have to authenticate to Rietveld from
> Roundup.
This is why I suggested ViewVC -- it already is at python.org and could
potentially be directly accessed with a button.
> The other way 'round actually works: if you put rep...@bugs.python.org
> into the reviewer list when uploading to Rietveld, Rietveld changes
> get automatically posted to the tracker. If you then also chose a
> sensible title (ie. one that includes [issueXXX]), Roundup will be able
> to associate it with the right issue.
These are less obvious.
Here is what I have gathered so far.
Open patch in browser tab to get its url.
Go to http://codereview.appspot.com/
Login in with Google account
Go to Create issue form
Copy and paste url into url field
To get comments mailed back to tracker,
put Issuexxxx in title and
put rep...@bugs.python.org in reviewer list
Should I open a tracker issue to add something to the tracker doc?
--
Terry Jan Reedy
_______________________________________________
Python-Dev mailing list
Python-Dev@python.org
http://mail.python.org/mailman/listinfo/python-dev
Unsubscribe:
http://mail.python.org/mailman/options/python-dev/archive%40mail-archive.com
