MPS issue job001365

Titlebuild broken on w3 (Windows) with Visual C++ 7.0
Statusclosed
Priorityessential
Assigned userRichard Kistruck
OrganizationRavenbrook
Descriptionbuild broken on w3 (Windows) with Visual C++ 7.0

Bruce Mitchener reported a slew of Windows build failures [Mitchener]. Many of these applied equally to VC++ 6.0, and are covered by job001366.

The remaining failures -- covered here -- only occur with VC++ 7.0.

RHSK 2006-04-10
I believe that the default "all" build:
    nmake /f w3i3mv.nmk
does not work with Visual C++ 7.0.

Some of the failures are while building "mps.lib"; there is no simple workaround.
AnalysisRHSK 2006-04-10
Currently I can't access a machine with VC++ 7.0. I have attempted speculative fixes of the problems in [Mitchener]. I have tested these on VC++ 6.0 only, and left this issue "open" until I can check that it builds under VC++ 7.0.
How foundcustomer
Evidence[Mitchener] http://mailman.ravenbrook.com/pipermail/mps-discussion/2005-March/000028.html
Observed in1.100.1
Introduced in1.100.0
Created byRichard Kistruck
Created on2006-04-10 13:34:55
Last modified byGareth Rees
Last modified on2014-03-07 10:09:38
History2006-04-10 RHSK Created.
2012-08-15 RB Closed: we no longer support VC7 and MPS builds cleanly on VC10.

Fixes

Change Effect Date User Description
159469 closed 2006-06-29 17:37:12 Richard Kistruck MPS: attempt to fix w3i3mv "all" build with VC7:
(unknown if these changes are effective, but they should be safe)
1. mpm.c: +1 to array size to make room for terminator of string initializer;
2. testlib.c: disable warning 4702: unreachable code, for va_end.
(integrate from version/1.106/...@158055)
158055 open 2006-04-07 15:57:08 Richard Kistruck MPS: attempt to fix w3i3mv "all" build with VC7:
(unknown if these changes are effective, but they should be safe)
1. mpm.c: +1 to array size to make room for terminator of string initializer;
2. testlib.c: disable warning 4702: unreachable code, for va_end.