Re: [Poesia-devel] change in configure file

[simone m. <sim...@il...>]

On Fri, 10 Oct 2003 15:01:10 +0100
Neil Ireson <N.I...@dc...> wrote:

> Hurray,
>
> I've finally got the Italian Filter to compile.

  :)

>  However this has 
> involved making some changes to the "TextFilter/Italian/src/Makefile" 
> which should not interfer with anyone else's compilation... but might. 

Do you have committed these changes? Because don't see them on cvs server...(as
developper).

> Basically the Italian Filter only compiled if all the include and 
> library files are in some predefined directory therefore I've included a 
> few environmental variables to enable compilation when this is not the 
> case. These are
> 
> setenv MYSQL_INCLUDE_DIR $HOME/poesia/include/mysql
> setenv MYSQL_LIB_DIR $HOME/poesia/lib/mysql
> setenv LOOKDB_INCLUDE_DIR $HOME/poesia/include/lookdb
> setenv LOOKDB_LIB_DIR $HOME/poesia/lib/lookdb
> setenv PCRE_INCLUDE_DIR $HOME/poesia/include
> setenv PCRE_LIB_DIR $HOME/poesia/lib

I partially agree with you.
These setenv are appropriate only for your machine, in which MySQL is installed
in those paths.
I think that is more suitable to add to configure script a flag to specify
MySQL include and library paths, like --mysql-include and --mysql-library.

> Also I've removed the checking for lookdb and mysql from the 
> configuration script as this also relies on the packages being in a 
> standard directory.

see above... I think that these are useful check to aid a simple user to compile
the system. By the way, I think that all required libraries should be checked.
This is only my opinion.... 

What do you think?

ciao simone