Support pluggable classloaders for AnnotationFrameFactory

Question

Support pluggable classloaders for AnnotationFrameFactory

jsight opened this issue 7 years ago · comments

Right now, this relies on the context classloader, but I would like to provide my own implementation for use by ByteBuddy.

Jeffrey Phillips Freeman · Answer 1 · Wed Jan 03 2018 08:07:12 GMT+0800 (China Standard Time)

Sure. Feel free to submit a pull request with your ideas and I'll review it.

…

On Jan 2, 2018 6:25 PM, "Jesse Sightler" ***@***.***> wrote: Right now, this relies on the context classloader, but I would like to provide my own implementation for use by ByteBuddy. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#39>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AC5JAo9yiDBXzI-pagPjoIWxAMq2C1C_ks5tGrrLgaJpZM4RRMfu> .

Deleted user · Answer 2 · Thu Jan 04 2018 13:47:29 GMT+0800 (China Standard Time)

If I can help you I'll do it just let me know ✋ @jsight

Jesse Sightler · Answer 3 · Fri Jan 05 2018 04:52:47 GMT+0800 (China Standard Time)

I can try to put together a PR with a few things like this that I have run into. I am currently working on migrating https://github.com/windup/windup to Ferma from Tinkerpop 2 and a forked version of frames.

Jeffrey Phillips Freeman · Answer 4 · Fri Jan 05 2018 05:06:27 GMT+0800 (China Standard Time)

@jsight Any reason your using a forked version of frames rather than Ferma? I tried to make Ferma very close to Frames for compatibility reasons.

I can obviously help with the PR if anyone needs it. If no one gets to it ill try to add it myself when I have time. Very much appreciate all the help.

Jesse Sightler · Answer 5 · Wed Jan 10 2018 02:02:03 GMT+0800 (China Standard Time)

#42

We used a forked version of Frames because we had started with Frames before Ferma existed. We also use a lot of really esoteric functionality for various historical reasons.