Open Bug 1272184 Opened 5 years ago Updated 3 years ago

Remove hacks that allow VS2013 builds running on Windows XP SP2


(Firefox Build System :: General, defect)

Windows XP
Not set


(Not tracked)


(Reporter: emk, Unassigned)



VS2015 does not require this hack. It will become a dead weight once VS2013 support has been dropped.
No longer blocks: 1186064
Depends on: 1186064
Bug 1023941, parts 1 through 5, were mostly backed out by bug 1035125, parts 1 through 5.

As far as I know the only bits remaining are some dead defines:

And this fallout from having both a static and dynamic CRT loaded:

I'm a bit scared of removing the latter, even though it's now unnecessary, because it was such a hairy bug to diagnose and fix, and it took a long time to get reported. If we ever go back to a static CRT for firefox.exe, we'd have to go through that process again. Ted, how do you feel about removing it?
Flags: needinfo?(ted)
I'm not a big fan of leaving dead code in, but I agree that leaving ourselves a super-hard-to-diagnose footgun isn't great either. Maybe we can just add a comment in `browser/app/` with a pointer to the bug saying if you ever set `USE_STATIC_LIBS = True` there, re-apply that fix?
Flags: needinfo?(ted)
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.