[m-rev.] for review: further Cygwin / MS C# fixes

Peter Ross pro at missioncriticalit.com
Fri Feb 10 14:45:04 AEDT 2012


On 10 February 2012 04:36, Julien Fischer <juliensf at csse.unimelb.edu.au> wrote:
>
> For review by Peter Ross.
>
> Branche: main, 11.07
>
> Fix further problems with building the csharp grade on Cygwin using the
> Microsoft compiler.  There were two main problems:
>
> (1) Mercury.config.bootstrap had not been updated so the environment
> and csharp compiler types for the stage 1 compiler were incorrect.
>
> (2) The '/' in Microsoft style /OPTIONS was sometimes being treated
> as a path separator for the purposes of path conversion.
>
> scripts/Mercury.config.bootstrap.in:
>        Bring this file back into sync with Mercury.config.
>
> compiler/compile_target_code.m:
>        Use - style options with the csharp compiler rather than / style
>        ones.
>        (In addition to causing the above problem, the use of / also tends
>        to cause trouble with POSIX style shells in general - we already
> avoid
>        the use of / style options with MSVC for the same reason.)
>
>        Break some overlong lines.
>
> compiler/handle_options.m:
>        Break an overlong line.
>
This looks fine.

--------------------------------------------------------------------------
mercury-reviews mailing list
Post messages to:       mercury-reviews at csse.unimelb.edu.au
Administrative Queries: owner-mercury-reviews at csse.unimelb.edu.au
Subscriptions:          mercury-reviews-request at csse.unimelb.edu.au
--------------------------------------------------------------------------



More information about the reviews mailing list