[m-rev.] diff: fix MC++ compilation rule

[Fergus Henderson]

On 05-Nov-2002, Peter Ross <pro at missioncriticalit.com> wrote:
> Fix a bug in my previous change to fix a bug in fjh's previous change.
> Revert back to using '/out:', and instead add the flag '/link' so that '/out:'
> is passed to the linker instead to avoid problems compiling the library.

That's not going to work, I'm pretty sure.  See MS_CL_LIBS in
library/Mmakefile.  You'll end up passing "/link /out:... /link ..."
when building the library in the il grade, and then the linker will complain
that "/link" is not a valid option.

What was wrong with the original approach of using `/o' (or was it `-o'?)?

By the way, I recently remembered why we use the (undocumented?) dash
options such as "-clr", etc.  options rather than the (documented)
slash options such as "/clr".  In fact cl supports both "-" and "/" as
the option character.  The original rationale for us using "-" was to
avoid confusion between Unix path names and option names.  However, this
rationale does not appear to have been documented in the source anywhere,
and using "-" has caused me some confusion because the "-" options were
not documented in the cl help message, so I didn't know what they did.

We should be consistent about which option character we use when
invoking cl.  My preference would be to use "/" because AFAIK the "-"
variants are not documented and hence might be less likely to be supported
in future versions.

-- 
Fergus Henderson <fjh at cs.mu.oz.au>  |  "I have always known that the pursuit
The University of Melbourne         |  of excellence is a lethal habit"
WWW: <http://www.cs.mu.oz.au/~fjh>  |     -- the last words of T. S. Garp.
--------------------------------------------------------------------------
mercury-reviews mailing list
post:  mercury-reviews at cs.mu.oz.au
administrative address: owner-mercury-reviews at cs.mu.oz.au
unsubscribe: Address: mercury-reviews-request at cs.mu.oz.au Message: unsubscribe
subscribe:   Address: mercury-reviews-request at cs.mu.oz.au Message: subscribe
--------------------------------------------------------------------------