Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [Fwd: Re: [jwt-dev] refactoring]

Hi

I've downloaded the Eclipse 3.5 Helios r2 updates, checked out org.apache.xalan from Orbit. Now, no more build errors, and I've successfully tested my committed refactorings.

I've also update the http://wiki.eclipse.org/JWT_CVS pge describing how to set up a development environment :
  * added dependencies : ATL plugin, org.apache.xalan from Orbit
  * TODO is jdom still used, where ?

Regards,
Marc

Marc Dutoo a écrit :
Hi

Opened a bug about it : https://bugs.eclipse.org/bugs/show_bug.cgi?id=311878

My analysis of the package conflicts :

* jwt-we-conf-model.edit/org.eclipse.jwt.we.conf.aspects.event.provider : already in jwt-we-conf-model, moved to .provider, no impact because was obsolete and not exported DONE * jwt-we-conf-property-model/org.eclipse.jwt.we.conf.aspects.factory : already in jwt-we-conf-model, moved to .property, 1 impact in jwt-we-conf-property-model.edit (as well as in manifest and plugin.xml) DONE * jwt-we-sample-staticaspect/org.jwt.sample.logging : just been removed, in duplicate with those in jwt-we-sample-logging DONE * jwt-we-helpers-application/org.eclipse.jwt.we : already in jwt-we STILL TODO



Still TODO : separate jwt-we-helpers-application/org.eclipse.jwt.we, test build and app


