Closed Bug 758914 Opened 11 years ago Closed 10 years ago always pulls from


(MailNews Core :: Build Config, defect)

Not set


(Not tracked)

Thunderbird 16.0


(Reporter: neil, Assigned: standard8)




(2 files, 1 obsolete file)

I have several trees that all pull from a local clone that I pull from, however I accidentally ran without pulling my local clones first. Imagine my surprise when proceeded to pull my mozilla folder anyway from, leaving me with a failed build.
Attached patch Possible patch (obsolete) — Splinter Review
This fixed things locally for me. I don't know why bug 688856 changed that code in the first place, but it seems wrong, so this patch backs that out.
Attachment #627516 - Flags: review?(gozer)
My only concern is how this interacts with hg shared stuff - if they correctly set the repo to clone from when using when using Probably needs a quick test.
Comment on attachment 627516 [details] [diff] [review]
Possible patch

So as I feared I just tried this locally and it would break the current use of hg share that we're using on the tinderboxes.

I think I have a possible alternative that I can propose.
Attachment #627516 - Flags: review?(gozer) → review-
Attached patch Alternative fixSplinter Review
I think this will fix it. If we've got a mozilla directory, but haven't specified the repo on the command line, then we'll fall back to using the value in .hg/hgrc.

This should work even for hg share as it stores the http://hg... repo in the mozilla/.hg/hgrc file.
Attachment #627653 - Flags: review?(bugspam.Callek)
Comment on attachment 627653 [details] [diff] [review]
Alternative fix

Happier if gozer reviews, as I haven't tested with hg share/hgtool myself yet.

But I'm leaving a request open on me, incase gozer takes longer and I find a free moment sooner.
Attachment #627653 - Flags: review?(gozer)
Attachment #627653 - Flags: review?(bugspam.Callek)
Attachment #627653 - Flags: review?(gozer) → review+
Attachment #627653 - Flags: review?(bugspam.Callek)
Attachment #627516 - Attachment is obsolete: true
Checked in:
Assignee: nobody → mbanner
Closed: 10 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 16.0
So that I can run from another working directory again.
Attachment #633483 - Flags: review?(mbanner)
Comment on attachment 633483 [details] [diff] [review]
Improve error messages, add topsrcdir

Looks good.
Attachment #633483 - Flags: review?(mbanner) → review+
Comment on attachment 633483 [details] [diff] [review]
Improve error messages, add topsrcdir

Pushed comm-central changeset a4dcfae6ddad.
You need to log in before you can comment on or make changes to this bug.