[Top][All Lists]
[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.