Skip to main content
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [papyrus-rt-dev] Papyrus-RT feature structure
Hi,

As Camille indicates, this new feature structure was introduced to solve bug 494931. Regarding the naming of the new nested feature it was discussed in the related Gerrit change. As can be seen fromt the patch sets this new feature was actually named "base" initially (in patch set 1), but Christian opposed and Camille changed it to "all" (in patch set 2) based on Christian's input. On the other hand, Christian did oppose to the new nested feature itself.

Personally I think it is a little bit too late in the release cycle of 1.0 to start discussing the feature structure. We have had these discussion on and off numerous times on this mailing list, including a discussion if/how we are going to support advanced users to install using the ordinary "Install New Software..." menu in Eclipse which we so far never really have provided a good support for. 

I would also like to understand who are the main stakeholders and what are the driving forces behind the different proposed feature structures? The driving force behind this change was a tool smith that was authoring an Oomph setup file. Which stakeholder in mind do we have for the different proposals? Tool smiths authoring Oomph setup files? Advanced end-users using "Install New Software..." (which we don't really support in a good way anyway)?

If we are considering the latter, we also have the aspect of how we categorize the features (so far we seem to only have focused on the feature structure and how it looks like *after* you actually have made the installation). To my understanding this new "all" feature is not even categorized, and thus does not even show up when using "Install New Software..." when you have the "Group Items by Category" enabled. And we other aspects when it comes to categories, e.g. why do we have a separate "Codegen" category where codegen, run-time and textual are placed? And only one "top level" for category for everything else (expect this new "all" feature then).

So at this stage of the release cycle, I would go for option 5, i.e. status quo. Regarding whether it shall be named "base" or "all" I really don't have any strong opinion. Maybe Christian who objected to the initial name of "base" on the Gerrit change should give his view regarding this.

/Peter Cigéhn

On 17 July 2017 at 09:12, Camille Letavernier <cletavernier@xxxxxxxxxxxxxxxxx> wrote:
Hi,

I think there were a ticket to rename "all" to "base". Basically, the different between "Papyrus-RT" and "Papyrus-RT All" is that one imports some optional Papyrus plug-ins (i.e. the Perspective), whereas the "Base/All" feature only imports the Papyrus-RT plug-ins and lets P2 resolve required dependencies.

So - none of these options actually reach the expected goal :) (But in most cases, it's just a matter of renaming "All" to "Base", keeping the current "Papyrus-RT" as-is, and optionally adding a parent "Papyrus-RT Complete" somewhere)

Camille

On Fri, Jul 14, 2017 at 5:43 PM, Ernesto Posse <eposse@xxxxxxxxxxxxx> wrote:
Hello everyone. While trying the updated version of the Oomph setup in preparation for the 1.0 release I noticed that the structure of features in the "Installation Details" dialog is a bit odd:

Screenshot 2017-07-14 11.21.09.png

First, there is a "Papyrus-RT Feature" which includes only "Papyrus-RT All Feature" (which could be seen as grammatically awkward) and it does *not* include *all* features. Codegen, C++, Textual, Compare and the RSA-RTE importer are all outside.

Is this the way it's supposed to be?

Charles tells me that some of these features, like Codegen, Runtime and C++ are supposed to be optional, and not part of the "core" or "base" (Profile+Core+Common+Tooling). That's fine, but 1) what about "Compare"? Is it a "core" feature? How about the importer? 

Why have two levels of nesting under "Papyrus-RT Feature"? That seems very awkward. I would suggest one of four options:

Option 1) Only one top-level Papyrus-RT Feature called "Papyrus-RT Complete Feature" (or something like that) with two sub-features "Papyrus-RT Base" and "Papyrus-RT Extras":

Complete/

├── Base

│   ├── Common

│   ├── Core

│   ├── Profile

│   └── Tooling

└── Extras

    ├── C++

    ├── Codegen

    ├── Compare

    ├── Importer

    ├── Textual

    └── Runtime


Option 2) Two separate top-level features "Papyrus-RT Base" and "Papyrus-RT Extras":

Base/

├── Common

├── Core

├── Profile

└── Tooling

Extras/

├── C++

├── Codegen

├── Compare

├── Importer

├── Textual

└── Runtime

Option 3) Like option 1, but "extras" are not bundled in one feature:

Complete/

├── Base

│   ├── Common

│   ├── Core

│   ├── Profile

│   └── Tooling

├── C++

├── Codegen

├── Compare

├── Importer

├── Textual

└── Runtime

Option 4) Similar to the status-quo but removing the extra level of nesting:

Base/

├── Common

├── Core

├── Profile

└── Tooling

C++

Codegen

Compare

Importer

Textual

Runtime


Option 5) The status-quo (with a better name for the "All" Feature):

Papyrus-RT/

└── Base

    ├── Common

    ├── Core

    ├── Profile

    └── Tooling

C++

Codegen

Compare

Importer

Textual

Runtime



Any thoughts or opinions?





--
Ernesto Posse
Zeligsoft

--
Ernesto Posse
Zeligsoft

_______________________________________________
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




--
Camille Letavernier

Senior Software Engineer
EclipseSource France


EclipseSource France SAS
Palaiseau-Entreprises
7 rue de la Croix Martre
91120 Palaiseau

General Manager: Remi Schnekenburger
Registered Office: 7 rue de la Croix Martre, 91120 Palaiseau, France
Commercial Register 824 977 516  R.C.S. EVRY

_______________________________________________
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