Hi Jonah

You haven't explained why Modisco or Parsley were disabled or why there was no discussion prior to such a widespread unnecessary disabling.

Yes the disabling has eventually been reverted, but unfortunately there was an 8 hour delay between the Gerrit that reverted and the actual merge. IMHO until the merge occurs, there was no reversion.

Regards

Ed Willink


On 08/01/2022 15:33, Jonah Graham wrote:
Hi Eike,

Actually CDO does depend on Mylyn, and therefore (until 2021-12) on CVS:

EMF CDO depends on EMF Compare's contribution:
org.eclipse.emf.cdo.compare -> org.eclipse.emf.compare

EMF Compare depends on EGit's contribution:
org.eclipse.emf.compare.egit ->  org.eclipse.jgit

EGit depends on Mylyn's contribution:
org.eclipse.mylyn.github.feature.feature.group -> org.eclipse.mylyn_feature.feature.group (Note that despite the name org.eclipse.mylyn.github.feature.feature.group  is contributed by EGit to simrel)

The difference is the granularity of the dependencies. For SimRel dependencies are normally handled at the contribution level. At the feature/bundle level there may indeed be no dependencies from CDO -> Mylyn/CVS, but with many contributions being interdependent on each other, and Mylyn historically cutting across so many projects, something that disabled Mylyn causes many things to be easily disabled.

Of course this problem is hindered by Mylyn being under-resourced at the moment.

I hope that helps,
Jonah


~~~
Jonah Graham
Kichwa Coders
www.kichwacoders.com <http://www.kichwacoders.com>


On Sat, 8 Jan 2022 at 10:01, Eike Stepper <step...@esc-net.de> wrote:

    Thanks for your explanation. But, for the record, CDO does not
    dpened, neither directly nor transitively on anything from Mylyn
    or from CVS.

    Cheers
    /Eike

    ----
    http://www.esc-net.de
    http://thegordian.blogspot.com
    http://twitter.com/eikestepper


    Am 08.01.2022 um 15:35 schrieb Kit Lo:

    Eike / Ed W. / Lorenzo / Team,

    Let me clarify. While contributing Eclipse Project M1 to SimRel
    2022-03 M1, SimRel Aggregator Validation found many dependency
    issues "directly" or "indirectly" related to the removal of CVS
    in Eclipse Project. Mylyn was the first offender. After disabling
    Mylyn, all the other downstream projects also needed to be
    disabled because they were "directly" or "indirectly" affected by
    the removal of CVS and/or Mylyn.

    Thanks to Jonah's analysis, most/all of the downstream projects
    were affected by Mylyn. Once Mylyn fixed their CVS dependency,
    Jonah was able to enable most/all the other downstream projects.

    Sorry for the inconvenience and the alarm caused by the change!

    Regards,
    Kit Lo
    Eclipse Babel Project Lead
    IBM Eclipse SDK (IES) Technical Lead and Release Manager

    Inactive hide details for "Lorenzo Bettini" ---01/08/2022
    05:56:28 AM---Same for EMF Parsley: how did you find the CVS
    dependen"Lorenzo Bettini" ---01/08/2022 05:56:28 AM---Same for
    EMF Parsley: how did you find the CVS dependency? Il Sab 8 Gen
    2022, 07:16 Ed Willink <ed.w

    From: "Lorenzo Bettini" <lorenzo.bett...@gmail.com>
    <mailto:lorenzo.bett...@gmail.com>
    To: "Cross project issues" <cross-project-issues-dev@eclipse.org>
    <mailto:cross-project-issues-dev@eclipse.org>
    Date: 01/08/2022 05:56 AM
    Subject: Re: [cross-project-issues-dev] CVS support plugin
    discontinuation - SimRel build changes needed
    Sent by: "cross-project-issues-dev"
    <cross-project-issues-dev-boun...@eclipse.org>
    <mailto:cross-project-issues-dev-boun...@eclipse.org>

    ------------------------------------------------------------------------



    Same for EMF Parsley: how did you find the CVS dependency?

    Il Sab 8 Gen 2022, 07:16 Ed Willink <_ed.willink@gmail.com_
    <mailto:ed.will...@gmail.com>> ha scritto:

        Hi

        Ditto for Modisco. My OOMPH generated Modisco installation is
        totally free of even transitive dependencies on CVS so there
        must be a fault in your dependency analysis.
        Surely such a dramatic disabling should have been
        investigated on the Gerrit / a prvate branch and then
        discussed? Why was it merged to master? Why has it not yet
        been reverted?

         Regards

            Ed Willink

        On 08/01/2022 05:25, Eike Stepper wrote:

            @Jonah Thanks for re-enabling CDO!

            @Kit How did you establish that CDO depends (possibly
            transitively) on CVS? The bundle manifests of CDO do not
            contain anything to that extent, and in my target
            platform for CDO there are no CVS bundles.

            Cheers
            /Eike

            ----_
            __http://www.esc-net.de_ <http://www.esc-net.de>_
            __http://thegordian.blogspot.com_
            <http://thegordian.blogspot.com>_
            __http://twitter.com/eikestepper_
            <http://twitter.com/eikestepper>


            Am 07.01.2022 um 23:21 schrieb Jonah Graham:
                Hi folks,

                I have reenabled all the projects as the failures
                were due to transitive dependencies on Mylyn. For
                Mylyn I have disabled the CVS related/dependent
                features only. AFAICT (validation passes, waiting for
                full build) this should be the full solution so none
                of the other projects need to do anything this time.

                See
                
