Closed Bug 235479 Opened 21 years ago Closed 5 years ago

html templates assume cvsroot heirarchy of 'mozilla'

Categories

(Webtools Graveyard :: MXR, defect)

x86
FreeBSD
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: Chris.Yeh, Assigned: Chris.Yeh)

Details

Attachments

(1 obsolete file)

The mozilla LXR assumes that all indexed trees exist under a 'mozilla' directory
at the toplevel of /cvsroot. This causes the generated bonsai urls to be fixed
one, and only one main directory from the cvsroot.

I.E. if you have a new project which sits in:

/cvsroot/newproj

The urls from the template-* files will all generate bonsai urls that start with
http://bonsai/cvslog.cgi?file=mozilla/$pathname



Which is wrong, because we really want a generated url that has:

http://bonsai/cvslog.cgi?file=newproj/$pathname
Patch creates a new config variable named 'bonsaicvsroot' that will determine
the correct starting directory name for bonsai urls by finding the last element
of the path to the source and inserting that into the template dynamically
using the existing variable replacement system.
Comment on attachment 142177 [details] [diff] [review]
patch to LXR config and template system

>@@ -797,6 +800,7 @@

the callback lists appear to be aligned as a two column list, please either
realign them or change the whitespace to the more traditional single space.
Attachment #142177 - Flags: review+
Assignee: endico → Chris.Yeh
Comment on attachment 142177 [details] [diff] [review]
patch to LXR config and template system

mozilla/webtools/lxr/lib/LXR/Config.pm 	1.6
mozilla/webtools/lxr/lib/LXR/Common.pm 	1.27
Attachment #142177 - Attachment is obsolete: true
QA Contact: timeless → lxr

mxr is gone, mass closing.
https://searchfox.org/ is a much better alternative.

Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → WONTFIX
Product: Webtools → Webtools Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: