Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [equinox-dev] IWeavingServiceFactory.createWeavingService takes a BundleDescription

Hey Tom!

I would love to discuss this with you, but unfortunately I will not be at EclipseCon (neither Andrew). Andy (Clement) will be, so you should be able to chat with him there.

Aside of that I was wondering how you are going to port over the equinox weaving hook framework extension to your new framework implementation (since it is using all the old equinox framework extension hooks at the moment, that I thought wouldn't be there in the new implementation). But I think I should take a look at your code before I waste your time here with theoretical discussions... ;-)

Anyway, if the equinox weaving hook would work for the new implementation, that would be awesome. It is by far not a brilliant piece of code, but it would let us reuse the old caching and aspectj weaving implementation, I guess. That would save us a lot of time.

Will try to take a look as well, but not before next week.

Cheers,
-Martin


On 12.03.13 13:43, Thomas Watson wrote:
I will be at EclipseCon.  If you and/or Martin are there, perhaps we can
get together to discuss the details there as well.

Tom



Inactive hide details for Andrew Eisenberg ---03/11/2013 06:11:01
PM---Thanks for the advice. I have the workspace set up now aAndrew
Eisenberg ---03/11/2013 06:11:01 PM---Thanks for the advice. I have the
workspace set up now and the hook is being activated.  Unfortunate

From: Andrew Eisenberg <andrew@xxxxxxxxxxxx>
To: Thomas Watson/Austin/IBM@IBMUS,
Cc: Equinox development mailing list <equinox-dev@xxxxxxxxxxx>,
mlippert@xxxxxxxxxx
Date: 03/11/2013 06:11 PM
Subject: Re: [equinox-dev] IWeavingServiceFactory.createWeavingService
takes a BundleDescription
Sent by: andrew.eisenberg@xxxxxxxxx

------------------------------------------------------------------------



Thanks for the advice. I have the workspace set up now and the hook is
being activated.  Unfortunately, the weaving is not happening.  I'll
have to explore a bit more and I'll bring Martin Lippert into the
conversation as well (since he's the original author of Equinox
Weaving).  After we learn a bit more, we'll get back to you.

On Fri, Mar 8, 2013 at 1:48 PM, Thomas Watson <_tjwatson@xxxxxx.com_
<mailto:tjwatson@xxxxxxxxxx>> wrote:

    It is a bit more involved to deploy this into your Kepler instance.
      You have to be running a Kepler instance with a workspace that has
    loaded both the org.eclipse.osgi and
    org.eclipse.osgi.compatibility.state projects.  Then you have to
    export the projects using "Export->Plug-in Development->Deployable
    plug-ins and fragments" and for the destination you have to select
    "Install into host. Repository:".  This will deploy the new
    framework to your running installation and you must then restart
    Eclipse.  If all goes well Eclipse will restart successfully on the
    new framework.

    Note that this is not ready for prime-time, so I suggest you do this
    on a Kepler installation that you don't mind blowing up.  I find it
    easier to just run Eclipse on the new framework by launching a
    self-hosted Eclipse instance from your Eclipse workspace while you
    have the org.eclipse.osgi and org.eclipse.osgi.compatibility.state
    projects loaded.

    Tom



    Inactive hide details for Andrew Eisenberg ---03/08/2013 03:34:50
    PM---Hi Tom, Thanks for taking a look at this. AJDT is one oAndrew
    Eisenberg ---03/08/2013 03:34:50 PM---Hi Tom, Thanks for taking a
    look at this.  AJDT is one of the main consumers

    From: Andrew Eisenberg <_andrew@eisenberg.as_
    <mailto:andrew@xxxxxxxxxxxx>>
    To: Equinox development mailing list <_equinox-dev@eclipse.org_
    <mailto:equinox-dev@xxxxxxxxxxx>>,
    Date: 03/08/2013 03:34 PM
    Subject: Re: [equinox-dev]
    IWeavingServiceFactory.createWeavingService takes a BundleDescription
    Sent by: _equinox-dev-bounces@eclipse.org_
    <mailto:equinox-dev-bounces@xxxxxxxxxxx>
    ------------------------------------------------------------------------




    Hi Tom,

    Thanks for taking a look at this.  AJDT is one of the main consumers
    of org.eclipse.equinox.weaving.hook through the
    org.eclipse.equinox.weaving.aspectj bundle.  It looks like you have
    also updated the org.eclipse.equinox.weaving.aspectj bundle.  I will
    get a chance to try this out next week.  Just to be sure, if I check
    out this branch of equinox and drop the osgi bundle into Kepler, will
    things hang together?

    regards,
    Andrew

    On Fri, Mar 8, 2013 at 12:57 PM, Thomas Watson
    <_tjwatson@xxxxxx.com_ <mailto:tjwatson@xxxxxxxxxx>> wrote:
     > Keep in mind this is a discussion for Luna, not Kepler.  I have been
     > spending some time lately around the new framework implementation
    based on
     > the OSGi R5 generic capability/requirement model.
     >
     > I was looking at porting the Equinox weaving hooks
     > (org.eclipse.equinox.weaving.hook) over to a new Equinox framework
     > implementation that is based internally on the generic
     > capability/requirement model of the OSGi R5 specification.  This
    framework
     > implementation no longer is based on the old Equinox resolver API
     > (org.eclipse.osgi.service.resolver).  As such the weaving hook
     > implementation no longer has access to BundleDescription objects
    or a State
     > object at runtime.  Instead it would have access to
     > org.osgi.framework.wiring.BundleRevsion/BundleWiring objects.
     >
     > How disruptive would it be to make a breaking API change to
     > org.eclipse.equinox.service.weaving.IWeavingServiceFactory to take a
     > org.osgi.framework.wiring.BundleRevsion instead of the old
     > org.eclipse.osgi.service.resolver.BundleDescription and no longer
    take a
     > org.eclipse.osgi.service.resolver.State?
     >
     > I noticed the bundle org.eclipse.equinox.weaving.hook exports the
    package
     > org.eclipse.equinox.service.weaving with no version (defaulting
    to 0.0.0).
     > For the next release I suggest we bump this version to 1.0.0 and
    indicate a
     > breaking change for implementors of
     > org.eclipse.equinox.service.weaving.IWeavingServiceFactory.  I'm
    not sure
     > how many others outside of the
    org.eclipse.equinox.weaving.aspectj bundle
     > implement this interface.  But without this type of change I see
    no way to
     > support Equinox Weaving on the new framework.  Beyond this I
    still want
     > someone to investigate moving as much of the Equiniox weaving
    hooks onto the
     > standard OSGi WeavingHook services [1].
     >
     > If you want to see the work I have done so far then load up the
     > twatson/container branches from both the rt.equinox.framework [2] and
     > rt.equinox.bundles [3] git repositories.  You should be able to
    self-host
     > Eclipse if you load up the framework project (org.eclipse.osg)
    and the
     > compatibility state project
    (org.eclipse.osgi.compatibility.state) which
     > provides an implementation of the old equinox PlatformAdmin/State
    mainly to
     > support PDE.  I have not done any testing of the weaving hook
    changes.  I
     > only spent the morning getting rid of compile errors which
    pointed me to
     > this API issue with BundleDescription in the Equinox Weaving API.
     >
     > [1] _https://bugs.eclipse.org/bugs/show_bug.cgi?id=377422_
     > [2]
     >
    _http://git.eclipse.org/c/equinox/rt.equinox.framework.git/log/?h=twatson/container_
     > [3]
     >
    _http://git.eclipse.org/c/equinox/rt.equinox.bundles.git/log/?h=twatson/container_
     >
     >
     > Tom
     >
     >
     > _______________________________________________
     > equinox-dev mailing list
     > _equinox-dev@eclipse.org_ <mailto:equinox-dev@xxxxxxxxxxx>
     > _https://dev.eclipse.org/mailman/listinfo/equinox-dev_
     >
    _______________________________________________
    equinox-dev mailing list_
    __equinox-dev@eclipse.org_ <mailto:equinox-dev@xxxxxxxxxxx>_
    __https://dev.eclipse.org/mailman/listinfo/equinox-dev_




_______________________________________________
equinox-dev mailing list
equinox-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/equinox-dev



Back to the top