task #9206
Solve cglib vs. cglib-nodep
100%
Description
Cglib and cglib-nodep are very similar libraries as they both include the cglib part but cglib-nodep also includes classes from asm. This has been done to be version independent from asm but has the drawback that it is not extendable (https://github.com/cglib/cglib/wiki ) .
We expect that cglib-nodep is sufficient for cdmlib and will test this.
All dependecies to cglib should be excluded therefore.
This is also to avoid multiple locations warnings from maven.
Also hibernate-cglib-repack probably can be removed. It is anyway only linked by cdmlib-remote. And it is not clear why.
For information on how cglib-nodep is build see: https://github.com/cglib/cglib/blob/master/cglib-nodep/pom.xml
Related issues
Associated revisions
ref #9204 remove cglib dependency management from cdmlib, only cglib-nodep is used
History
#1 Updated by Andreas Müller 6 months ago
- Related to feature request #9204: Update cdmlib and taxeditor dependencies added
#2 Updated by Andreas Müller 6 months ago
- Description updated (diff)
#3 Updated by Andreas Müller 6 months ago
- Description updated (diff)
#4 Updated by Andreas Müller 6 months ago
- Status changed from New to Resolved
- Assignee changed from Andreas Müller to Andreas Kohlbecker
- % Done changed from 0 to 50
This should be fixed. Andreas K., can you please review and especially check if the removal of hibernate-cglib-repack is ok? I can't see why it should be still needed.
#5 Updated by Andreas Kohlbecker 3 months ago
- Status changed from Resolved to Closed
- % Done changed from 50 to 100
the dependency to hibernate-cglib-repack
was introduced by ben on Wed Feb 25 2009.
I don't see any further need for this, since it only mediates a transient dependency to cglib:cglib-nodep
which is included anyway more explicitely and cdmlib builds and works properly without it.
Therefore I am closing this issue as solved.
#6 Updated by Andreas Kohlbecker 3 months ago
- Assignee changed from Andreas Kohlbecker to Andreas Müller