Ed,

You should consider communicating with a less demanding tone.

Firstly, it's not Jonah's responsibility to analyze and explain the dependencies between projects, especially not long transitive chains of dependencies.   I've been working on a tool to do such analysis and it just so happens that MoDisco and Parsley depend on CDO and CDO's transitive dependency on Mylyn has already been explained:

Secondly, Jonah did not disable anything.  The platform team did that and needed to do that because otherwise they could not contribute M1.  And while it's unfortunate that it happened, it wasn't unnecessary.

We can and should all be very thankful that Jonah analyzed the problem and re-enabled our contributions on our behalf with no effort on our part (and without prior long discussions that would have taken more than 8 hours).

Thanks Jonah from all of us!

Regards,
Ed



On 08.01.2022 19:30, Ed Willink wrote:

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

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

<#DAB4FAD8-2DD7-40BB-A1B8-4E2AA1F9FDF2>

_______________________________________________
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

Reply via email to