Summary: | JOGL ALL JAR File Change incl. it's Native Jar URL Derivation | ||
---|---|---|---|
Product: | [JogAmp] Jogl | Reporter: | Sven Gothel <sgothel> |
Component: | core | Assignee: | Sven Gothel <sgothel> |
Status: | RESOLVED FIXED | ||
Severity: | blocker | CC: | gouessej, org.jogamp, rami.santina, sgothel, xixsimplicityxix |
Priority: | --- | ||
Version: | 2 | ||
Hardware: | All | ||
OS: | all | ||
Type: | --- | SCM Refs: |
gluegen 9a71703904ebfec343fb2c7266343d37a2e4c3db
jogl a393e45613d87101dbb13763df263c2f9291d2d0
jogl-demos 1d1b473d08612d01d274642113ea37a7052daceb
joal faefb3ed4e9876dd1f756500caed3904604a11cf
jocl 95bf6160c908364fbce2f82df85063d12c9b279c
jocl-demos 0e6bf2a0a9b3df729d6984bdc1a857889a3c1bdf
|
Workaround: | --- |
Description
Sven Gothel
2012-06-25 21:46:05 CEST
Hi You should do it as early as possible or give it up in order to minimize the impact on all projects that rely on JogAmp. We cannot abandon Maven support, several contributors of thir party libraries really want us to provide it (Nifty GUI, Jzy3D, Ardor3D, etc...). (In reply to comment #1) > Hi > > You should do it as early as possible or give it up in order to minimize the > impact on all projects that rely on JogAmp. We cannot abandon Maven support, > several contributors of thir party libraries really want us to provide it > (Nifty GUI, Jzy3D, Ardor3D, etc...). Thank you for your agreement. I will do it now in branch 'maven' (gluegen, jogl). pushed maven branch of gluegen, jogl and jogl-demos - pls review. I have tested a few jar file scenarios manually (linux x86_64), - all existing all native JARs - missing all naytive JARs - all atomic [native] JARs works here. Further more a few optimization went into the GlueGen code avoiding repetitive ClassLoader look-ups and native JAR load attempts. (In reply to comment #3) > pushed maven branch of gluegen, jogl and jogl-demos - pls review. > > I have tested a few jar file scenarios manually (linux x86_64), > - all existing all native JARs > - missing all naytive JARs > - all atomic [native] JARs > works here. > > Further more a few optimization went into the GlueGen code > avoiding repetitive ClassLoader look-ups and native JAR load attempts. The GlueGen commit 9a71703904ebfec343fb2c7266343d37a2e4c3db handles the 'all' case, as well the 'atomic' JAR case. Hence the original disclaimer (no atomic) is invalid. Got the OK from Rami as well. Hence we have 4 core people who agree on the change (incl. Mark). Will merge to master to push joal and jocl as well and kick-off testing. this looks as a good standard with performance advantages, i vote for it... Regards, Rami locally tested w/ signed JARs, jogl & jogl-demos test applets and webstart (In reply to comment #7) > locally tested w/ signed JARs, jogl & jogl-demos test applets and webstart .. and maven |