[squeak-dev] Bogus Class Category in 3.10.1-7175 - Mantis #7070

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

[squeak-dev] Bogus Class Category in 3.10.1-7175 - Mantis #7070

Ken Causey-3
Note that in the released 3.10.1-7175 you will find a category called
#Morphic-CandidatesForGo.  Do NOT click any of the classes listed in
that category as they are not really there but removed in 3.10-7159.
The result is an annoying, seemingly unstoppable, series of debug
prompts.  I made a mistake when building 3.10.1-7175, you can look at

http://bugs.squeak.org/view.php?id=7070

For more of the gorey details.

My point with this email, other than to apologize, is to ask what I
should do.  First note that this issue is related to my pre-release
cleanup and is not due to any changeset in the update stream.  Take a
3.10-7159 and update it to 3.10.1-7175 and you will not have this
problem.

The first solution might be to simply build a replacement
3.10.1-7175-basic.zip without the problem and replace the copy on the
FTP site.  However I don't think that would be a good idea.  Having
spent a lot of time over the years helping users with issues I know how
confusing it can be to try to replicate an issue and one of the keys is
being able to start with the same image.  Imagine a user has downloaded
my faulty release and is experiencing this problem and is confused.
Imagine also a helpful person who wants to assist them with this
problem.  The helper finds out they are using 3.10.1-7175 and downloads
it from the FTP site.  Unfortunately this helper has not heard of this
issue and tries the newer 3.10.1-7175 image and is unable to replicate
the problem.  Substitute appropriately modified "Who's on First"
variant; hilarity ensues at least if you are not a participant.

So, I started a Known Issues list at

http://ftp.squeak.org/3.10/KNOWNISSUES

that prominently features this issue.  I would like to provide a
workaround of some kind there and I would appreciate any insights into a
code snippet to address this issue in 3.10.1-7175 that I can provide as
a workaround.

Otherwise the only other solution I can think of is to yet again start
on a quick fix release and call it 3.10.2.  Thoughts?

I apologize for this mistake, I can only hope (unrealistically) that was
my only one.  Clearly having someone jump into a release branch as I
have done here, without the proper knowledge, has consequences.

Ken



