Re: make distcheck and minor Makefile discrepancies

[Philip Van Hoof <spam pvanhoof be>]

On Sun, 2006-11-12 at 12:22 +0100, �stein Gisn�wrote:

> I was referring to the COPYING in libtinymail-camel.
> http://svn.tinymail.org/svn/tinymail/trunk/libtinymail-camel/camel-lite/COPYING

Aha, I see. Well. the thingy here is, is that camel-lite is a fork of
evolution-data-server's Camel. Because both e-d-s and tinymail have the
same license, I simply decided to refer (in text) to the top-level
COPYING file. 

Which, as far as I know (unless tinymail's license would change, which
isn't going to happen), is correct in a legal way.

In my opinion shouldn't autotools try to touch that file ever. Just,
never. It has nothing to do with whatever license I'll use, so its
developers shouldn't even think about putting in scripts that touch my
license files.

By that I mean that it's clearly an autotools bug :(. I wonder how I can
tell autotools not to consider whatever I've put in the COPYING file.

> > About the other problem, you need to --enable-gkt-doc followed by a
> > normal make before doing make distcheck, and then it works.
> >
> > The normal make is to make that missing xml file.
> 
> Have you tried this yourself recently? I haven't completed the whole
> run yet, but I at least have to run autogen.sh with:
> ./autogen.sh --enable-gtk-doc --with-html-component=mozembed
> --enable-tests --enable-python-bindings
> 
> I also end up having gcc complain during 'make'. It can't resolve
> TnyCamelFolder in libtinymail-camel/tny-camel-nntp-folder.h and
> TnyCamelStoreAccount in
> libtinymail-camel/tny-camel-nntp-store-account.h. Adding #includes as
> shown in the patch (attached) works for me.

You forgot to attach this patch :).

> It looks like for example libtinymail-camel/tny-camel-imap-folder.h
> works in a similar way without this #include, so I suspect something
> else is wrong. Ideas anyone?

Your patch is probably correct, but it's not in attachment :)

> Later, during 'make distcheck' I get an error because some source
> files in po/POTFILES are missing. 'xargs -a po/POTFILES.in ls >
> /dev/null' gives me the clues.. *svn up* Seems like this and other
> problems have been fixed recently ;)

Indeed :)

> I also get an error when distcheck is running ./configure for the new
> dist. More precisely it happens when
> libtinymail-camel/camel-lite/configure.in is being run by distcheck:
> config.status: error: cannot find input file: camel/providers/imap4/Makefile.in
> configure: error: ../../../libtinymail-camel/camel-lite/configure
> failed for libtinymail-camel/camel-lite
> 
> It looks like the new libtinymail-camel/camel-lite/configure.in
> includes camel/providers/imap4/Makefile in its AC_OUTPUT while the
> source directory libtinymail-camel/camel-lite/camel/providers/imap4 is
> left of the distribution.

This has also been fixed by removing it from the AC_OUTPUT. For now, the
experimental imap4 Camel provider is not supported. It also wouldn't
really be used a lot by tinymail (the imap4 stuff of Camel for example
implements continuations, which is something I haven't really
experimented with yet).

> Now, after updating fixes from svn, make distcheck succeeds! Nice
> work! The only patches I still use are
> libtinymail-camel/{tny-camel-nntp-folder.h,tny-camel-nntp-store-account.h}.

:)


-- 
Philip Van Hoof, software developer at x-tend 
home: me at pvanhoof dot be 
gnome: pvanhoof at gnome dot org 
work: vanhoof at x-tend dot be 
http://www.pvanhoof.be - http://www.x-tend.be