condenseChanges broken in 3.9

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

condenseChanges broken in 3.9

Gary Chambers
Hi all, managed to fix condenseChanges finally.

See: http://bugs.impara.de/view.php?id=3737

Reply | Threaded
Open this post in threaded view
|

Re: condenseChanges broken in 3.9

stéphane ducasse-2
Hi gary

could you post a CS?

Stef

On 30 mai 06, at 11:23, Gary Chambers wrote:

> Hi all, managed to fix condenseChanges finally.
>
> See: http://bugs.impara.de/view.php?id=3737
>


Reply | Threaded
Open this post in threaded view
|

Re: condenseChanges broken in 3.9

stéphane ducasse-2
In reply to this post by Gary Chambers
I'm going through your discussion and it would be nice if we could  
get one or two tests
at least from the queries you did so that we know that this is really  
fixed.

Stef

On 30 mai 06, at 11:23, Gary Chambers wrote:

> Hi all, managed to fix condenseChanges finally.
>
> See: http://bugs.impara.de/view.php?id=3737
>


Reply | Threaded
Open this post in threaded view
|

RE: condenseChanges broken in 3.9

Gary Chambers
Change set might be awkward since the changes were condensed!

> -----Original Message-----
> From: [hidden email]
> [mailto:[hidden email]]On Behalf Of
> stephane ducasse
> Sent: 30 May 2006 10:45 AM
> To: The general-purpose Squeak developers list
> Subject: Re: condenseChanges broken in 3.9
>
>
> I'm going through your discussion and it would be nice if we could  
> get one or two tests
> at least from the queries you did so that we know that this is really  
> fixed.
>
> Stef
>
> On 30 mai 06, at 11:23, Gary Chambers wrote:
>
> > Hi all, managed to fix condenseChanges finally.
> >
> > See: http://bugs.impara.de/view.php?id=3737
> >
>
>

Reply | Threaded
Open this post in threaded view
|

RE: condenseChanges broken in 3.9

Dan Ingalls
>Change set might be awkward since the changes were condensed!

Well, it sounds that way, but condenseChanges should never affect the most recent version, which is all that is needed for fileOut of the changeSet.  Did you really mean there's a problem?

        - D

> > -----Original Message-----
>> From: [hidden email]
>> [mailto:[hidden email]]On Behalf Of
>> stephane ducasse
>> Sent: 30 May 2006 10:45 AM
>> To: The general-purpose Squeak developers list
>> Subject: Re: condenseChanges broken in 3.9
>>
>>
>> I'm going through your discussion and it would be nice if we could
>> get one or two tests
>> at least from the queries you did so that we know that this is really
>> fixed.
>>
>> Stef
>>
>> On 30 mai 06, at 11:23, Gary Chambers wrote:
>>
>> > Hi all, managed to fix condenseChanges finally.
>> >
>> > See: http://bugs.impara.de/view.php?id=3737
>> >
>>
>>


Reply | Threaded
Open this post in threaded view
|

RE: condenseChanges broken in 3.9

Gary Chambers
The only method change was the shared #moveChangesTo: in
TFileInOutDescription, as described in the report on Mantis. There was a bit
of manual work involved in fixing up the Traits in the image though (just as
part of the investigation into the problem). For instance, some methods
which were not local to particular traits and classes had their own compiled
method instances, even without super sends.

These problems were, as it turns out, probably irrelvant. The main problem
was just the fact that the source for the shared compiled methods was
written to the new changes multiple times while updating the source pointer
in the compiled method.

> -----Original Message-----
> From: [hidden email]
> [mailto:[hidden email]]On Behalf Of Dan
> Ingalls
> Sent: 30 May 2006 10:44 PM
> To: The general-purpose Squeak developers list
> Subject: RE: condenseChanges broken in 3.9
>
>
> >Change set might be awkward since the changes were condensed!
>
> Well, it sounds that way, but condenseChanges should never affect
> the most recent version, which is all that is needed for fileOut
> of the changeSet.  Did you really mean there's a problem?
>
> - D
>
> > > -----Original Message-----
> >> From: [hidden email]
> >> [mailto:[hidden email]]On Behalf Of
> >> stephane ducasse
> >> Sent: 30 May 2006 10:45 AM
> >> To: The general-purpose Squeak developers list
> >> Subject: Re: condenseChanges broken in 3.9
> >>
> >>
> >> I'm going through your discussion and it would be nice if we could
> >> get one or two tests
> >> at least from the queries you did so that we know that this is really
> >> fixed.
> >>
> >> Stef
> >>
> >> On 30 mai 06, at 11:23, Gary Chambers wrote:
> >>
> >> > Hi all, managed to fix condenseChanges finally.
> >> >
> >> > See: http://bugs.impara.de/view.php?id=3737
> >> >
> >>
> >>
>
>