ejabberd - Comments for "expat.h: No such file or directory"

The patch is complete, at

legoscia — Thu, 09 Aug 2007 23:26:53 +0000

The patch is complete, at least concerning the expat part. Seems I never bothered to look at the SVN version ☺

And this patch should really be applied to conftools/expat.m4 in expat CVS. I'll try to write them a more coherent bug report some day.

But the patch does confuse me a bit. aclocal.m4 is supposed to be automatically generated by aclocal, using bits and pieces from all the libraries that the program in question uses (this is described in the automake manual), but it seems that the aclocal.m4 of ejabberd is very much handcrafted, having grown lots of local hacks over the years.

Now I'm really confused ☺

I am not sure to understand

mremond — Thu, 09 Aug 2007 12:02:14 +0000

I am not sure to understand right now. However, I can consider a fix proposal.
Do you have a suggestion ?
How is it related to this fix: https://support.process-one.net/browse/EJAB-204
It the patch incomplete ?

--
Mickaël Rémond
Process-one

Hm… memory is hazy.

legoscia — Mon, 06 Aug 2007 13:16:06 +0000

Hm… memory is hazy. Apparently I just commented on that bug, which seemed to be about the same problem…

The bug is that the

legoscia — Mon, 06 Aug 2007 12:38:21 +0000

The bug is that the configure script doesn't properly distinguish between CPPFLAGS and CFLAGS. CPPFLAGS names the flags that are necessary for both preprocessing and compiling, while CFLAGS names the flags that are only needed for compiling, i.e:

cpp $CPPFLAGS foo.c

cc $CFLAGS $CPPFLAGS foo.c

However, the configure tests for expat don't include CPPFLAGS when using the compiler.

I reported this to the expat people four years ago, and was a bit surprised by their response (they basically misunderstood what the bug report was about), but didn't bother responding as most people never will understand autoconf anyway, and my workaround is good enough for me.

So yes, the bug is really in expat, but ejabberd does include the erroneous code in src/aclocal.m4.

It does not seem a bug in

mremond — Mon, 06 Aug 2007 08:43:04 +0000

It does not seem a bug in configure. To me it does the right thing. if you pass CFLAGS (expat is a C program), it works correctly.
I do not know if it is the job of configure to look into /usr/local for expat.

what do you think ?

--
Mickaël Rémond
Process-one

It's a bug in the configure

legoscia — Sun, 05 Aug 2007 15:00:55 +0000

It's a bug in the configure script. You need to set CFLAGS to the same value as CPPFLAGS.