I'm sorry, but right now I've got a lot of build errors in my workspace (ex in org.eclipse.jwt.converter.Converter : can't see import org.eclipse.emf.common.util.URI ), possibly because I'm not using Eclipse 3.5 r2 (so no latest atl plugins etc.).

Though I've already committed the items marked DONE above, since they have almost no impact, dear Chris, could you please test my changes for me, in addition to renaming jwt-we-helpers-application/org.eclipse.jwt.we ?

(obviously, I'll try having a workspace in order at some point -_-)

Regards,
Marc

Marc Dutoo a écrit :
Hi Chris, Mickael

Mickael, thanks for your expert inputs.

Since it is a bad practice, I guess I should do the refactoring sooner rather than later, though I'd prefer have a bigger error margin. I'll try having it done and tested before 16:00. I'll keep you in touch.

Regards,
Marc

Christian Saad a écrit :
Hi Mickael,

great to hear from you :)

Thanks for the confirmation! After removing the reexports, I had one good run (more or less, if it gets through then it currently fails at the tests) and now again having problems (see below), e.g. with the org.eclipse.jwt.we.conf.edit/src/org/eclipse/jwt/we/conf/aspects/event package which is definitely shared between two plugins (and also some others).

Marc, would it be possible to resolve this more or less easily? (I would prefer putting the files in different packages rather than trying to get it to work the way it currently is because I've no idea which part of the build causes this problem and if it responds to manifest declarations or not)

Regards,
Chris


@dot:
[mkdir] Created dir: /opt/users/hudsonbuild/.hudson/jobs/cbi-soa-jwt-integration/workspace/build/I201005060556/eclipse/plugins/org.eclipse.jwt.we.conf.edit/@dot [javac] Compiling 24 source files to /opt/users/hudsonbuild/.hudson/jobs/cbi-soa-jwt-integration/workspace/build/I201005060556/eclipse/plugins/org.eclipse.jwt.we.conf.edit/@dot
    [javac] ----------
[javac] 1. ERROR in /opt/users/hudsonbuild/.hudson/jobs/cbi-soa-jwt-integration/workspace/build/I201005060556/eclipse/plugins/org.eclipse.jwt.we.conf.edit/src/org/eclipse/jwt/we/conf/aspects/event/AspectEventManagerNotifyChangedListener.java (at line 35) [javac] private final AspectEventManager aspectEventAdapter = new AspectEventManager();
    [javac]                   ^^^^^^^^^^^^^^^^^^
    [javac] AspectEventManager cannot be resolved to a type
    [javac] ----------
[javac] 2. ERROR in /opt/users/hudsonbuild/.hudson/jobs/cbi-soa-jwt-integration/workspace/build/I201005060556/eclipse/plugins/org.eclipse.jwt.we.conf.edit/src/org/eclipse/jwt/we/conf/aspects/event/AspectEventManagerNotifyChangedListener.java (at line 35) [javac] private final AspectEventManager aspectEventAdapter = new AspectEventManager();
    [javac]      ^^^^^^^^^^^^^^^^^^
    [javac] AspectEventManager cannot be resolved to a type
    [javac] ----------
[javac] 3. ERROR in /opt/users/hudsonbuild/.hudson/jobs/cbi-soa-jwt-integration/workspace/build/I201005060556/eclipse/plugins/org.eclipse.jwt.we.conf.edit/src/org/eclipse/jwt/we/conf/aspects/event/AspectEventManagerNotifyChangedListener.java (at line 38)
    [javac]     this.aspectEventAdapter.notifyChanged(notification);
    [javac]     ^^^^^^^^^^^^^^^^^^^^^^^
    [javac] AspectEventManager cannot be resolved to a type
    [javac] ----------
    [javac] ----------
[javac] 4. ERROR in /opt/users/hudsonbuild/.hudson/jobs/cbi-soa-jwt-integration/workspace/build/I201005060556/eclipse/plugins/org.eclipse.jwt.we.conf.edit/src/org/eclipse/jwt/we/conf/aspects/provider/AspectEnrichedItemProviderAdapter.java (at line 34)
    [javac]     import org.eclipse.jwt.we.conf.Aspect;
    [javac]            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[javac] The import org.eclipse.jwt.we.conf.Aspect cannot be resolved
    [javac] ----------
[javac] 5. ERROR in /opt/users/hudsonbuild/.hudson/jobs/cbi-soa-jwt-integration/workspace/build/I201005060556/eclipse/plugins/org.eclipse.jwt.we.conf.edit/src/org/eclipse/jwt/we/conf/aspects/provider/AspectEnrichedItemProviderAdapter.java (at line 35)
    [javac]     import org.eclipse.jwt.we.conf.AspectInstance;
    [javac]            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[javac] The import org.eclipse.jwt.we.conf.AspectInstance cannot be resolved
    [javac] ----------
[javac] 6. ERROR in /opt/users/hudsonbuild/.hudson/jobs/cbi-soa-jwt-integration/workspace/build/I201005060556/eclipse/plugins/org.eclipse.jwt.we.conf.edit/src/org/eclipse/jwt/we/conf/aspects/provider/AspectEnrichedItemProviderAdapter.java (at line 36)
    [javac]     import org.eclipse.jwt.we.conf.ConfPackage;
    [javac]            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[javac] The import org.eclipse.jwt.we.conf.ConfPackage cannot be resolved



Am 06.05.2010 11:31, schrieb Mickael Istria:
  Hi,

You're right, it's weird anyway because it did work before, at least
Mickael built it this way I guess, maybe he's got an idea ?
This is probably because CBI build is p2-based (uses p2.gathering=true
property), and is more rigorous, and far better, about dependency
resolution, whereas the old builder was not p2-based (standard
old-school build + p2 generation on the old-school update-site).

So maybe don't change too much things yet, just enough to now whether
that's the right solution or not...
I think having the same package in several bundles is indeed the cause
of the problem. That's one of OSGi worst practices, so that it can be
the cause of such troubles. Basically for a bundle
org.eclipse.jwt.my_bundle, all classes inside of it must be in a
org.eclipse.jwt.my_bundle or child package. Otherwise, it could be
confusing for developers, install-time resolver (p2) and runtine
resolver (Equinox).

Regards,

--

Mickael Istria
R&D Engineer
BonitaSoft - Open your processes <http://www.bonitasoft.com> email : mickael.istria@xxxxxxxxxxxxxx


This message and any attachment (the "message") is intended solely for
the addressees and is confidential. If you receive this message by
mistake, please delete it and notify the sender immediately. Any use not in accordance with its purpose, any out-spread or disclosure, either as
a whole or partially, is prohibited except with formal approval.
Internet cannot guarantee the integrity of this message, therefore
BonitaSoft will not be liable for the message if modified.



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


Back to the top