Fixups for MSVC > 8 after bug 678161

RESOLVED FIXED in mozilla9

Status

defect
RESOLVED FIXED
8 years ago
a year ago

People

(Reporter: glandium, Assigned: glandium)

Tracking

Trunk
mozilla9
All
Windows 7
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: fixed-in-bs)

Attachments

(1 attachment)

(Assignee)

Description

8 years ago
One issue is in bug 678161 comment 11:

+ifeq ($(_MSC_VER),1500)
+JSSHELL_BINS += $(DIST)/bin/msvcr100.dll
+endif

1500 should be 1600.

Another issue is that we don't remove the CRT dlls for MSVC versions we don't build against.
http://hg.mozilla.org/mozilla-central/rev/c37f23b36e54
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla9
Attachment #555677 - Attachment description: Add the CRT dlls we don't use to the remove-files lists. Also fix inclusion of the MSVC 10 CRT in the js shell archive → Add the CRT dlls we don't use to the remove-files lists. Also fix inclusion of the MSVC 10 CRT in the js shell archive [Checked in: Comment 3]
Assignee: nobody → mh+mozilla
Flags: in-testsuite-
Isn't @BINPATH@ unneeded/unwanted in removed-files.in?
(Assignee)

Comment 5

8 years ago
Ah yes, that's true. Please file a followup bug.
Depends on: 696422
(In reply to Mike Hommey [:glandium] from comment #5)
> Ah yes, that's true. Please file a followup bug.

I filed bug 696422.

Updated

a year ago
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.