Closed
Bug 558252
Opened 15 years ago
Closed 15 years ago
Add the Camino Hg repo to MXR
Categories
(Webtools Graveyard :: MXR, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: alqahira, Assigned: timeless)
References
Details
Attachments
(1 file)
1.87 KB,
patch
|
reed
:
review+
|
Details | Diff | Splinter Review |
We need MXR to index the new Camino hg repo, http://hg.mozilla.org/camino/
"Starting Points" (http://mxr.mozilla.org/) Name:
Camino
"Starting Points" description:
This is this the source code of the Camino browser.
Initial checkout command is:
hg clone http://hg.mozilla.org/camino/ $camino_or_destination_of_choice
Update command is standard (where pwd = $camino_or_destination_of_choice):
hg pull -u
Please let me know if you need any additional information.
Reed: this does the right thing (including bootstrapping).
I won't push it until someone from somewhere agrees that they want this added (I'm pretty sure we do).
smokey: congrats and welcome to hg :)
Assignee: nobody → timeless
Attachment #438074 -
Flags: review?(reed)
Summary: Please add the Camino Hg repo to MXR → Add the Camino Hg repo to MXR
Comment 2•15 years ago
|
||
Comment on attachment 438074 [details] [diff] [review]
pending push
>+ This is this the source code of the <a
>+href="http://caminobrowser.org/">Camino browser</a>.
"This is this the" ? Also might want to mention "for Mac" or something... dunno.
>+ /^(?:(?:.*)-(?:central|tracing)|camino)$/ && do {
Is camino the only weird non -central one here, or could we go ahead and add support for others, too?
>+ unless (-d "$src_dir/.hg") {
You could probably get away with |-d "$src_dir"| here, but I like the .hg check, so fine.
>+ my $dir = basename($src_dir);
>+ print LOG `$TIME $HGCOMMAND $HGCLONE https://hg.mozilla.org/$dir $src_dir`;
>+ }
> print LOG `cd $src_dir; $TIME $HGCOMMAND $HGUPDATE $STDERRTOSTDOUT`;
Seems like you're doing some useless work by doing an update immediately after cloning... you could just do if/else or unless/else if you wanted, but I don't particularly care.
r=me with the html issue fixed... the rest are your call.
Attachment #438074 -
Flags: review?(reed) → review+
http://hg.mozilla.org/webtools/mxr/rev/505ae7c12ab0
someone will have to add cron scripts as usual
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 4•15 years ago
|
||
(In reply to comment #3)
> http://hg.mozilla.org/webtools/mxr/rev/505ae7c12ab0
Thanks timeless!
> someone will have to add cron scripts as usual
How does that work? Do I need to file a bug somewhere to make that happen, or?
Updated•6 years ago
|
Product: Webtools → Webtools Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•