What version of Jxplorer are you using? It is from the svn?

Do you have a diff file of your changes?

On 10/19/2010 9:50 PM, Richard Gomes wrote:
> Hi guys,
>
> I've recently downloaded JXplorer and I found two issues in script 
> jxplorer.sh and I also found another potential issue.
>
>
> 1st issue:
>
> When running jxplorer.sh from any directory (except from the installation 
> directory) or when there are symlinks, the script may fails if an user HAS a 
> valid Java environment defined.
>
> The solution is easy: We only need to move these lines...
>
> basename=`basename $0`
> dirname=`dirname $0`
> cd ${dirname}
> dirname=`pwd`
>
> ... to the top, which means that $dirname will always contain the actual 
> directory where jxplorer.sh is sitting on, regardless of the location where 
> it is launched, regardless presence of symlinks or
> not, regardless a valid Java environment defined or not.
>
>
> 2nd issue:
>
> JAVAV must be defined relative to $OPTJX and not the relative to the 
> arbitrary location "/opt/jxplorer".
>
> #wrong: JAVAV=/opt/jxplorer/jre/bin/java
> JAVAV=${OPTJX}/jre/bin/java
>
>
> Potential issue:
>
> When an user DOES NOT have a Java environment defined, jxplorer.sh provides 
> one, which is a relative directory called jre.
> Well... I've downloaded the zipped Java distribution (not a pre-packaged 
> distribution to my environment) and there's no "jre" directory in there! 
> which means that jxplorer.sh will fail anyway when an
> user does not have a Java environment configured.
>
> In fact... it does not make sense to distribute a jre directory because we 
> don't know which architecture is need... x86? PowerPC? ARM? 32bits? 64bits? 
> whatever?
>
> So, in this case, jxplorer.sh should simply provide a message and quit.
>
> Please find attached the jxplorer.sh file I changed in order to address all 
> these issues I mentioned above. I've tested in order to simulate an user 
> which HAS a valid Java environment defined and also
> when no Java environment is defined. I removed unused code which would only 
> pollute the script instead of provide any useful history information of 
> changes.
>
> I hope it helps.
>
>
> Cheers :)
>
>
>
>
> ------------------------------------------------------------------------------
> Download new Adobe(R) Flash(R) Builder(TM) 4
> The new Adobe(R) Flex(R) 4 and Flash(R) Builder(TM) 4 (formerly
> Flex(R) Builder(TM)) enable the development of rich applications that run
> across multiple browsers and platforms. Download your free trials today!
> http://p.sf.net/sfu/adobe-dev2dev
>
>
>
> _______________________________________________
> Jxplorer-users mailing list
> Jxplorer-users@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/jxplorer-users

-- 

  Douglas E. Engert  <deeng...@anl.gov>
  Argonne National Laboratory
  9700 South Cass Avenue
  Argonne, Illinois  60439
  (630) 252-5444

------------------------------------------------------------------------------
Nokia and AT&T present the 2010 Calling All Innovators-North America contest
Create new apps & games for the Nokia N8 for consumers in  U.S. and Canada
$10 million total in prizes - $4M cash, 500 devices, nearly $6M in marketing
Develop with Nokia Qt SDK, Web Runtime, or Java and Publish to Ovi Store 
http://p.sf.net/sfu/nokia-dev2dev
_______________________________________________
Jxplorer-users mailing list
Jxplorer-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jxplorer-users

Reply via email to