[Octopus-devel] [Octopus-notify] svn commit: r6360 - in trunk: m4 src by xavier

[Xavier Andrade]

Hi David,

I did it this way, like gsl does, because for libxc octopus needs to know 
where both the library (lib/) and the modules (include/) are. If you just 
allow the user to pass the location of the library, they have to modify 
the compiler flags by hand to include the modules. I think we should do 
the same for other libraries that have the same problem (netcdf and 
etsf_io at least).

Cheers,

Xavier

On Mon, 22 Mar 2010, David Strubbe wrote:

> Xavier,
> 
> Why did you set it via --with-libxc-prefix? None of the other libraries have
> their locations specified in that manner, and I think it is confusing to do
> it all in different ways.  (Compare --with-netcdf, with-blas, etc.)
> 
> David
> 
> On Sat, Mar 20, 2010 at 5:37 PM, <octopus-notify at tddft.org> wrote:
>            Author: xavier
>              Date: Sun Mar 21 00:37:36 2010
>       New Revision: 6360
>         Changeset: http://www.tddft.org/trac/octopus/changeset/6360
>
>       Added:
>         trunk/m4/libxc.m4
>       Modified:
>         trunk/Makefile.am
>         trunk/configure.ac
>         trunk/src/common-rules.make
>
>       Log:
>       Now octopus does not compile libxc, it attemps to find an
>       installed
>       version (the location can be specified using the
>       --with-libxc-prefix
>       argument).
>
>       I kept the libxc directory until it is moved to its own
>       repository.
> 
>
>       _______________________________________________
>       Octopus-notify mailing list
>       Octopus-notify at tddft.org
>       http://www.tddft.org/mailman/listinfo/octopus-notify
> 
> 
> 
>