In preparation of Cyrus IMAP 2.5: autoconf and automake

Bron Gondwana brong at fastmail.fm
Tue May 8 18:15:59 EDT 2012


On Tue, May 08, 2012 at 11:44:49PM +0200, Dilyan Palauzov wrote:
> >That's so unmaintainable it's not funny.
> >
> >What's the plan here?
> 
> One possibility is to split this on many, many, many lines.  The
> other possibility is to replace thе long line with
> 
> EXTRA_DIST = \
>    sieve/addr.h sieve/sieve.h sieve/sieve_err.et sieve/tests \
> 
> this will include everything from the sieve/tests/ directory in the tarball.

Sounds much better (though see my comment about "one per line please"
for the same maintainability reasons.  Otherwise you either get
out-of-order monstrosities where you basically NEED to pattern match
on it with your editor just to see if something is already in there,
or you reflow every time you add something in the middle, causing
totally unreadable diffs.

Bron.


More information about the Cyrus-devel mailing list