classpath-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [cp-patches] Build system update


From: Guilhem Lavaux
Subject: Re: [cp-patches] Build system update
Date: Sun, 23 Oct 2005 18:51:34 +0200
User-agent: Mozilla Thunderbird 1.0.2 (X11/20050322)

Mark Wielaard wrote:
hi guilhem,

Hi Mark,


On Thu, 2005-10-20 at 21:20 +0200, Guilhem Lavaux wrote:

I have finally merged classpath into kaffe's tree. But to make things work smoothly I had to add a few hooks into configure.ac. This patch adds support for "--with-native-libdir" and "--with-glibj-dir". The first one moves the native libraries to another directory instead of ${libdir}/${PACKAGE} (which is still the default). The second one do the same for the java library.


This is probably an historic mistake now that I think about it. I know
of no other package that does that. But thanks for keeping the default
the same for now.

Are there similar flags for kaffe so it can pick up native jni libraries
and glibj from alternative directories? That would kind of complement
the "big merge" by making it hopefully possible in the future for a
packager to install them from gnu classpath once and reuse them for
multiple runtimes they install.


I've checked that classpath installs at the same place. So it should be ok for other runtimes. There isn't at the moment a flag in kaffe's configure to relocate the libraries. However it should be fairly easy to add it. The default paths can be modified using KAFFE* environment variables. So it is just a matter of modifying kaffe startup script.

Looks fine to me. But there is one typo (see below) and I would
appreciate it if you could test with an alternative runtime like jamvm
that currently uses the defaults to see if the "no configure options
given" experience still works out of the box with those.


+AC_CONFIG_COMMANDS([copy-vmmresources],[chmod 755 lib/copy-vmmetainf.sh])


One m to much, should be copy-vmresources.


Thanks ! It is because I tried various names before sticking to copy-vmresources. :)

Besides this I can check in the patch now.

Cheers,

Guilhem.




reply via email to

[Prev in Thread] Current Thread [Next in Thread]