Closed Bug 1274765 Opened 3 years ago Closed 3 years ago

Caught exception: vcs share base not defined; refusing to operate sub-optimally

Categories

(Firefox OS Graveyard :: GonkIntegration, defect)

ARM
Gonk (Firefox OS)
defect
Not set

Tracking

(firefox49 fixed)

RESOLVED FIXED
Tracking Status
firefox49 --- fixed

People

(Reporter: gerard-majax, Assigned: gerard-majax)

References

Details

(Whiteboard: fixed-in-pine)

Attachments

(2 files)

this started after recent m-c -> pine merge
MozReview-Commit-ID: BbNtgKArGbV
Attachment #8755127 - Flags: review+
Blocks: 1270317
That looks like the proper fix.
Well, it's not helping :/ https://treeherder.mozilla.org/#/jobs?repo=pine&filter-tier=1&filter-tier=2&filter-tier=3&selectedJob=28127

What is wrong, you have an idea Wander?
Flags: needinfo?(wcosta)
(In reply to Alexandre LISSY :gerard-majax from comment #4)
> Well, it's not helping :/
> https://treeherder.mozilla.org/#/jobs?repo=pine&filter-tier=1&filter-
> tier=2&filter-tier=3&selectedJob=28127
> 
> What is wrong, you have an idea Wander?

I commented at Bug 1270317, it feels like mercurial on docker images is outdated.
Flags: needinfo?(wcosta)
Instead of using mockbuilds-repo to install mercurial, we do so through
pip.
Attachment #8755518 - Flags: review?(garndt)
Comment on attachment 8755518 [details] [diff] [review]
Upgrade mercurial on docker images. r=garndt

Review of attachment 8755518 [details] [diff] [review]:
-----------------------------------------------------------------

I see nothing wrong with this.  Given that these images probably will be retired in the future as b2g work happens less, I don't think we should invest the time in making these images as robust as our desktop equivalents
Attachment #8755518 - Flags: review?(garndt) → review+
backed out for failign https://treeherder.mozilla.org/logviewer.html#?job_id=28551580&repo=mozilla-inbound#L57 , sorry guys
Flags: needinfo?(lissyx+mozillians)
Flags: needinfo?(lissyx+mozillians)
Issue is gone, ok to re-apply the patch?
Flags: needinfo?(cbook)
(In reply to Wander Lairson Costa [:wcosta] from comment #15)
> Issue is gone, ok to re-apply the patch?

relanded !
Flags: needinfo?(cbook)
https://hg.mozilla.org/mozilla-central/rev/1fa20da1e3ae
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.