autoconf-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: 16-ac-lang-preproc-require.patch


From: Akim Demaille
Subject: Re: 16-ac-lang-preproc-require.patch
Date: 13 Nov 2000 09:50:21 +0100
User-agent: Gnus/5.0807 (Gnus v5.8.7) XEmacs/21.1 (Channel Islands)

| >     (AC_LANG_PREPROC(Fortran 77), AC_LANG_PREPROC_REQUIRE): New
| >     macros.  Issue a warning when looking for the Fortran 77
| >     preprocessor instead of an error.
| 
| I'm not sure that we really want to go ahead after that warning. Howvalid
| "configure" will be if we ignore it?

Maybe I didn't think enough about this issue, but since I have no
experience of Fortran...

BTW, this weekend I had a stupid idea: how about introducing
`Preprocessed Fortran 77' as a language, which has ac_ext=.F?

| >     For some reason (don't ask), this revamping revealed that
| >     AC_PROG_CC_STDC, because of the `break', does not clean its tmp
| >     files.
| 
| Well, we don't test the macros that are AC_REQUIRE'd. This doesn't seem to
| be a very clean appoach, since the caller can cleanup for the callee.
| mktests.sh may need to be changed.

I understand your concern, but I'm also afraid to see the test suite
slowing down too much.  Personally I prefer it as is currently.

| >  # This file is part of Autoconf.                       -*- Autoconf -*-
| >  # Programming languages support.
| > -# Copyright 1992, 93, 94, 95, 96, 98, 99, 2000
| > +# Copyright 2000
| 
| You meant 1992-2000?

Nope.  I must have not paid attention when I created aclang: it was
just this year.  And I don't think we should track the copyrights of
the content of the file.

| > +# AC_REQUIRE_CPP
| > +# --------------
| > +# Require the preprocessor for the current language.
| > +# FIXME: AU_ALIAS once AC_LANG is officially documented (2.51?).
| 
| Huh? AC_LANG is documented.

I mean AC_LANG_*, i.e. in the present case AC_LANG_PREPROC_REQUIRE.

BTW, I was not sure and hesitated between PREPROC and PREPROCESSOR.
I'd like to have other opinions.



| > +# AC_LANG_PREPROC(Fortran 77)
| > +# ---------------------------
| > +# Find the C compiler.  Must be AC_DEFUN'd to be AC_REQUIRE'able.
| 
| Find the C compiler? Make sure to change the comment.

*blush* :)

| > +AC_DEFUN([AC_LANG_PREPROC(Fortran 77)],
| > +[m4_warn([syntax],
| > +         [$0: No preprocessor defined for ]_AC_LANG)])
| 
| See above. This should be a fatal error unless I'm missing something.

Really, I'm not sure.  In the case of Fortran 77, we just don't want
to check for the preproc.




reply via email to

[Prev in Thread] Current Thread [Next in Thread]