[squeak-dev] Fwd: The Inbox: XML-Parser-cbc.41.mcz

David T. Lewis lewis at mail.msen.com
Wed Feb 8 00:52:01 UTC 2017


Monty,

It should be fixed now.

Chris,

Sorry for mixing this up in the first place, but thanks for finding
the problem and patching my mistake.

Dave

On Tue, Feb 07, 2017 at 08:50:45PM +0100, monty wrote:
> yes
> 
> > Sent: Tuesday, February 07, 2017 at 7:28 AM
> > From: "David T. Lewis" <lewis at mail.msen.com>
> > To: "The general-purpose Squeak developers list" <squeak-dev at lists.squeakfoundation.org>
> > Subject: Re: [squeak-dev] Fwd:  The Inbox: XML-Parser-cbc.41.mcz
> >
> > Hi Monty,
> > 
> > Thanks for straightening this out. I will fix it before tomorrow if no one
> > else gets it first. Just to confirm that I have it right, we should:
> > 
> > - Merge the missed changes from Collections-monty.714 into Collections.
> > - Resave XML-Parser back to your XML-Parser-monty.40 version.
> > 
> > Dave
> > 
> > On Tue, Feb 07, 2017 at 11:04:06AM +0100, monty wrote:
> > > Thanks David (and Chris), but you missed Collections-monty.714.mcz when copying the other two to the trunk, which is causing the problem. It's a 3 part fix:
> > > 
> > > 1) define correctly spelled versions of the extensions directly on String and ByteString (Collections-monty.714.mcz)
> > > 
> > > 2) change all senders in MOFile to the correctly spelled versions (System-monty.908.mcz)
> > > 
> > > 3) change all senders in XML-Parser to the correctly spelled versions, and for backwards compatibility, leave the wrongly spelled extensions but have them just forward to the new ones (XML-Paresr-monty.40.mcz)
> > > 
> > > If MOFile is going to keep using these messages, besides correct spelling they should be defined directly on String and ByteString and not be XML-Parser extensions, otherwise XML-Parser can't be removed or replaced without breaking MOFile and preventing the installation of some packages.
> > > 
> > > > Sent: Monday, February 06, 2017 at 8:03 PM
> > > > From: "David T. Lewis" <lewis at mail.msen.com>
> > > > To: "The general-purpose Squeak developers list" <squeak-dev at lists.squeakfoundation.org>
> > > > Subject: Re: [squeak-dev] Fwd:  The Inbox: XML-Parser-cbc.41.mcz
> > > >
> > > > On Mon, Feb 06, 2017 at 09:57:19AM -0800, Chris Cunningham wrote:
> > > > > Hi.  This needs to be put in place of  XML-Parser-monty.40.mcz.  That
> > > > > version is causing the trunk to not load (missing a few methods that should
> > > > > be there - this version adds them in, as described).
> > > > > 
> > > > > Could someone with the right authority fix this, please?
> > > > > 
> > > > > -cbc
> > > > 
> > > > Thanks Chris, I moved it to trunk. It looks like the misspelled versions of
> > > > the methods are no longer referenced in the image, but maybe external packages
> > > > still need them?
> > > > 
> > > > Dave
> > > > 
> > > > 
> > > > 
> > > 
> > 
> > 
> 


More information about the Squeak-dev mailing list