[m-rev.] for review: clean up environment variable handling
Simon Taylor
stayl at cs.mu.OZ.AU
Sun Apr 13 15:46:08 AEST 2003
On 11-Apr-2003, Fergus Henderson <fjh at cs.mu.OZ.AU> wrote:
> On 11-Apr-2003, Simon Taylor <stayl at cs.mu.OZ.AU> wrote:
> >
> > Clean up the handling of environment variables.
> ...
> > scripts/Mercury.config.in:
> > Fix syntax errors.
> ...
> > Index: scripts/Mercury.config.in
> ...
> > @@ -24,62 +24,63 @@
> > # be used by the substitution for SHARED_LIBS.
> > MATH_LIB=$(MERCURY_MATH_LIB)
> >
> > +DEFAULT_MERCURY_LINKAGE=@DEFAULT_LINKAGE@
>
> That change wasn't mentioned in the log message.
>
> > NEWS:
> > Document the change.
>
> You didn't post a diff for the NEWS file.
>
> Otherwise that looks fine.
Estimated hours taken: 0.5
Branches: main
Clean up the handling of environment variables.
scripts/Mmake.rules:
scripts/c2init.in:
scripts/mgnuc.in:
scripts/ml.in:
scripts/mmc.in:
scripts/parse_ml_options.sh-subr.in:
scripts/binary_step.in:
Remove support for the environment variables which were
previously used to override the location of the standard
library (MERCURY_ALL_C_INCL_DIRS, MERCURY_ALL_MC_C_INCL_DIRS,
MERCURY_INT_DIR, MERCURY_C_LIB_DIR, MERCURY_MOD_LIB_MODS,
MERCURY_TRACE_LIB_MODS).
scripts/Mercury.config.in:
Fix syntax errors.
Set the DEFAULT_MERCURY_LINKAGE variable here rather
than in the mmc script.
scripts/mmc.in:
Setting DEFAULT_MCFLAGS is no longer required -- mmc
now reads its configuration information from the
Mercury.config file. This also fixes a bug which
caused failures in the `mmc --make' tests in
debugging grades on mundroo -- `mmc --make' wasn't
including /usr/local/lib and /usr/local/include in
the search paths for libraries and headers.
configure.in:
Check that the installed compiler is capable of
reading the Mercury.config file.
Bootstrap tag: bootstrap_20030413_read_config_file.
NEWS:
doc/user_guide.texi:
Document the change.
Index: NEWS
===================================================================
RCS file: /home/mercury1/repository/mercury/NEWS,v
retrieving revision 1.307
diff -u -u -r1.307 NEWS
--- NEWS 20 Mar 2003 22:51:35 -0000 1.307
+++ NEWS 10 Apr 2003 14:09:32 -0000
@@ -144,6 +144,11 @@
* The Mercury compiler now uses `.' and not `:' as the module separator
in all output.
+* The environment variables which were previously used to override the
+ location of the standard library (MERCURY_ALL_C_INCL_DIRS,
+ MERCURY_ALL_MC_C_INCL_DIRS, MERCURY_INT_DIR, MERCURY_C_LIB_DIR,
+ MERCURY_MOD_LIB_MODS, MERCURY_TRACE_LIB_MODS) have been removed.
+
Portability improvements:
* Nothing yet.
Index: configure.in
===================================================================
RCS file: /home/mercury1/repository/mercury/configure.in,v
retrieving revision 1.357
diff -u -u -r1.357 configure.in
--- configure.in 21 Mar 2003 08:00:25 -0000 1.357
+++ configure.in 10 Apr 2003 14:23:49 -0000
@@ -194,10 +194,10 @@
").
EOF
if
- # Test for the `--bug-intermod-2002-06-13' option.
+ # Test for the `--read-config-file-2003-03-01' option
echo $BOOTSTRAP_MC conftest >&AC_FD_CC 2>&1 &&
$BOOTSTRAP_MC \
- --bug-foreign_import-2002-08-06 \
+ --read-config-file-2003-03-01 \
--halt-at-warn $link_static_opt conftest \
</dev/null >&AC_FD_CC 2>&1 &&
test "`./conftest 2>&1 | tr -d '\015'`" = "Hello, world"
--------------------------------------------------------------------------
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