search

yong-hu / objenesis

Automatically exported from code.google.com/p/objenesis
0 stars 0 forks source link

Objenesis needs to import sun.reflect package in OSGi Manifest #15

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
The SunReflectionFactorySerializationInstantiator class directly uses 
sun.reflect.ReflectionFactory, which is fine. However the bundle manifest 
headers should explicitly import this package.

It may have to do with how the maven plugin is generating the headers. The 
version used (2.0.0) is quite old now. I will try a newer version and see if it 
generates better headers.

Original issue reported on code.google.com by m...@thebishops.org on 24 Sep 2012 at 9:10

GoogleCodeExporter commented 9 years ago 
I have modified the pom files to both update the maven-bundle-plugin version as 
well as give it the ability to correctly generate the manifest headers.

The patch has a few white space trims, sorry about that. My IDE strips trailing 
whitespace.

Original comment by m...@thebishops.org on 24 Sep 2012 at 9:51

Attachments:

GoogleCodeExporter commented 9 years ago 
I need to fix the patch, it has to optionally import jrockit and newmonics 
classes.

Original comment by m...@thebishops.org on 25 Sep 2012 at 12:15

GoogleCodeExporter commented 9 years ago 
Here is a revised patch that optionally includes jrockit, sun.reflect and 
newmonics packages.

Original comment by m...@thebishops.org on 25 Sep 2012 at 4:14

Attachments:

GoogleCodeExporter commented 9 years ago 
I am running OSGi Framework Felix 4.0.3 (R4.3).

Original comment by m...@thebishops.org on 25 Sep 2012 at 9:38

GoogleCodeExporter commented 9 years ago 
Done. Thanks a lot for the patch

Original comment by henri.tr...@gmail.com on 30 Sep 2012 at 1:23

GoogleCodeExporter commented 9 years ago

Original comment by henri.tr...@gmail.com on 14 Aug 2013 at 11:47

GoogleCodeExporter commented 9 years ago

Original comment by henri.tr...@gmail.com on 14 Aug 2013 at 11:49