Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [papyrus-rt-dev] Update the Model Import tool for Neon

Hi,

 

I’ve pushed a new patch set to include the RSA-RTE importer in a separate feature. However, this raises some additional questions:

 

-          Currently, we have 3 builds and 3 features:

o   Profile

o   Core

o   Tooling

 

It seems that the RSA Importer makes sense in the Tooling build, but not in the tooling feature. Thus, the new patch set adds a new feature (RSA Importer) included in the Tooling build. Should we configure a separate build for this?

 

-          I’ve added the RSA Importer feature to the top-level Papyrus-RT Feature. Does that make sense as well? If the end-user doesn’t want to install the RSA Importer, then he needs to manually check Profile + Core + Tooling, without the RSA Importer. But the top-level feature is supposed to contain everything else, so... not sure what’s best

 

Regards,
Camille

 

De : papyrus-rt-dev-bounces@xxxxxxxxxxx [mailto:papyrus-rt-dev-bounces@xxxxxxxxxxx] De la part de LETAVERNIER Camille
Envoy
é : lundi 11 avril 2016 09:54
À : papyrus-rt developer discussions <papyrus-rt-dev@xxxxxxxxxxx>
Objet : [PROVENANCE INTERNET] Re: [papyrus-rt-dev] Update the Model Import tool for Neon

 

Hi Peter,

 

Ø  maybe we should put this into its own feature, instead of including it directly in the tooling feature

 

I have no opinion on this, so, whatever the Papyrus-RT team prefers :) I’ll add a separate feature

 

Ø  When the RSARTE specific import extension is installed, do you also have to install the RSA import feature from Papyrus? Or will the RSA import functionality included automatically?

 

The standard RSA Import from Papyrus is automatically installed when installing the Papyrus-RT Import (provided that the Papyrus Extra update site is available, so this depends on your deployment). The extension adds no extra UI, it works exactly in the same way (Except that it adds support for Papyrus-RT rather than discarding the profile and stereotypes). Additional UI might be required if e.g. the RSA-RTE Import needs specific import options. But so far, this didn’t seem to be a use case

 

Regards,
Camille

 

De : papyrus-rt-dev-bounces@xxxxxxxxxxx [mailto:papyrus-rt-dev-bounces@xxxxxxxxxxx] De la part de Peter Cigéhn
Envoyé : lundi 11 avril 2016 09:27
À : papyrus-rt developer discussions <papyrus-rt-dev@xxxxxxxxxxx>
Objet : Re: [papyrus-rt-dev] Update the Model Import tool for Neon

 

Hi Camille,

 

As I commented on the Gerrit change, maybe we should put this into its own feature, instead of including it directly in the tooling feature? Since the import tool is a rather "one-shot"-kind of functionality, I guess it would be good if it could be installed optionally for those that don't want/need it. The new import feature could still be included in the top level Papyrus-RT feature in the same way as profle, core and tooling is already.

 

For my understanding also: When the RSARTE specific import extension is installed, do you also have to install the RSA import feature from Papyrus? Or will the RSA import functionality included automatically?

 

In practice, how will it work from an end-user perspective? Will it be the different sets of import options in the import wizard, or do the RSARTE specific import just extends the existing RSA import function, in practice causing it to to be able to also import RSARTE models?

 

/Peter Cigéhn

 

On 7 April 2016 at 13:06, LETAVERNIER Camille <Camille.LETAVERNIER@xxxxxx> wrote:

Hi all,

 

A few months ago, the Model Import tool in Papyrus (Neon) has been refactored to support extensions. The UML-RT part has been removed from the Papyrus plug-in, and contributed to Papyrus-RT. The patch was waiting for the migration of Papyrus-RT to Neon, which is now complete (At least for the tooling part, which is affected by this patch)

 

I’ve rebased the patch to the latest master, so it’s now ready to be merged. Just a few remarks:

 

-          The patch contributes a new Tooling plug-in and the associated Test plug-in, in version 0.8.0. Papyrus-RT still seems to be in 0.7.2

o   That’s not an issue, but maybe we want to remain consistent

-          The patch only adds new plug-ins, and doesn’t change any existing file, except:

o   Tooling pom.xml (Addition of a new module)

o   Tooling Tests pom.xml (Addition of a new module)

o   Tooling Feature (Addition of a new plug-in)

 

So I don’t expect any (major) conflict with other contributions

 

483828: [Model Import] Extend the Papyrus Migration Tool to support UML-RT models

https://bugs.eclipse.org/bugs/show_bug.cgi?id=483828

 

62188: Bug 483828: [Model Import] Extend the Papyrus Migration Tool to support UML-RT models [Id2d75c74]

https://git.eclipse.org/r/#/c/62188/

 

Camille


_______________________________________________
papyrus-rt-dev mailing list
papyrus-rt-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/papyrus-rt-dev

 


Back to the top