cmpilato@xxxxxxxxxx wrote:
> Author: cmpilato
> Date: 2006-03-30 16:05:45-0800
> New Revision: 1299
>
> Modified:
> trunk/lib/viewvc.py
>
> Log:
> * lib/viewvc.py
> (Request.run_viewvc): Move the 'rev' param auto-translation up a bit,
> and don't perform pathtype checks if we're already in need-to-redirect
> mode.
Russ (or others), I'd love some additional eyes on r1298, r1299, and r1300.
I'm flying all day tomorrow, and the server that houses my mail is going
down for scheduled maintenance tonight, so I dunno if I'll be able to
realistically react to anything until sometime Saturday.
You'll see that I went with 'revision' instead of 'r' for the new parameter.
Can't explain it -- it just looks better to me. I'm pretty sure I got all
the instances of 'rev', and managed to remember that CVS needs a mapping of
a different kind ('rev' => 'revision' instead of 'rev' => 'pathrev').
Initial testing seems to look good, but I'm a little concerned about the
ordering of redirect stuffs that I attempted to fix in r1299.
Anyway, feel free to hack on it -- I've not yet updated svn.collab.net, but
I want to do so ASAP (as well as notify SourceForge admins that they should
accept this change as soon as they can).
--
C. Michael Pilato <cmpilato@xxxxxxxxxx>
CollabNet <> www.collab.net <> Distributed Development On Demand
signature.asc
Description: OpenPGP digital signature
|
