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 Chris

I see you're making progress.

Why not add more visibility:=reexport in manifests ? It's an easy way to solve a lot of problems, and quite logical in a lot of case, for instance you can't expect people using aspects without using EMF.

Your opinion ?

Regards,
Marc

Christian Saad a écrit :
Hi Marc,

thanks for fixing this so quickly, alas, Hudson is down again atm....

I've checked out your changes and also for me they don't give any errors. I've refactored the helpers plugins but haven't uploaded them yet as I'd like to do some local testing first.

Regards,
Chris

PS: jdom is used by the WE (to parse view files)

Am 06.05.2010 17:29, schrieb Marc Dutoo:
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
_______________________________________________
jwt-dev mailing list
jwt-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/jwt-dev



Back to the top