Closed Bug 925894 Opened 11 years ago Closed 11 years ago

Windows b2g multilocale completely broken even on green runs on Win64-rev2

Categories

(Infrastructure & Operations Graveyard :: CIDuty, task)

x86_64
Windows Server 2008
task
Not set
normal

Tracking

(firefox26 fixed, firefox27 fixed, firefox-esr24 fixed, b2g-v1.2 fixed)

RESOLVED FIXED
Tracking Status
firefox26 --- fixed
firefox27 --- fixed
firefox-esr24 --- fixed
b2g-v1.2 --- fixed

People

(Reporter: philor, Assigned: glandium)

References

Details

(Keywords: regression)

Attachments

(1 file)

https://tbpl.mozilla.org/php/getParsedLog.php?id=28993659&tree=Fx-Team is a red Windows b2g build, red for some unclear reason, but https://tbpl.mozilla.org/php/getParsedLog.php?id=28994302&tree=Fx-Team is a "green" run containing all the same errors as the "c:/Program Files (x86)/Microsoft Visual Studio 10.0/VC/include/" path completely confuses things. https://tbpl.mozilla.org/php/getParsedLog.php?id=28978157&tree=Mozilla-Central, a recent m-c and thus not -rev2, doesn't have the same errors.
Depends on: 926035
This is surely going to fix the "No rule to make target 'c:/Program'" part of this bug. No idea about the rest.

Landed on birch to assess.
Attachment #817605 - Flags: review?(gps)
Assignee: nobody → mh+mozilla
Status: NEW → ASSIGNED
(In reply to Mike Hommey [:glandium] from comment #1)
> Created attachment 817605 [details] [diff] [review]
> Normalize cl generated dependencies before testing for spaces in them
> 
> This is surely going to fix the "No rule to make target 'c:/Program'" part
> of this bug. No idea about the rest.

I should explain why I expect this to work: we skip dependencies with a space in them when gathering from cl -showIncludes output on the premise that they are system paths (and, well, they confuse make, as per this bug). The problem with the -rev2 builds is that various paths have been set in the short path form (c:\Progra~2\...). And what cl.py does to normalize paths actually transforms such paths back to the long form (c:\Program Files (x86)\...). But the test for spaces is done before the normalization, so those paths are not skipped anymore.
Attachment #817605 - Flags: review?(gps) → review+
Comment on attachment 817605 [details] [diff] [review]
Normalize cl generated dependencies before testing for spaces in them

https://hg.mozilla.org/integration/mozilla-inbound/rev/df950d461d3e
Attachment #817605 - Flags: checked-in+
https://hg.mozilla.org/mozilla-central/rev/df950d461d3e
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Component: Platform Support → Buildduty
Product: Release Engineering → Infrastructure & Operations
Product: Infrastructure & Operations → Infrastructure & Operations Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: