[m-rev.] for review: mercury.bat

Peter Ross peter.ross at miscrit.be
Thu Apr 19 19:45:40 AEST 2001


On Thu, Apr 19, 2001 at 02:11:02PM +1000, Fergus Henderson wrote:
> Recently there has been some discussion about getting Mercury to work on
> Windows without Cygwin.  The change enclosed below is not a complete solution.
> But I'd appreciate comments about
> - whether this is a worthwhile goal
> - whether this approach is a reasonable approach to achieve that goal
> - whether I should commit this part now,
>   or whether we should wait until there is a solution
>   to problem (1) [see below] before committing this
> 
> The main drawback, I think, is that solving (1) is likely to require
> a fairly substantial amount of code duplication: scripts/ml is 1400
> lines of Bourne shell script.
> 

Not requiring cygwin for the build architecture is a worthwhile goal,
but isn't this what Simon is working on currently?

Maybe a better solution first would be to work on cutting down the size
of ml.  WHen this is done, then I think the double maintainence problem
would be substantially reduced.

I would be hesitant to commit this now, because even this solution
requires cygwin to be installed so that calls to invoke_shell_command
succeed inside the compiler.

[snip]

> +
> +echo on
> +%MC% %MERC_ALL_MC_C_INCL_DIRS% --cc "%CC%" --grade "%DEFAULT_GRADE%" --cflags-for-threads "%CFLAGS_FOR_THREADS%" --cflags-for-regs "%CFLAGS_FOR_REGS%" --cflags-for-gotos "%CFLAGS_FOR_GOTOS%" --c-flag-to-name-object-file "%CFLAG_TO_NAME_OBJECT_FILE%" --object-file-extension "%OBJECT_FILE_EXTENSION%" --num-real-r-regs "%NUM_REAL_R_REGS%" --num-real-r-temps "%NUM_REAL_R_TEMPS%" --conf-low-tag-bits "%LOW_TAG_BITS%" --bits-per-word "%BITS_PER_WORD%" --bytes-per-word "%BYTES_PER_WORD%" %HAVE_DELAY_SLOT% %HAVE_BOXED_FLOATS% %DEFAULT_OPT_LEVEL% -I "%INTDIR%" %1 %2 %3 %4 %5 %6 %7 %8 %9

Does this limit us to only passing 9 arguments to the compiler?
--------------------------------------------------------------------------
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
--------------------------------------------------------------------------



More information about the reviews mailing list