[squeak-dev] The Inbox: Monticello-mva.662.mcz

Milan Vavra vavra_milan at yahoo.com
Thu Feb 9 19:10:35 UTC 2017


You are right. I am sorry. I can not reproduce this issue anymore.

So now the commit message is wrong on Monticello-mva.662.mcz.

This was intended as two part commit to make things easier to see
in History that seemed to have value on its own.
It no longer has value on its own and the commit message on it is wrong.

The bigger issue is that of

'unable to use filter on large remote repositories like the trunk'.

Try this:

Open the trunk Repository browser.
Make sure there is no selection on the left
(deselect 39Deprecated).
Select a package (Squeak-Version-tfel.4726.mcz) on the right.
Try typing 6 to filter for all those that have 6 in them.
This filters the list but is very slow.
As it stops, it downloads the package it found.
Now try to type 66 to filter for all those that have 66 in them.
Not try to type 660 to filter for all those that have 660 in them.
Type backspace to remove the filter.

Commit message for Monticello-mva.662.mcz should have been:

This is part 1 of 2 to resolve
'unable to use filter on large remote repositories like the trunk'
This part stores the index of the selected item versionSelection when
changed
with #versionSelection: instead of looking it up linearly with #indexOf:
each time #versionSelection is called.

Commit message for Monticello-mva.663.mcz would have been:

This is part 2 of 2 to resolve
'unable to use filter on large remote repositories like the trunk'
This part calculates and stores the versionList lazily when set to nil
with #versionList: instead of calculating it each time #versionList is
called.

So where do we go from here?

I am not sure how to proceed.

The commit message of the Monticello-mva.662.mcz
I have uploaded to the inbox is clearly wrong. Despite of the value of the
code for the combined fix.

Can I just create a new Monticello-mva.662.mcz and upload it to the inbox
with the proposed commit message?
Will that overwrite the old one?

Or should I upload the Monticello-mva.663.mcz to the inbox so that you can
see that when combined with Monticello-mva.663.mcz it solves the
'unable to use filter on large remote repositories like the trunk'
issue?

Best Regards,

Milan Vavra



--
View this message in context: http://forum.world.st/The-Inbox-Monticello-mva-662-mcz-tp4933676p4933726.html
Sent from the Squeak - Dev mailing list archive at Nabble.com.


More information about the Squeak-dev mailing list