Andrzej Bialecki wrote:
> Sami Siren wrote:
>>> How the code ended up in this place on Linux? The $cygwin condition
>>> should have prevented that, because it evaluates to true only on Cygwin,
>>> where this utility is required to translate the paths.
>>>
>>> You also changed the "if" syntax - before it was using the /bin/test
>>> utility to evaluate the expression, now it uses a shell built-in - I'm
>>> not sure if these two follow the same evaluation rules on all supported
>>> platforms ... Please revert it to the earlier syntax.
>>>
>>
>> revert it so it desn't work on linux, are you sure?
>>
>
> I'll make a patch and check that it works on Linux too - apparently it
> was me who botched the AND syntax in my previous commit... ;)
>
The effect of this problem was not purely cosmetic. I just realized it
disables the native libs because JAVA_LIBRARY_PATH does not get set. If
I change the if to
if [ test $cygwin -a "X${JAVA_LIBRARY_PATH}" != "X" ]; then
JAVA_LIBRARY_PATH=`cygpath -p -w "$JAVA_LIBRARY_PATH"`
fi
it works, ok for me.
--
Sami SIren
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Nutch-developers mailing list
Nutch-developers@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nutch-developers