signature.asc (196 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

[squeak-dev] Re: Bogus Class Category in 3.10.1-7175 - Mantis #7070

Klaus D. Witzel
On Wed, 28 May 2008 17:59:14 +0200, Ken Causey wrote:

> Note that in the released 3.10.1-7175 you will find a category called
> #Morphic-CandidatesForGo.  Do NOT click any of the classes listed in
> that category as they are not really there but removed in 3.10-7159.
> The result is an annoying, seemingly unstoppable, series of debug
> prompts.  I made a mistake when building 3.10.1-7175, you can look at
>
> http://bugs.squeak.org/view.php?id=7070
>
> For more of the gorey details.
>
> My point with this email, other than to apologize, is to ask what I
> should do.  First note that this issue is related to my pre-release
> cleanup and is not due to any changeset in the update stream.  Take a
> 3.10-7159 and update it to 3.10.1-7175 and you will not have this
> problem.
>
> The first solution might be to simply build a replacement
> 3.10.1-7175-basic.zip

I'd say: put a text (big letters from default font) into that .image and  
update 3.10.1-7175-basic.zip; an explanation of what's wrong with the  
.image and what to do from here.

This will not address possible problems of people who have already  
downloaded the current one but, that's easy to find out: they can be asked  
if they see the text.

/Klaus


Reply | Threaded
Open this post in threaded view
|

Re: [squeak-dev] Re: Bogus Class Category in 3.10.1-7175 - Mantis #7070

Ken Causey-3
On Wed, 2008-05-28 at 18:47 +0200, Klaus D. Witzel wrote:
> On Wed, 28 May 2008 17:59:14 +0200, Ken Causey wrote:
...

> >  I made a mistake when building 3.10.1-7175, you can look at
> >
> > http://bugs.squeak.org/view.php?id=7070
> >
> > For more of the gorey details.
> >
> > My point with this email, other than to apologize, is to ask what I
> > should do.  ...
>
> I'd say: put a text (big letters from default font) into that .image and  
> update 3.10.1-7175-basic.zip; an explanation of what's wrong with the  
> .image and what to do from here.
>
> This will not address possible problems of people who have already  
> downloaded the current one but, that's easy to find out: they can be asked  
> if they see the text.
Thanks, that's a good idea and something I will do as soon as I find a
way to fix this in the existing image.

Ken

> /Klaus




signature.asc (196 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

[squeak-dev] Re: Bogus Class Category in 3.10.1-7175 - Mantis #7070

Klaus D. Witzel
On Wed, 28 May 2008 19:02:01 +0200, Ken Causey wrote:

> On Wed, 2008-05-28 at 18:47 +0200, Klaus D. Witzel wrote:
>> On Wed, 28 May 2008 17:59:14 +0200, Ken Causey wrote:
> ...
>> >  I made a mistake when building 3.10.1-7175, you can look at
>> >
>> > http://bugs.squeak.org/view.php?id=7070
>> >
>> > For more of the gorey details.
>> >
>> > My point with this email, other than to apologize, is to ask what I
>> > should do.  ...
>>
>> I'd say: put a text (big letters from default font) into that .image and
>> update 3.10.1-7175-basic.zip; an explanation of what's wrong with the
>> .image and what to do from here.
>>
>> This will not address possible problems of people who have already
>> downloaded the current one but, that's easy to find out: they can be  
>> asked
>> if they see the text.
>
> Thanks, that's a good idea and something I will do as soon as I find a
> way to fix this in the existing image.

You are probably already aware that

(SystemOrganization allMethodSelectors "does #elements"
        reject: [:missing | Smalltalk includesKey: missing])
  do: [:missing | SystemOrganization basicRemoveElement: missing]

still leaves some existing classes in category Morphic-CandidatesForGo.

> Ken
>
>> /Klaus
>



Reply | Threaded
Open this post in threaded view
|

Re: [squeak-dev] Bogus Class Category in 3.10.1-7175 - Mantis #7070

timrowledge
In reply to this post by Ken Causey-3
IF there is a chunk of code you can come up with to fix the problem  
AND IF there is a decent way to test for the problem existing THEN  
consider putting an update out that can check and fix plus put a  
suitably fixed image on the ftp site and remove the broken one.

That would stop anymore downloads of the broken image and provide an  
easy fix path for any copies already out there and won't cause  
problems for people loading updates to an older 3.10-7159 image

tim
--
tim Rowledge; [hidden email]; http://www.rowledge.org/tim
Logic:   The art of being wrong with confidence...



Reply | Threaded
Open this post in threaded view
|

Re: [squeak-dev] Bogus Class Category in 3.10.1-7175 - Mantis #7070

Ken Causey-3
In reply to this post by Ken Causey-3
I took Nicolas' fix and modified and extended it a bit and added it
prominently as a DO-IT along with a simple note to the Welcome window
and replaced the copies on the FTP site.

Ken

On Wed, 2008-05-28 at 10:59 -0500, Ken Causey wrote:

> Note that in the released 3.10.1-7175 you will find a category called
> #Morphic-CandidatesForGo.  Do NOT click any of the classes listed in
> that category as they are not really there but removed in 3.10-7159.
> The result is an annoying, seemingly unstoppable, series of debug
> prompts.  I made a mistake when building 3.10.1-7175, you can look at
>
> http://bugs.squeak.org/view.php?id=7070
>
> For more of the gorey details.
>
> My point with this email, other than to apologize, is to ask what I
> should do.  First note that this issue is related to my pre-release
> cleanup and is not due to any changeset in the update stream.  Take a
> 3.10-7159 and update it to 3.10.1-7175 and you will not have this
> problem.
>
> The first solution might be to simply build a replacement
> 3.10.1-7175-basic.zip without the problem and replace the copy on the
> FTP site.  However I don't think that would be a good idea.  Having
> spent a lot of time over the years helping users with issues I know how
> confusing it can be to try to replicate an issue and one of the keys is
> being able to start with the same image.  Imagine a user has downloaded
> my faulty release and is experiencing this problem and is confused.
> Imagine also a helpful person who wants to assist them with this
> problem.  The helper finds out they are using 3.10.1-7175 and downloads
> it from the FTP site.  Unfortunately this helper has not heard of this
> issue and tries the newer 3.10.1-7175 image and is unable to replicate
> the problem.  Substitute appropriately modified "Who's on First"
> variant; hilarity ensues at least if you are not a participant.
>
> So, I started a Known Issues list at
>
> http://ftp.squeak.org/3.10/KNOWNISSUES
>
> that prominently features this issue.  I would like to provide a
> workaround of some kind there and I would appreciate any insights into a
> code snippet to address this issue in 3.10.1-7175 that I can provide as
> a workaround.
>
> Otherwise the only other solution I can think of is to yet again start
> on a quick fix release and call it 3.10.2.  Thoughts?
>
> I apologize for this mistake, I can only hope (unrealistically) that was
> my only one.  Clearly having someone jump into a release branch as I
> have done here, without the proper knowledge, has consequences.
>
> Ken



signature.asc (196 bytes) Download Attachment