:gerard-majax [PTO 01/08/2023-10/09/2023] Assignee
	Description • 8 years ago

https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&revision=cf971ca7f132471feca4f62b4cdd523358f4507a&filter-searchStr=b2g&filter-tier=1&filter-tier=2&filter-tier=3

	:gerard-majax [PTO 01/08/2023-10/09/2023] Assignee
	Comment 1 • 8 years ago

Adding $(info MOZCONFIG_CONTENT=$(MOZCONFIG_CONTENT)) on line 114 gives this. I'm a bit worried about the part referring to errors.

	:gerard-majax [PTO 01/08/2023-10/09/2023] Assignee
	Comment 2 • 8 years ago

Local revert of bug 1278990 and the issue is gone ...

	Rob Thijssen [:grenade (EET/UTC+0300)]
	Comment 3 • 8 years ago

I'm looking at my changes to mozconfig.cache now to see what could be causing this. Prime suspect, from an instinct perspective, is the wget call...
https://dxr.mozilla.org/mozilla-central/source/build/mozconfig.cache#49

	:gerard-majax [PTO 01/08/2023-10/09/2023] Assignee
	Comment 4 • 8 years ago

Review commit: https://reviewboard.mozilla.org/r/63158/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/63158/

	:gerard-majax [PTO 01/08/2023-10/09/2023] Assignee
	Comment 5 • 8 years ago

https://treeherder.mozilla.org/#/jobs?repo=try&revision=3ac4aa3be670

	Rob Thijssen [:grenade (EET/UTC+0300)]
	Comment 6 • 8 years ago

Comment on attachment 8769181 [details]
Bug 1285529 - Do not set SCCACHE when CCACHE_DIR is there

https://reviewboard.mozilla.org/r/63158/#review60018

win32,win64 still get their sccache config so lgtm.

	:gerard-majax [PTO 01/08/2023-10/09/2023] Assignee
	Comment 7 • 8 years ago

Not good, it is still failing.

	Rob Thijssen [:grenade (EET/UTC+0300)]
	Comment 8 • 8 years ago

Review commit: https://reviewboard.mozilla.org/r/63184/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/63184/

	Rob Thijssen [:grenade (EET/UTC+0300)]
	Comment 9 • 8 years ago

Review commit: https://reviewboard.mozilla.org/r/63186/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/63186/

	:gerard-majax [PTO 01/08/2023-10/09/2023] Assignee
	Comment 10 • 8 years ago

Comment on attachment 8769181 [details]
Bug 1285529 - Do not set SCCACHE when CCACHE_DIR is there

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/63158/diff/1-2/

	Rob Thijssen [:grenade (EET/UTC+0300)]
	Comment 11 • 8 years ago

Comment on attachment 8769216 [details]
Bug 1285529 - Ensure wget failure do not break us

https://reviewboard.mozilla.org/r/63184/#review60026

	:gerard-majax [PTO 01/08/2023-10/09/2023] Assignee
	Comment 12 • 8 years ago

Green! https://treeherder.mozilla.org/#/jobs?repo=try&revision=97a66fdb0fa0&filter-tier=1&filter-tier=2&filter-tier=3&selectedJob=23560679

	Pulsebot
	Comment 13 • 8 years ago

Pushed by alissy@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/ae11e3061f5b
Do not set SCCACHE when CCACHE_DIR is there r=grenade

	Pulsebot
	Comment 14 • 8 years ago

Pushed by kwierso@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/751186781016
Backed out changeset ae11e3061f5b to allow me to do a merge, relanding later
https://hg.mozilla.org/integration/autoland/rev/d5d91aa3a430
Do not set SCCACHE when CCACHE_DIR is there r=grenade CLOSED TREE

	Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)
	Comment 15 • 8 years ago

So, I backed this out to fix up a merge conflict that popped up when trying to merge m-c to autoland today.
I then did the merge and relanded this patch to autoland after the conflict was dealt with.

Then I realized that the conflict was caused by my backout from bug 1278990, and this patch's changes were only to the chunk of the file 1278990's patch added, so that whole chunk should be removed again.

Re-backed out in https://hg.mozilla.org/integration/autoland/rev/0215f0988f9c

	Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)
	Comment 16 • 8 years ago

So I guess this patch should just be folded into the patch from bug 1278990's patch when it re-lands?

	:gerard-majax [PTO 01/08/2023-10/09/2023] Assignee
	Comment 17 • 8 years ago

(In reply to Wes Kocher (:KWierso) from comment #16)
> So I guess this patch should just be folded into the patch from bug
> 1278990's patch when it re-lands?

Looking at the reasons of the backout, it is possible that this bug explicitely fixes this problem :/

	:gerard-majax [PTO 01/08/2023-10/09/2023] Assignee
	Comment 18 • 8 years ago

Marking as dupe since bug 1278990 will integrate those fixes.