Hi, Ernesto,
When the refactoring of the build is completed, we can expect to have *more* of this kind of issue, not less. But that is what (I think) we want: this kind of refactoring is disruptive to the system design, and so this build structure highlights that. It should help us to reduce the incidence of breaking changes by throwing up Gerrit failures where a monolithic build would not, but when we do need to introduce breaking changes (as in this case), there will be hoops to jump through.
On 26 July, 2016 at 10:14:22, Ernesto Posse (eposse@xxxxxxxxxxxxx) wrote:
I think the problem is with the codegen pom. The
codegen pom does not build the profile, but it has some
targetplatform projects as submodules, and I think those include
the following, for the nightly and releases TPs:
location "https://hudson.eclipse.org/papyrus-rt/job/Papyrus-RT-Product/lastSuccessfulBuild/artifact/repository/"
{
org.eclipse.papyrusrt.umlrt.profile.feature.feature.group
lazy
org.eclipse.papyrusrt.umlrt.common.ui.feature.feature.group
lazy
org.eclipse.papyrusrt.umlrt.common.feature.feature.group
lazy
org.eclipse.papyrusrt.umlrt.core.feature.feature.group lazy
}
so I think the build would not work until the updated profile
is available at
So it may be necessary to merge this gerrit overriding the -1. I
don't know if there is an alternative solution. Perhaps once Celine
is done with her reorganization of the build, we won't have this
sort of issue.
The codegen build failed again:
[ERROR]
ERROR: UMLRTStateMachProfileUtil.xtend - /jobs/genie.papyrus-rt/Papyrus-RT-CodegenRTS-BuildFrom-gerrit-Master/workspace/source/plugins/xtumlrt/common/org.eclipse.papyrusrt.xtumlrt.external/src/org/eclipse/papyrusrt/xtumlrt/external/predefined/UMLRTStateMachProfileUtil.xtend
10: org.eclipse.papyrusrt.umlrt.profile.statemachine.UMLRTStateMachines.RTTrigger cannot be resolved to a type.
I just realized one thing, though. That plugin's MANIFEST
has
org.eclipse.papyrusrt.umlrt.profile;bundle-version="0.7.2",
Has the profile version been updated?
I just rebased it. I'll push it to Gerrit and
hopefully it will work.
Hi Ernesto,
Thank you very much. Let me know if you have any question
/feedback
Regards
Rémi
-------------------------------------------------------
Rémi SCHNEKENBURGER
+33 (0)1 69 08 48 48
CEA Saclay Nano-INNOV
Institut CARNOT CEA LIST
![Description : PapyrusLogo_SmallFormat]() www.eclipse.org/papyrus
Hi Remi. I did see that failure and commented
on Gerrit. The codegen build fails because it cannot find the
UML-RT state machines profile. After a closer look, it looks like
the issue is that that one has been renamed
from UMLRealTimeStateMach to UMLRTStateMachines.
I'll get the gerrit and fix the
references.
Hi team,
I pushed a new version of the contribution
that sounds OK for me, but not for some of the gerrit jobs,
especially the codegen one [1]. However, when I look to the
compilation errors, it seems that it only comes from the fact that
the profile plugin used for building the gerrit is not
up-to-date.
So, I have again 2 choices:
-
Ignoring the gerrit from codegen, and pushing the gerrit. Then I
could ensure that the main builds stay green. In this case, and if
I cannot solve the issues fast, I could revert the commit and work
locally again on it.
-
Ask for a local review from Ernesto or any other people involved in
the codegen part to check if everything is fine, and then only
commit.
What do you think? I am reluctant to override
the -1 from a reviewer on gerrit, but I know in this case that we
are in a deadlock :/
Regards,
Rémi
[1] https://hudson.eclipse.org/papyrus-rt/view/Gerrit/job/Papyrus-RT-Gerrit-master/567/
-------------------------------------------------------
Rémi SCHNEKENBURGER
+33 (0)1 69 08 48 48
CEA Saclay
Nano-INNOV
Institut CARNOT CEA LIST
![Description : PapyrusLogo_SmallFormat]() www.eclipse.org/papyrus
_______________________________________________
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
_______________________________________________
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
_______________________________________________
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
|
