Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [eclipse-pmc] PMC approval needed for patch https://git.eclipse.org/r/#/c/50300/ ?

One more note before you merge:
a) make sure EMF-Databinding still compiles
b) make sure EMF-Databinding (who IIRC is compiled against an older API
   - Helios) still works without getting recompiled

If you break EMF-Databinding you break the complete e4 tooling and most
likely a lot more.

Tom

On 20.03.18 09:18, Tom Schindl wrote:
> On 19.03.18 22:35, Lars Vogel wrote:
>> +1 from my side (as I suggest to Conrad to ask the PMC for
>> permission). I know Conrad and he has a very deep understanding of DB
>> and I trust his judgment.
>>
>> Also, the API relevant change is minimal with only two (small) classes.
> 
> Really? I count a whole lot of API changes because generics have been
> added. When you generify an API you have ONLY one chance to get it
> right, so this needs to be reviewed with special care.
> 
> Having said that, I have no time to review this stuff.
> 
> Tom
> _______________________________________________
> eclipse-pmc mailing list
> eclipse-pmc@xxxxxxxxxxx
> To change your delivery options, retrieve your password, or unsubscribe from this list, visit
> https://dev.eclipse.org/mailman/listinfo/eclipse-pmc
> 


Back to the top