>
>
> - All jars are enumerated with their full file name incl. version info.
> This will break when the jars are updated.
> I wrote chk_spreadsheet_support.m in such a way that version info in the
> name of the jar is ignored.
>

I thought that if specific jars versions are desired for a specific version
of the io package (such as in the case of xercesImpl-2.9.1), the io's
PKG_ADD can search for the desired version.

Assume that after a user has installed the tree and after a while you have
updated the io package such that a newer version of a jar is required, the
newer PKG_ADD will look for the updated version. If your
chk_spreadsheet_support  already checks for the compatibility with founded
jars from the jars content, than I agree that including the versions in the
name is redundant.


>
> - Like Benjamin, you've put the jars in /bin. Benjamin did this (had to do
> this) because he invoked file_in_path(), that has no knowledge of /lib
> I'd rather have the jars in /lib. After all, the jars are Java libraries,
> not executables (yeah I know about the .dlls in /bin).
> The location of /lib can easily be found using
>   libdir = octave_config_info ("libdir")
>
>
Currently, at least with the current installation tree, the lib subdir
includes:

a. Libraries to link with when when using mkoctfile from within octave (some
pkconfig files are also included).
For that reason (mkoctfile) this dir is set by -L option by the LDFLAGS env.
var. as the directory in which libraries to link with reside.

b. The octave lib sub dir as generated by the build process.

I am not an expert of how to construct the tree, I have just followed
Benjamin's conventions. If I will simply put the jars in the lib subdir,
they will mix with the libraries to link with !

- Why did you include chk_spreadsheet_support() in a for...end loop?
> AFAICS, all you need to do is to check for a loaded Java package, and if
> loaded, call chk_spreadsheet_support ('/path/to/jars').
> chk_spreadsheet_support itself will then sort it all out: search for the
> needed jars there and add the ones it found to the classpath.
>
>
I have tried to do it in a more general way such that jars may reside in a
set of directories (not all of them in a single place) abd the file_in_path
founds the specific path for each jar in a loop.

If you assume that all the jars are in a single directory, you are right -
giving a single dir name as an input to  chk_spreadsheet_support does it
all.

I've attached a revised PKG_ADD (for io package). It works for me.
> Will you (in turn) check if you agree with it, please?
>

It is OK, assuming that all jars are located in a single known place. I
would suggest may be that you will enhance chk_spreadsheet_support to get as
input a search path (several delimited paths) rather than a single path (you
can also utilize file_in_path for such searching), this will make the io
package more 'robust' to different installations and platforms. I have no
experience with installations and directories on other platforms, and the
PKG_ADD should be general enough to properly do the job for most of them.

Also, I would add a respective PKD_DEL that will remove the jars paths from
the java package if the io package is unloaded.
------------------------------------------------------------------------------
Get your Android app more play: Bring it to the BlackBerry PlayBook 
in minutes. BlackBerry App World™ now supports Android™ Apps 
for the BlackBerry® PlayBook™. Discover just how easy and simple 
it is! http://p.sf.net/sfu/android-dev2dev
_______________________________________________
Octave-dev mailing list
Octave-dev@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/octave-dev

Reply via email to