proto-1.0 23: Autoconf configuration for documentation generation

Index: ChangeLog from Simon Odou <simon@lrde.epita.fr> * configure.ac: Add documentation generation support (from Olena). configure.ac | 14 ++++++++------ 1 files changed, 8 insertions(+), 6 deletions(-) Index: configure.ac --- configure.ac (revision 22) +++ configure.ac (working copy) @@ -39,7 +39,10 @@ metalic/mlc/Makefile olena/Makefile olena/oln/Makefile - olena/oln/config/pconf.hh:olena/oln/config/pconf-hh.in]) + olena/oln/config/pconf.hh:olena/oln/config/pconf-hh.in + doc/ref/out/exdoc.config:doc/ref/out/exdoc.config.in + doc/ref/exdoc.mk:doc/ref/exdoc.mk.in + doc/ref/doxygen.config:doc/ref/doxygen.config.in]) ### Olena components. @@ -191,10 +194,9 @@ AC_ARG_WITH([doc], [Doxygen is needed to build the doc], - [], [AC_CHECK_PROG([DOXYGEN], [doxygen], [doxygen]) - AC_CHECK_DOXYGEN_VERSION - ]) + AC_CHECK_DOXYGEN_VERSION], + []) ## if test -z "$DOXYGEN"; then ## AC_MSG_WARN([`doxygen' not found -- `doxygen' must be installed to generate the documentation]) @@ -214,12 +216,12 @@ ### The reference manuals directory is a component. - OLN_NON_COMPONENT([doc/ref], + OLN_COMPONENT([doc/ref], [doc-ref], [oln_cv_build_ref], [the Olena reference manuals], [OLN_DOC], - [doc/ref/Makefile], + [doc/ref/Makefile], [dnl Configuration for the manuals directory. AC_ARG_VAR([TAR], [the tape archiver program])

Simon Odou <simon@lrde.epita.fr> wrote:
+ AC_CHECK_DOXYGEN_VERSION],
Ça existe ça ?? Sinon, c'est une assez mauvaise pratique d'utiliser AC comme préfixe pour des macros maison. -- Didier Verna, didier@lrde.epita.fr, http://www.lrde.epita.fr/~didier EPITA / LRDE, 14-16 rue Voltaire Tel.+33 (1) 44 08 01 85 94276 Le Kremlin-Bicêtre, France Fax.+33 (1) 53 14 59 22 didier@xemacs.org

Didier Verna <didier@lrde.epita.fr> writes:
Simon Odou <simon@lrde.epita.fr> wrote:
+ AC_CHECK_DOXYGEN_VERSION],
Ça existe ça ?? Sinon, c'est une assez mauvaise pratique d'utiliser AC comme préfixe pour des macros maison.
J'ai fait comme Akim dans TC pour bison. La macro s'appelle AC_CHECK_BISON1875b. tc/trunk/tc/config/bison.m4 -- Damien Thivolle damien@lrde.epita.fr

Simon Odou <simon@lrde.epita.fr> writes:
Index: ChangeLog from Simon Odou <simon@lrde.epita.fr>
* configure.ac: Add documentation generation support (from Olena).
configure.ac | 14 ++++++++------ 1 files changed, 8 insertions(+), 6 deletions(-)
Index: configure.ac --- configure.ac (revision 22) +++ configure.ac (working copy) @@ -39,7 +39,10 @@ metalic/mlc/Makefile olena/Makefile olena/oln/Makefile - olena/oln/config/pconf.hh:olena/oln/config/pconf-hh.in]) + olena/oln/config/pconf.hh:olena/oln/config/pconf-hh.in + doc/ref/out/exdoc.config:doc/ref/out/exdoc.config.in + doc/ref/exdoc.mk:doc/ref/exdoc.mk.in + doc/ref/doxygen.config:doc/ref/doxygen.config.in])
### Olena components. @@ -191,10 +194,9 @@
AC_ARG_WITH([doc], [Doxygen is needed to build the doc], - [], [AC_CHECK_PROG([DOXYGEN], [doxygen], [doxygen]) - AC_CHECK_DOXYGEN_VERSION - ]) + AC_CHECK_DOXYGEN_VERSION], + [])
Nickel d'avoir changé ça, je ne l'avais jamais remarqué :)
## if test -z "$DOXYGEN"; then ## AC_MSG_WARN([`doxygen' not found -- `doxygen' must be installed to generate the documentation]) @@ -214,12 +216,12 @@
### The reference manuals directory is a component.
- OLN_NON_COMPONENT([doc/ref], + OLN_COMPONENT([doc/ref], [doc-ref], [oln_cv_build_ref], [the Olena reference manuals], [OLN_DOC], - [doc/ref/Makefile], + [doc/ref/Makefile], [dnl Configuration for the manuals directory.
AC_ARG_VAR([TAR], [the tape archiver program])
Euh pourquoi remettre la doc en COMPONENT ? Par défaut, on ne veut pas qu'elle soit construite, imagine un peu l'utilisateur qui fait `make', il va se prendre une compilation d'une heure dans la figure parce que la doc va être recompilée... En plus c'est en contradiction avec le changement sur le AC_ARG_WITH([doc], ....) OLN_NON_COMPONENT, ca oblige à lancer configure avec --with-doc pour que la doc soit construite. Bref, je pense qu'il est bien mieux de laisser la doc en OLN_NON_COMPONENT. -- Damien Thivolle damien@lrde.epita.fr

Damien Thivolle <damien@lrde.epita.fr> writes:
Simon Odou <simon@lrde.epita.fr> writes:
- OLN_NON_COMPONENT([doc/ref], + OLN_COMPONENT([doc/ref], [doc-ref], [oln_cv_build_ref], [the Olena reference manuals], [OLN_DOC], - [doc/ref/Makefile], + [doc/ref/Makefile], [dnl Configuration for the manuals directory.
AC_ARG_VAR([TAR], [the tape archiver program])
Euh pourquoi remettre la doc en COMPONENT ? Par défaut, on ne veut pas qu'elle soit construite, imagine un peu l'utilisateur qui fait `make', il va se prendre une compilation d'une heure dans la figure parce que la doc va être recompilée... En plus c'est en contradiction avec le changement sur le AC_ARG_WITH([doc], ....)
OLN_NON_COMPONENT, ca oblige à lancer configure avec --with-doc pour que la doc soit construite. Bref, je pense qu'il est bien mieux de laisser la doc en OLN_NON_COMPONENT.
Merci, j'ai corrigé. Et au passage, je me suis rendu compte d'un autre problème à propos de Doxygen. Sa présence était vérifié dès que l'option --with-doc était donnée alors que son utilisation nécessite --with-doc-ref, non ? -- Simon Odou simon@lrde.epita.fr
participants (3)
-
Damien Thivolle
-
Didier Verna
-
Simon Odou