Andreas, (and /usr/local savvy guys!)
Please tell me how things have gone. Basically I'm waiting for these issues to be solved to make the 0.7.2 release.
With the current version it does no longer work *quite* out of the box on the BSD's. It will work when doing a CPPFLAGS=/usr/local/include LDFLAGS=/usr/local/lib ./configure but this is not what I call out of the box :-).
I'm not an expert in tunning the system's directories, but I wonder why if /usr/local has part of the system, it is not included in the search path in BSDs. Doesn't make sense to me (nut I don't know much here). OTOH, for instance, /usr/local is not a part of the system in Slackware, and I can't find it with: `gcc -print-search-dirs`. Doubts started when I received this mail, from Manuel Menal: [mmenal at hurdfr.org] <q> Hey, While porting Dillo to the new implementation of POSIX threads for GNU/Hurd (code name: "Rubbish, I asked for mine with Minced Garlic, Please Take this back"), I found some minor bugs in the Dillo "installation system". The first bug I found out was a bug that was already reported as #349 in the Dillo bug report system. It seems a quite simple and not so annoying bug, but in fact it is quite pervert, as it makes the configure misdetect what is installed in the system: when it checks for jpeglib.h (& others), it tries to preprocess a simple C file which only includes jpeglib.h, and check if there is some output. If there is some output, then it assumes there is no jpeglib.h. Which is not true in this case: the output is just warning of cpp, which complains about the -I/usr/local/include (cpp 3.1 already has /usr/local/include it its system directories). [...] </q> Thinking that /usr/local was for extensions, I decided to remove the forced include from configure.in. Can anyone shed some light on this topic?
Also, if png.h is on libpng/png.h and NOT as well in png.h, it will complain, it does not find png.h. However, this warning is misleading, since configure will find everything just fine and dillo compiles and works just fine.
I understand.
I attach a patch which fixes both problems.
The /usr/local stuff really only is an issue for gcc3.x since gcc3.x already has /usr/local as default directories, so I test for that case. That *should* be ok.
(explained above)
The png.h stuff is more or less my original solution :-) ... there was a reason for that convoluted approach ... :-)
Yes, I added the break clause because it tested libpng twice and added two "-lpng" in my system. I'll commit the patch as you sent here (which BTW, includes the "break"). Cheers Jorge.-