Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 288504 - New MoDisco common features
Summary: New MoDisco common features
Status: CLOSED FIXED
Alias: None
Product: MoDisco
Classification: Modeling
Component: Contribution (show other bugs)
Version: 0.8.0   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 0.8.0 M4   Edit
Assignee: Hugo Bruneliere CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-09-03 12:16 EDT by Gregoire Dupe CLA
Modified: 2013-04-29 07:55 EDT (History)
5 users (show)

See Also:
gdupe: iplog+


Attachments
New MoDisco common features (48.43 KB, text/plain)
2009-09-03 12:16 EDT, Gregoire Dupe CLA
fabien.giquel: iplog+
Details
org.eclipse.gmt.modisco.infra.common.ui plug-in project archive (12.01 KB, application/x-zip-compressed)
2009-09-03 12:17 EDT, Gregoire Dupe CLA
no flags Details
New MoDisco common ui features (12.30 KB, application/x-zip-compressed)
2009-09-09 05:45 EDT, Gregoire Dupe CLA
fabien.giquel: iplog+
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Gregoire Dupe CLA 2009-09-03 12:16:05 EDT
Created attachment 146419 [details]
New MoDisco common features

You can find in the attached patch and in the attached archive a set of common
features for MoDisco. These features are necessary to develop the next
features.

The provided features are: 
 - a MoDisco project nature, that will be used to store all the MoDisco
specific files into a dedicated type of project;
 - a wizard to create new MoDisco projects ;
 - a MoDisco project builder, that will be used to notify the file events
happening into the MoDisco project to the different MoDisco components ;
 - a MoDisco EMF protocol, that will be used to allow MoDisco EMF models to
easily and robustly refer each others.

The contributors of these features are: Fabien Giquel and Grégoire Dupé

All this contribution is made available under EPL by its different authors
whose Employer Consent Forms should have been received by the Foundation.

Regards,
Grégoire Dupé
Comment 1 Gregoire Dupe CLA 2009-09-03 12:17:31 EDT
Created attachment 146420 [details]
org.eclipse.gmt.modisco.infra.common.ui plug-in project archive
Comment 2 Hugo Bruneliere CLA 2009-09-03 12:37:40 EDT
Hi guys,

Thanks for this important improvement of the MoDisco infrastructure.
I will do my best to check your contribution and integrate it as soon as possible.

Regards,

Hugo
Comment 3 Hugo Bruneliere CLA 2009-09-04 04:34:01 EDT
Hi,

I successfully integrated your patch and I'm now looking at the new plug-in you contributed in this bug.

I don't think the current name of this plug-in is the more relevant one.
Should it rather be "org.eclipse.gmt.modisco.common.ui" as indicated by the name of the packages it contains (and as it only depends from the "org.eclipse.gmt.modisco.common.core" plug-in)? 

Best regards,

Hugo
Comment 4 Gregoire Dupe CLA 2009-09-04 05:24:38 EDT
Hi,

The mistake is in the package names. The right name is: "org.eclipse.gmt.modisco.infra.common.ui". 

The package "org.eclipse.gmt.modisco.common.core" should be a soon as possible renamed as "org.eclipse.gmt.modisco.infra.common.core". The added of the "infra" prefix will help to precise that those components are a part of the MoDisco infrastructure.

Regards,
Grégoire
Comment 5 Hugo Bruneliere CLA 2009-09-04 08:32:11 EDT
I saw your point of view.

But I think this is a decision that will require some internal discussions as there are several plug-ins which are concerned:
  - org.eclipse.gmt.modisco.common.core
  - org.eclipse.gmt.modisco.infra.common.ui (not committed yet) 
  - org.eclipse.gmt.modisco.common.editor
  - org.eclipse.gmt.modisco.common.editor.extension.xxx
  - org.eclipse.gmt.modisco.common.editor.extension.yyy
  - org.eclipse.gmt.modisco.infrastructure.features (the feature gathering the "infrastructure" plug-ins)

Instead of the current state, I would rather see something like this:
  - org.eclipse.gmt.modisco.infrastructure.common.core
  - org.eclipse.gmt.modisco.infrastructure.common.ui
  - org.eclipse.gmt.modisco.infrastructure.common.editor
  - org.eclipse.gmt.modisco.infrastructure.common.editor.extension.xxx
  - org.eclipse.gmt.modisco.infrastructure.common.editor.extension.yyy
  - org.eclipse.gmt.modisco.infrastructure.feature

As a consequence, we will let this bug in an temporary "awaiting decision" state until we internally agree on a solution.

Best regards,

Hugo
Comment 6 Hugo Bruneliere CLA 2009-09-08 11:33:09 EDT
Hi Gregoire,

I have tested the current version of the "common.ui" plug-in. 
When I create a new MoDisco project using the provided project nature and wizard, there is an error within the MANIFEST.MF of this automatically created project: there is a dependency to the "org.eclipse.gmt.modisco.infra.query.manager" plug-in which is not available from the MoDisco SVN.

Please, could you either delete this dependency or contribute this other plug-in as well?

Best regards,

Hugo
Comment 7 Gregoire Dupe CLA 2009-09-09 05:45:56 EDT
Created attachment 146734 [details]
New MoDisco common ui features

Hello,

You can find in the attached archive (org.eclipse.gmt.modisco.infra.common.ui-20090909-1138.zip) a new version of the "org.eclipse.gmt.modisco.infra.common.ui" plug-in. In this new version, I have:
 - suppressed the dependency to "org.eclipse.gmt.modisco.infra.query.manager";
 - added the "build.properites" file generation (to remove warning on the generated project);
 - added the ".qualifier" suffix in the "infra.common.ui" plug-in version.

Regards,
Grégoire
Comment 8 Hugo Bruneliere CLA 2009-09-09 07:34:04 EDT
Hi Gregoire,

I tested your upgraded version and it now works fine.

I will launch the IP process and come back to you if I need more information from your side.

Best regards,

Hugo
Comment 9 Hugo Bruneliere CLA 2009-09-09 10:19:45 EDT
Gregoire and Fabien,

Please, could you both make the required confirmation via this bug (same way as in Bug #283604)?

Thanks in advance,

Best regards,

Hugo
Comment 10 Gregoire Dupe CLA 2009-09-09 11:31:07 EDT
I confirm that

1. I, DUPE Grégoire, and others contributors listed in this bug, authored 100%
of the content
2. I have the rights to donate the content to Eclipse under the EPL
3. I have already provided the filled Employer Consent Forms to the Foundation

Grégoire DUPE.
Comment 11 Fabien Giquel CLA 2009-09-17 05:09:59 EDT
I confirm that

1. I, GIQUEL Fabien, and others contributors listed in this bug, authored 100%
of the content
2. I have the rights to donate the content to Eclipse under the EPL
3. I have already provided the filled Employer Consent Forms to the Foundation

Fabien GIQUEL.
Comment 12 Hugo Bruneliere CLA 2009-10-30 11:04:26 EDT
Hi guys,

Your contribution has been approved and has just been committed onto the MoDisco SVN.

Best regards,

Hugo