win64 buildbot doesn't set WIN32_REDIST_DIR

VERIFIED FIXED

Status

Release Engineering
General
P3
normal
VERIFIED FIXED
6 years ago
4 years ago

People

(Reporter: m_kato, Assigned: m_kato)

Tracking

other
x86_64
Windows Server 2008
Dependency tree / graph

Firefox Tracking Flags

(firefox12 unaffected, firefox13 fixed, firefox14 verified)

Details

(Whiteboard: [fixed by bug 672799] [configs])

(Assignee)

Description

6 years ago
When I check build log (https://tbpl.mozilla.org/php/getParsedLog.php?id=8075007&tree=Firefox&full=1) for win64, WIN32_REDIST_DIR isn't set.

So package (ftp://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/latest-mozilla-central/firefox-11.0a1.en-US.win64-x86_64.zip) has no MSVC DLLs.

Is this expected?
I don't know.

What are the consequences?

Comment 2

5 years ago
(In reply to Armen Zambrano G. [:armenzg] - Release Engineer from comment #1) 
> What are the consequences?

AIUI, the build will be unusable *unless* you happen to already have a Windows dev environment on your machine that contains those DLLs.

Should be a pretty simple env change to get that var added.
OS: Windows Vista → Windows Server 2008
Priority: -- → P3
Whiteboard: [configs]
(Assignee)

Updated

5 years ago
Duplicate of this bug: 718323
Blocks: 713132
Blocks: 694371
This is also a blocker of bug 713132: the sooner the better...
(In reply to Chris Cooper [:coop] from comment #2)
> Should be a pretty simple env change to get that var added.

Ping for action.


(In reply to Serge Gautherie (:sgautherie) from comment #4)
> This is also a blocker of bug 713132: the sooner the better...

Blocking bug 713132 means we keep getting unnoticed regressions there...
(In reply to Serge Gautherie (:sgautherie) from comment #5)

> Ping for action.

What is the status?

> Blocking bug 713132 means we keep getting unnoticed regressions there...

This is needed to fix Firefox (12 and) 13.
Thanks for your interest but win64 is not a priority for product drivers.
We already bent left and right to speed up the work to have the disappointment of not being wanted after all.

It might sound repetitive but with more than a thousand bugs open for our team I can only come to say "patches welcome" (as much I would have wanted not to need to say it).
Sorry Serge but I hope you can understand that we can't fix everything.
[D'oh, I never received, or missed, your comment.]

(In reply to Armen Zambrano G. [:armenzg] - Release Engineer from comment #7)

> Thanks for your interest but win64 is not a priority for product drivers.

I don't actually care about win64: I care about this bug blocking (bug 713132) "fixing" all (the other) platforms (and applications) :-/

> We already bent left and right to speed up the work to have the
> disappointment of not being wanted after all.

I sympathize!

> It might sound repetitive but with more than a thousand bugs open for our
> team I can only come to say "patches welcome" (as much I would have wanted
> not to need to say it).

Well, just saying it's currently "postponed" is enough to let me know I shouldn't continue wasting my time waiting for this (P3) to happen: thanks for that.

> Sorry Serge but I hope you can understand that we can't fix everything.

Then I'll try to exclude win64 in bug 713132...
Depends on: 569375
(Assignee)

Comment 9

5 years ago
Now, since firefox for Win64 swtichs to VS2010, mozconfigs/win64/vs2010-mozconfig has WIN32_REDIST_DIR environment value for VS2010.
(In reply to Makoto Kato from comment #9)
> Now, since firefox for Win64 swtichs to VS2010,
> mozconfigs/win64/vs2010-mozconfig has WIN32_REDIST_DIR environment value for
> VS2010.

Good.
Ftr, that was bug 672799.

V.Fixed per bug 713132 comment 7.
*firefox14: verified.
*firefox13: fixed by bug 672799, and would be "unaffected, doesn't seem to build win64" anyway.
*firefox12: unaffected, doesn't seem to build win64.
Status: NEW → RESOLVED
Last Resolved: 5 years ago
status-firefox12: --- → unaffected
status-firefox13: --- → fixed
status-firefox14: --- → verified
Depends on: 672799
Resolution: --- → FIXED
Status: RESOLVED → VERIFIED
Assignee: nobody → m_kato
Whiteboard: [configs] → [fixed by bug 672799] [configs]
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.