libtool-patches
[Top][All Lists]
Advanced

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

Re: distchecking without FC?


From: Ralf Wildenhues
Subject: Re: distchecking without FC?
Date: Sat, 16 Jun 2007 11:13:36 +0200
User-agent: Mutt/1.5.13 (2006-08-11)

Hi Noah,

* Noah Misch wrote on Fri, Jun 15, 2007 at 02:42:04AM CEST:
> 
> It was possible to generalize the f77demo/fcdemo configure script
> hacks to make them work with no Fortran compiler at all.  Even so, it
> looked increasingly fragile.  Therefore, I took a different approach:
> list the files to distribute directly in the top-level Makefile.am,
> and do not bother to configure the test directories for `make dist'.

This would break down at the point when Autoconf or Automake decide to
generate a different set of files (and we try to support both older and
newer version).  Arguably this doesn't happen too often, and Stepan has
too little time to carry out his plans...

> If better, we could compromise between complexity
> and redundancy by generating the list of files at `bootstrap' time.

If you've already tried this, please post the patch.  Thanks.
Otherwise I suppose we can cross the bridge when we come to it
(you could put a FIXME above the list).

> In any case, `make dist' runs an order magnitude faster.

FWIW, I could care less about `dist' time.  `check' and `all' are much
more interesting.

> While examining this, I tried `make distcheck'.  It failed due to a
> `config.log' left in tests/fcdemo when its `configure' fails.  This
> patch incorporates a two-line fix to avoid that problem.

This is fine.

It seems the others like this patch, so I won't stand in the way.  ;-)

Cheers, and thanks for your work,
Ralf

> 2007-06-14  Noah Misch  <address@hidden>
> 
>       * Makefile.am (configure-subdirs, @DIST_MAKEFILE_LIST@, DIST_SUBDIRS):
>       Delete.
>       (cdemo_dist, demo_dist, depdemo_dist, f77demo_dist, fcdemo_dist)
>       (mdemo_dist, mdemo2_dist, pdemo_dist, tagdemo_dist): New variables.
>       (EXTRA_DIST): Add the contents of those variables.
>       (clean-local-legacy): Delete `config.log' in test directories.
>       * configure.ac (CONF_SUBDIRS, DIST_MAKEFILE_LIST): Delete.




reply via email to

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