Re: Patch: java/lang/natRuntime.cc

[Michael Koch <konqueror at gmx dot de>]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Am Montag, 18. August 2003 09:17 schrieb Ranjit Mathew:
> > +      char* libraryPath = ::getenv ("LD_LIBRARY_PATH");
>
> Sorry, but I don't think this is portable. For example,
> on Windows, PATH is used to search for DLLs. My memory
> is a bit hazy now, but on AIX it was something like
> LD_LOAD_LIBRARY_PATH.

Well, getenv() is used in one other place in this file too. Thats why 
I decided to use this. I can write a patch included the AIX variable 
later today

> > +      // Initialize with prefix.
> > +      sb->append (JvNewStringLatin1 (PREFIX "/lib"));
>
> On Windows, this might be the "bin" folder (or maybe not,
> depending on where the Win32 GCJ distributor decides to
> put the DLLs).

This file has many FIXMEs asking if this works on windows. I think we 
should commit this patch and then split this file into a POSIX and a 
WIN32 version.


Michael
- -- 
Homepage: http://www.worldforge.org/
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.2 (GNU/Linux)

iD8DBQE/QdLrWSOgCCdjSDsRAghbAJ9NskVwenKZddCYlRn6qm7uT0FwLACcCt9s
Og0J6JdYnfyIsXkhKILz0WM=
=tjw+
-----END PGP SIGNATURE-----