Ok, I think I will retrofit my tryout on top of the existing work of guille.
camillo


On 2011-03-15, at 13:29, Guillermo Polito wrote:

> Hi!
> 
> On Tue, Mar 15, 2011 at 3:18 AM, Tudor Girba <[email protected]> wrote:
> 
>> Hi,
>> 
>> It seems to me that there are problems due to the merging of Keymapping sub
>> packages into one  Keymapping package:
>> - first, camillo seems to publish in Keymapping, while Guillermo publishes
>> in Keymapping-*. This means that one or the other will most certainly lose
>> code.
>> 
> 
> Actually, we had a couple of problems:
> - I saw the changes made by Camillo, but his version is broken and it was
> not working.
> - I couldn't merge some changes I did because we did not get really
> synchronized at first.
> - We want to change the same things, but we don't like each other's solution
> nor process of development :$.
> 
> 
>> - second, the version published by Camillo reached
>> Keymapping-CamilloBruni.3, but there already were versions that reached
>> Keymapping-cds.113, which makes Monticello list the new commits by Camillo
>> at the bottom
>> 
>> Would it be possible if you two would get synchronized?
>> 
>> 
>> Also, ConfigurationOfKeymapping is still broken:
>> - loading "(self project version: '1.5') load" still raises the
>> Character>>+ problem due to the initialization
>> - however, loading "(self project version: '1.5-baseline') load" works
>> because it takes the new versions into account
>> 
>> Would it be possible to update the configuration?
>> 
>> 
> I created development version 1.6, which contains a lot of refactorings
> already.  It's preety better.  I removed the string matching between
> shortcuts, and the ugly selectors to look at the match of a mapping.  I'd
> like it to reviewed...
> 
> 
>> 
>> Cheers,
>> Doru
>> 
>> 
>> 
> Cheers,
> Guille
> 
> 
>> On 14 Mar 2011, at 15:20, Camillo Bruni wrote:
>> 
>>> 
>>> On 2011-03-14, at 15:17, Tudor Girba wrote:
>>> 
>>>> I saw :). Just two questions:
>>>> - Is the ConfigurationOfKeymapping updated?
>>> 
>>> that I pushed to the existing branch
>>> 
>>>> - Is it already usable?
>>> not yet, didn't have too much time so far, but the basic tests are
>> working.
>>> the interface will stay the same, but the internals will be much cleaner
>> and more explicit
>>> 
>>>> Cheers,
>>>> Doru
>>>> 
>>>> 
>>>> On 14 Mar 2011, at 15:15, Camillo Bruni wrote:
>>>> 
>>>>> I pushed everying into a single Keymapping package for now. As soon as
>> there is full functionality we should separate it again.
>>>>> 
>>>>> 
>>>>> camillo
>>>>> 
>>>>> 
>>>>> On 2011-03-13, at 22:40, Tudor Girba wrote:
>>>>> 
>>>>>> Excellent initiative Camillo!
>>>>>> 
>>>>>> Regarding multiple packages: Having multiple packages limits the
>> conflicts, and saving them individually (for now) it's a small price to pay.
>>>>>> 
>>>>>> Cheers,
>>>>>> Doru
>>>>>> 
>>>>>> 
>>>>>> On 13 Mar 2011, at 22:28, Camillo Bruni wrote:
>>>>>> 
>>>>>>> furthermore, lets use a single repos/package (or whatever this is
>> called in MC).
>>>>>>> I do not like to commit 3 times while refactoring.
>>>>>>> later on we can still split it up so people can actually decide on
>> what to load.
>>>>>>> 
>>>>>>> camillo
>>>>>>> 
>>>>>>> On 2011-03-13, at 22:20, Camillo Bruni wrote:
>>>>>>> 
>>>>>>>> I can push my changes. but I don't think we should rely too much on
>> your old code.
>>>>>>>> 
>>>>>>>> Im trying to keep the structure of the classes, that was already
>> very nice IMO.
>>>>>>>> I manly adress the following issues:
>>>>>>>> 
>>>>>>>> - use of arrays as result (dedicatet results object)
>>>>>>>> - string to match the shortcuts with the incoming keyboard event
>> (dropped all of that and started to work on tests to use the shortcuts
>> directly)
>>>>>>>> - weird event matching directly on morph (simplified and using a
>> recursive function call now)
>>>>>>>> - horrible unreadable variable names (wherever I started I tried to
>> put long names to make the code readable)
>>>>>>>> 
>>>>>>>> I suggest we can work together on the new code base, since the
>> interface will stay fairly compatible.
>>>>>>>> 
>>>>>>>> camillo
>>>>>>>> 
>>>>>>>> On 2011-03-13, at 22:08, Guillermo Polito wrote:
>>>>>>>> 
>>>>>>>>> Camillo, I was fixing some tests and going to refactor some ugly
>> parts of
>>>>>>>>> the package.
>>>>>>>>> 
>>>>>>>>> Is there a way to join forces so we don't step into the other work?
>>>>>>>>> 
>>>>>>>>> Guille
>>>>>>>>> 
>>>>>>>>> On Sun, Mar 13, 2011 at 6:04 PM, Camillo Bruni <
>> [email protected]>wrote:
>>>>>>>>> 
>>>>>>>>>> I started on the last Lille sprint a complete rewrite of the
>> Keymapping
>>>>>>>>>> package. As of now it is not yet functional but the growing
>> test-coverage
>>>>>>>>>> should help to solve this issue.
>>>>>>>>>> 
>>>>>>>>>> m(^_-)m
>>>>>>>>>> camillo
>>>>>>>>>> 
>>>>>>>>>> On 2011-03-03, at 15:16, Camillo Bruni wrote:
>>>>>>>>>> 
>>>>>>>>>>> Right,
>>>>>>>>>>> 
>>>>>>>>>>> the stable has a preconditio which limits it to pharo 1.2.
>>>>>>>>>>> Furthermore the initialization code seems to be incompatible as
>> it uses
>>>>>>>>>> to:do: on Character which is AFAIK not implemented in the core
>> image Pharo
>>>>>>>>>> 1.3. Hence apply the following changes:
>>>>>>>>>>> 
>>>>>>>>>>> KMKeyEvent class >> initializeControlSequences
>>>>>>>>>>> 
>>>>>>>>>>> ...
>>>>>>>>>>> $a asciiValue to: $z asciiValue do: [:each |
>>>>>>>>>>> d add: each asCharacter -> (each - $a asciiValue + 1)].
>>>>>>>>>>> ...
>>>>>>>>>>> 
>>>>>>>>>>> then it should work.
>>>>>>>>>>> 
>>>>>>>>>>> m(^_-)m
>>>>>>>>>>> camillo
>>>>>>>>>>> 
>>>>>>>>>>> 
>>>>>>>>>>> On 2011-03-03, at 09:25, Tudor Girba wrote:
>>>>>>>>>>> 
>>>>>>>>>>>> Hi,
>>>>>>>>>>>> 
>>>>>>>>>>>> I am very interested to get Keymapping integrated into Glamour.
>> Could
>>>>>>>>>> someone help me to load it?
>>>>>>>>>>>> 
>>>>>>>>>>>> I tried:
>>>>>>>>>>>> - in Pharo 1.2:
>>>>>>>>>>>> Gofer it squeaksource: 'Keymapping'; package:
>>>>>>>>>> 'ConfigurationOfKeymapping'; load.
>>>>>>>>>>>> (ConfigurationOfKeymapping project version: #stable) load
>>>>>>>>>>>> 
>>>>>>>>>>>> - in Pharo 1.3:
>>>>>>>>>>>> Gofer it squeaksource: 'Keymapping'; package:
>>>>>>>>>> 'ConfigurationOfKeymapping'; load.
>>>>>>>>>>>> (ConfigurationOfKeymapping project version: '1.5') load
>>>>>>>>>>>> 
>>>>>>>>>>>> 
>>>>>>>>>>>> Cheers,
>>>>>>>>>>>> Doru
>>>>>>>>>>>> 
>>>>>>>>>>>> 
>>>>>>>>>>>> On 27 Feb 2011, at 09:59, Tudor Girba wrote:
>>>>>>>>>>>> 
>>>>>>>>>>>>> I also I cannot load Keymapping 1.5 in Pharo 1.2. I get DNU for
>>>>>>>>>> Character>>+. This is due to an initialization in KMKeyEvent (see
>> the
>>>>>>>>>> attached debug log).
>>>>>>>>>>>>> 
>>>>>>>>>>>>> I did the followings:
>>>>>>>>>>>>> Gofer it
>>>>>>>>>>>>> squeaksource: 'Keymapping';
>>>>>>>>>>>>> package: 'ConfigurationOfKeymapping';
>>>>>>>>>>>>> load.
>>>>>>>>>>>>> 
>>>>>>>>>>>>> (ConfigurationOfKeymapping project version: #stable) load
>>>>>>>>>>>>> 
>>>>>>>>>>>>> Am I missing something, or is this version not supposed to work
>> in
>>>>>>>>>> Pharo 1.2?
>>>>>>>>>>>>> 
>>>>>>>>>>>>> Cheers,
>>>>>>>>>>>>> Doru
>>>>>>>>>>>>> 
>>>>>>>>>>>>> <PharoDebug.log>
>>>>>>>>>>>>> 
>>>>>>>>>>>>> 
>>>>>>>>>>>>> On 26 Feb 2011, at 21:08, Francisco Ortiz Peñaloza wrote:
>>>>>>>>>>>>> 
>>>>>>>>>>>>>> You're telling me that if i do a clean installation of 1.5 it
>> would
>>>>>>>>>> work?
>>>>>>>>>>>>>> 
>>>>>>>>>>>>>> Thanks in advance,
>>>>>>>>>>>>>> Francisco
>>>>>>>>>>>>>> 
>>>>>>>>>>>>>> On Sat, Feb 26, 2011 at 3:50 PM, Guillermo Polito
>>>>>>>>>>>>>> <[email protected]> wrote:
>>>>>>>>>>>>>>> Mmm, If you had 1.4 and updated to 1.5, you will have some
>> problems
>>>>>>>>>> because
>>>>>>>>>>>>>>> I did some refactorings on that... :/.
>>>>>>>>>>>>>>> 
>>>>>>>>>>>>>>> On Sat, Feb 26, 2011 at 10:12 AM, Francisco Ortiz Peñaloza
>>>>>>>>>>>>>>> <[email protected]> wrote:
>>>>>>>>>>>>>>>> 
>>>>>>>>>>>>>>>> Guille i was using 1.4 and worked excellent, just tried 1.5
>> and on
>>>>>>>>>>>>>>>> every stroke i made i've got a DNU on #realtarget
>>>>>>>>>>>>>>>> 
>>>>>>>>>>>>>>>> Installed on last PharoCore 1.2, should i try it on 1.3?
>>>>>>>>>>>>>>>> 
>>>>>>>>>>>>>>>> Great work,
>>>>>>>>>>>>>>>> Fran
>>>>>>>>>>>>>>>> 
>>>>>>>>>>>>>>>> On Sat, Feb 26, 2011 at 5:29 AM, laurent laffont
>>>>>>>>>>>>>>>> <[email protected]> wrote:
>>>>>>>>>>>>>>>>> 
>>>>>>>>>>>>>>>>> On Sat, Feb 26, 2011 at 5:42 AM, Guillermo Polito
>>>>>>>>>>>>>>>>> <[email protected]> wrote:
>>>>>>>>>>>>>>>>>> 
>>>>>>>>>>>>>>>>>> What do we have now?
>>>>>>>>>>>>>>>>>> 
>>>>>>>>>>>>>>>>>> - Can provide settings for a set of morphs
>>>>>>>>>>>>>>>>>> 
>>>>>>>>>>>>>>>>>> - Can provide settings for a TextEditors (Smalltalk editor
>> and
>>>>>>>>>> related)
>>>>>>>>>>>>>>>>>> 
>>>>>>>>>>>>>>>>>> - Settings integration
>>>>>>>>>>>>>>>>>> I added some methods to the Settings Tree Builder in order
>> to
>>>>>>>>>> avoid
>>>>>>>>>>>>>>>>>> references from the users code.
>>>>>>>>>>>>>>>>>> 
>>>>>>>>>>>>>>>>>> - I ran Slint over it and cleaned it a lot more :). ( And
>> learnt
>>>>>>>>>> that
>>>>>>>>>>>>>>>>>> Slint is there :P )
>>>>>>>>>>>>>>>>>> 
>>>>>>>>>>>>>>>>>> More info in here:  http://guilleel3.blogspot.com/
>>>>>>>>>>>>>>>>> 
>>>>>>>>>>>>>>>>> A new blog, cool !
>>>>>>>>>>>>>>>>> Can I have Emacs-like keybinding in code editor, to switch
>> browser,
>>>>>>>>>> ...
>>>>>>>>>>>>>>>>> ?
>>>>>>>>>>>>>>>>> Laurent.
>>>>>>>>>>>>>>>>> 
>>>>>>>>>>>>>>>>>> 
>>>>>>>>>>>>>>>>>> Guille
>>>>>>>>>>>>>>>>> 
>>>>>>>>>>>>>>>>> 
>>>>>>>>>>>>>>>> 
>>>>>>>>>>>>>>> 
>>>>>>>>>>>>>>> 
>>>>>>>>>>>>>> 
>>>>>>>>>>>>> 
>>>>>>>>>>>>> --
>>>>>>>>>>>>> www.tudorgirba.com
>>>>>>>>>>>>> 
>>>>>>>>>>>>> "Every thing has its own flow."
>>>>>>>>>>>>> 
>>>>>>>>>>>>> 
>>>>>>>>>>>>> 
>>>>>>>>>>>>> 
>>>>>>>>>>>> 
>>>>>>>>>>>> --
>>>>>>>>>>>> www.tudorgirba.com
>>>>>>>>>>>> 
>>>>>>>>>>>> "Every thing should have the right to be different."
>>>>>>>>>>>> 
>>>>>>>>>>>> 
>>>>>>>>>>>> 
>>>>>>>>>>>> 
>>>>>>>>>>> 
>>>>>>>>>>> 
>>>>>>>>>> 
>>>>>>>>>> 
>>>>>>>>>> 
>>>>>>>> 
>>>>>>>> 
>>>>>>> 
>>>>>>> 
>>>>>> 
>>>>>> --
>>>>>> www.tudorgirba.com
>>>>>> 
>>>>>> "Value is always contextual."
>>>>>> 
>>>>>> 
>>>>>> 
>>>>>> 
>>>>> 
>>>>> 
>>>> 
>>>> --
>>>> www.tudorgirba.com
>>>> 
>>>> "If you interrupt the barber while he is cutting your hair,
>>>> you will end up with a messy haircut."
>>>> 
>>>> 
>>> 
>>> 
>> 
>> --
>> www.tudorgirba.com
>> 
>> "Some battles are better lost than fought."
>> 
>> 
>> 
>> 
>> 


Reply via email to