Closed Bug 839775 Opened 11 years ago Closed 11 years ago

delete build/win32/mozconfig.vs2010 from mozilla-central

Categories

(Firefox Build System :: General, defect)

x86_64
Windows 7
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: jimm, Assigned: jimm)

References

Details

Attachments

(1 file)

AFAIK, we don't use this anymore. I'm not sure if this file should be deleted or updated to include the 8.0 sdk paths.
Blocks: elm-merge
I'm fine with deleting it. Maybe there are some people out there who use it though.
Attached patch patchSplinter Review
we can always add it back if it breaks something somewhere.
Assignee: nobody → jmathies
I can't imagine it's useful to anyone outside of releng what with all the hardcoded paths.
Summary: build/win32/mozconfig.vs2010 needs the 8.0 sdk update → delete build/win32/mozconfig.vs2010 from mozilla-central
(In reply to Ted Mielczarek [:ted.mielczarek] from comment #3)
> I can't imagine it's useful to anyone outside of releng what with all the
> hardcoded paths.

SeaMonkey releng relies on this being here too. So I'd prefer/ask that we hold off on removing it.
(In reply to Justin Wood (:Callek) from comment #4)
> (In reply to Ted Mielczarek [:ted.mielczarek] from comment #3)
> > I can't imagine it's useful to anyone outside of releng what with all the
> > hardcoded paths.
> 
> SeaMonkey releng relies on this being here too. So I'd prefer/ask that we
> hold off on removing it.

I didn't realize that. Why isn't SM using 64-bit builders?
(In reply to Jim Mathies [:jimm] from comment #5)
> I didn't realize that. Why isn't SM using 64-bit builders?

Multiple Reasons:

* SeaMonkey is built on community machines, not "MoCo" releng machines
* SeaMonkey does not have any systems imaged with win64
* SeaMonkey runs tests on the same system as it builds on, which means w2k3, which matches relatively closely with what MoCo supports.

We do want to find a solution to match even better, but without more VMs or Hardware thats impossible short term.
per comment 6
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → WONTFIX
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: