Integration deadlock?

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
5 messages Options
Reply | Threaded
Open this post in threaded view
|

Integration deadlock?

Stephane Ducasse-3
I have a change that crosscuts calypso and iceberg.
I will produce a PR and each project should check because else I will get stuck
waiting in an integration deadlock.

https://pharo.fogbugz.com/f/cases/21459/Fixing-last-users-of-theNonMetaclass

Reply | Threaded
Open this post in threaded view
|

Re: Integration deadlock?

Denis Kudriashov
Did you really saw that Calypso was affected?
I do not know how to review this changes now. The pull request is closed. And I doubt that I will be able to merge changes back (I do not know how to do it).

2018-03-01 16:39 GMT+01:00 Stephane Ducasse <[hidden email]>:
I have a change that crosscuts calypso and iceberg.
I will produce a PR and each project should check because else I will get stuck
waiting in an integration deadlock.

https://pharo.fogbugz.com/f/cases/21459/Fixing-last-users-of-theNonMetaclass


Reply | Threaded
Open this post in threaded view
|

Re: Integration deadlock?

Stephane Ducasse-3
I did a file out because the PR was totally literally broken.
I change theMetaClass and theNonMetaClass to classSide instanceSide

On Fri, Mar 2, 2018 at 4:52 PM, Denis Kudriashov <[hidden email]> wrote:

> Did you really saw that Calypso was affected?
> I do not know how to review this changes now. The pull request is closed.
> And I doubt that I will be able to merge changes back (I do not know how to
> do it).
>
> 2018-03-01 16:39 GMT+01:00 Stephane Ducasse <[hidden email]>:
>>
>> I have a change that crosscuts calypso and iceberg.
>> I will produce a PR and each project should check because else I will get
>> stuck
>> waiting in an integration deadlock.
>>
>>
>> https://pharo.fogbugz.com/f/cases/21459/Fixing-last-users-of-theNonMetaclass
>>
>

Reply | Threaded
Open this post in threaded view
|

Re: Integration deadlock?

Denis Kudriashov
Ok. I always use instanceSide and classSide.
So no problem in Calypso

2018-03-02 19:18 GMT+01:00 Stephane Ducasse <[hidden email]>:
I did a file out because the PR was totally literally broken.
I change theMetaClass and theNonMetaClass to classSide instanceSide

On Fri, Mar 2, 2018 at 4:52 PM, Denis Kudriashov <[hidden email]> wrote:
> Did you really saw that Calypso was affected?
> I do not know how to review this changes now. The pull request is closed.
> And I doubt that I will be able to merge changes back (I do not know how to
> do it).
>
> 2018-03-01 16:39 GMT+01:00 Stephane Ducasse <[hidden email]>:
>>
>> I have a change that crosscuts calypso and iceberg.
>> I will produce a PR and each project should check because else I will get
>> stuck
>> waiting in an integration deadlock.
>>
>>
>> https://pharo.fogbugz.com/f/cases/21459/Fixing-last-users-of-theNonMetaclass
>>
>


Reply | Threaded
Open this post in threaded view
|

Re: Integration deadlock?

Stephane Ducasse-3
Can you check because I fixed some.

On Sat, Mar 3, 2018 at 10:23 AM, Denis Kudriashov <[hidden email]> wrote:

> Ok. I always use instanceSide and classSide.
> So no problem in Calypso
>
> 2018-03-02 19:18 GMT+01:00 Stephane Ducasse <[hidden email]>:
>>
>> I did a file out because the PR was totally literally broken.
>> I change theMetaClass and theNonMetaClass to classSide instanceSide
>>
>> On Fri, Mar 2, 2018 at 4:52 PM, Denis Kudriashov <[hidden email]>
>> wrote:
>> > Did you really saw that Calypso was affected?
>> > I do not know how to review this changes now. The pull request is
>> > closed.
>> > And I doubt that I will be able to merge changes back (I do not know how
>> > to
>> > do it).
>> >
>> > 2018-03-01 16:39 GMT+01:00 Stephane Ducasse <[hidden email]>:
>> >>
>> >> I have a change that crosscuts calypso and iceberg.
>> >> I will produce a PR and each project should check because else I will
>> >> get
>> >> stuck
>> >> waiting in an integration deadlock.
>> >>
>> >>
>> >>
>> >> https://pharo.fogbugz.com/f/cases/21459/Fixing-last-users-of-theNonMetaclass
>> >>
>> >
>>
>