_https://git.eclipse.org/r/c/simrel/org.eclipse.simrel.build/+/189399_
                
<https://git.eclipse.org/r/c/simrel/org.eclipse.simrel.build/+/189399>

                Jonah
                ~~~
                Jonah Graham
                Kichwa Coders_
                __www.kichwacoders.com_ <http://www.kichwacoders.com>


                On Fri, 7 Jan 2022 at 17:11, Jonah Graham
                <_jonah@kichwacoders.com_
                <mailto:jo...@kichwacoders.com>> wrote:
                    Thanks Kit!

                    It looks like we have quite a few dependencies to
                    resolve here :-) CDT doesn't directly depend on
                    CVS, perhaps there is a transient dependency that
                    does though.

                    Does anyone know which of these directly depend
                    on CVS? I am trying to figure it out now and hope
                    to make a new contribution soon if possible with
                    a smaller set of disabled projects by removing
                    individual features.

                    PS Thank you for making such a large change in M1
                    - it gives all the downstream consumers more time
                    to actually understand the implication of the
                    earlier announced changes.

                    Jonah

                    ~~~
                    Jonah Graham
                    Kichwa Coders_
                    __www.kichwacoders.com_ <http://www.kichwacoders.com>


                    On Fri, 7 Jan 2022 at 16:54, Kit Lo
                    <_ki...@us.ibm.com_ <mailto:ki...@us.ibm.com>>
                    wrote:
                        As announced in
                        
_https://www.eclipse.org/lists/cross-project-issues-dev/msg18643.html_
                        
<https://www.eclipse.org/lists/cross-project-issues-dev/msg18643.html> Eclipse
                        Platform project has stopped building CVS.
                        While contributing to SimRel 2022-03 M1 we
                        found some projects are dependent on CVS
                        causing SimRel build to fail. To make SimRel
                        work we ended up disabling multiple projects.
                        Please re-enable them once the support for
                        CVS has been disabled. Here is the list of
                        projects that were disabled:

                        - cdt
                        - dltk
                        - egit
                        - embedcdt
                        - emf-cdo
                        - emf-compare
                        - emf-parsley
                        - linuxtools
                        - m2m-atl
                        - modisco
                        - mylyn
                        - oomph
                        - pdt
                        - ptp
                        - tcf
                        - tracecompass

                        Thank you!

                        Regards,
                        Kit Lo
                        Eclipse Babel Project Lead
                        IBM Eclipse SDK (IES) Technical Lead and
                        Release Manager

                        _______________________________________________
                        cross-project-issues-dev mailing list_
                        __cross-project-issues-dev@eclipse.org_
                        <mailto:cross-project-issues-dev@eclipse.org>
                        To unsubscribe from this list, visit
                        
_https://www.eclipse.org/mailman/listinfo/cross-project-issues-dev_
                        
<https://www.eclipse.org/mailman/listinfo/cross-project-issues-dev>

                _______________________________________________
                cross-project-issues-dev mailing list
                _cross-project-issues-dev@eclipse.org_
                <mailto:cross-project-issues-dev@eclipse.org>
                To unsubscribe from this list, visit
                
_https://www.eclipse.org/mailman/listinfo/cross-project-issues-dev_
                
<https://www.eclipse.org/mailman/listinfo/cross-project-issues-dev>


            _______________________________________________
            cross-project-issues-dev mailing list
            _cross-project-issues-dev@eclipse.org_
            <mailto:cross-project-issues-dev@eclipse.org>
            To unsubscribe from this list, visit
            _https://www.eclipse.org/mailman/listinfo/cross-project-issues-dev_
            <https://www.eclipse.org/mailman/listinfo/cross-project-issues-dev>

                
            Virus-free. _www.avast.com_
            
<https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=emailclient>


        _______________________________________________
        cross-project-issues-dev mailing list_
        __cross-project-issues-dev@eclipse.org_
        <mailto:cross-project-issues-dev@eclipse.org>
        To unsubscribe from this list, visit
        _https://www.eclipse.org/mailman/listinfo/cross-project-issues-dev_
        
<https://www.eclipse.org/mailman/listinfo/cross-project-issues-dev>_______________________________________________
        cross-project-issues-dev mailing list
        cross-project-issues-dev@eclipse.org
        To unsubscribe from this list, visit
        https://www.eclipse.org/mailman/listinfo/cross-project-issues-dev





    _______________________________________________
    cross-project-issues-dev mailing list
    cross-project-issues-dev@eclipse.org
    To unsubscribe from this list, 
visithttps://www.eclipse.org/mailman/listinfo/cross-project-issues-dev

    _______________________________________________
    cross-project-issues-dev mailing list
    cross-project-issues-dev@eclipse.org
    To unsubscribe from this list, visit
    https://www.eclipse.org/mailman/listinfo/cross-project-issues-dev


_______________________________________________
cross-project-issues-dev mailing list
cross-project-issues-dev@eclipse.org
To unsubscribe from this list, 
visithttps://www.eclipse.org/mailman/listinfo/cross-project-issues-dev

--
This email has been checked for viruses by Avast antivirus software.
https://www.avast.com/antivirus
_______________________________________________
cross-project-issues-dev mailing list
cross-project-issues-dev@eclipse.org
To unsubscribe from this list, visit 
https://www.eclipse.org/mailman/listinfo/cross-project-issues-dev

Reply via email to