|
On 23.08.2021 21:33, Michaud Michael wrote:
Ede,
Just tested last BeanShellEditor. Works well.
If you don't mind, now that all extensions have been updated to use new FeatureInstaller / I18N, I'd like to make a full update of OpenJUMP PLUS by copying jar files before you start changes on packaging. PLUS distro has been broken for several months, and it would be nice to have a complete version.
that'll just add more binary clutter to trunk again. i'd really really prefer we just, step by step replace the lib/{ext,plus}/jars with the maven downloads instead. now is the perfect point in time. the result will be a working PLUS distro as _plus_ trunk will be clean as well. it's like 'two birds one stone' :)
As you like. I proposed to do it to give us more time to achieve this new migration but if you think it will not take long, go ahead. I should be able to move new extension versions to OpenJUMP distro tonight.
if they work fine locally at you machine, they will as well when added as downloadable distro zips. each time you add binaries to the source repo it's size grows by the size of the added binary. let's stop doing that!
I see. And a question about how you plan to manage dependencies of extension. For example, the lib directory of the new BeanshellEditor contains jedit and the BeanShell jar file, but the later is already included in OpenJUMP CORE.
right. wasn't aware. just pushed a commit removing the dependency as it is already in CORE, which is a dependency of the extension.
check the beanshell distro zip 'bsheditor-extension-2.0.0.zip' created via 'mvn package'. it's clean now.
>Any way to avoid such duplication ?
of course. clean pom.xml setup :) if CORE already contains the dependency, then there is no need actually disruptive to add it again.
Oh ! Did not realize that adding OpenJUMP to an extension via maven also makes all transitive dependencies available.
>On the other hand, some extensions may need specific versions of a library. Do you want to use some specialized library to manage extensions and dependencies or just maven ?
yeah, no. we currently and never did support diverging dependency versions for anything in OJ. status quo is, first jar found in classpath containing the package class will be used. i have a a lose plan to extend the classloader to actually contain one classloader per extension, which would effectively enable different dependency versions within OJ, but that's just an idea for now!
extensions either have to work with what is in OJ CORE or apply for the dependency in OJ to be modified accordingly.
OK, I thought we already had a classloader per plugin. No problem. I don't think we need it for now, and my hope is that we'll never need it ;-)
finally if you want i'll even do the distro/pom modification for all extensions and prepare the OJ pom although i feel it is needed that you are familiar with it as well. ..sunny regards ede
OK, go ahead. Let me know if you need help for some tests.
Michaël
PS I think sqlite driver is the only one extension I did not keep uptodate.
_______________________________________________
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
| 