Re: [Pharo-dev] I need reviewer of https://pharo.fogbugz.com/default.asp?15646

You guys are moving so fast. I went to review it but it is already integrated :P Anyway I tried it and it worked well, load Stef's 15646 slice no problem. Well done Nicolai. cheers -ben On Wed, Jun 3, 2015 at 5:41 PM, Nicolai Hess wrote: > > > 2015-06-03 8:22 GMT+02:00 Marcus Denker : >> >> Inde

Re: [Pharo-dev] I need reviewer of https://pharo.fogbugz.com/default.asp?15646

2015-06-03 15:46 GMT+02:00 Thierry Goubier : > > > 2015-06-03 15:09 GMT+02:00 Nicolai Hess : > >> >> >> self versionFromFileNamed: fileName >> is called after it isn't found in the MCCacheRepository >> and if it is not found in its own special repository cache , it is loaded >> with >> self loadVe

Re: [Pharo-dev] I need reviewer of https://pharo.fogbugz.com/default.asp?15646

2015-06-03 15:09 GMT+02:00 Nicolai Hess : > > > self versionFromFileNamed: fileName > is called after it isn't found in the MCCacheRepository > and if it is not found in its own special repository cache , it is loaded > with > self loadVersionFromFileNamed: > and this again looks up in the MCCache

Re: [Pharo-dev] I need reviewer of https://pharo.fogbugz.com/default.asp?15646

2015-06-03 14:47 GMT+02:00 Thierry Goubier : > > > 2015-06-03 14:19 GMT+02:00 Nicolai Hess : > >> >> >> 2015-06-03 13:22 GMT+02:00 Thierry Goubier : >> >>> >>> >>> 2015-06-03 11:41 GMT+02:00 Nicolai Hess : >>> 2015-06-03 8:22 GMT+02:00 Marcus Denker : > Indeed… this has to

Re: [Pharo-dev] I need reviewer of https://pharo.fogbugz.com/default.asp?15646

2015-06-03 14:19 GMT+02:00 Nicolai Hess : > > > 2015-06-03 13:22 GMT+02:00 Thierry Goubier : > >> >> >> 2015-06-03 11:41 GMT+02:00 Nicolai Hess : >> >>> >>> >>> 2015-06-03 8:22 GMT+02:00 Marcus Denker : >>> Indeed... this has to do something with caching: when looking a .mcz the system j

Re: [Pharo-dev] I need reviewer of https://pharo.fogbugz.com/default.asp?15646

2015-06-03 13:22 GMT+02:00 Thierry Goubier : > > > 2015-06-03 11:41 GMT+02:00 Nicolai Hess : > >> >> >> 2015-06-03 8:22 GMT+02:00 Marcus Denker : >> >>> Indeed… this has to do something with caching: when looking a .mcz the >>> system just takes the one it finds, which might be >>> one with the sa

Re: [Pharo-dev] I need reviewer of https://pharo.fogbugz.com/default.asp?15646

2015-06-03 11:41 GMT+02:00 Nicolai Hess : > > > 2015-06-03 8:22 GMT+02:00 Marcus Denker : > >> Indeed... this has to do something with caching: when looking a .mcz the >> system just takes the one it finds, which might be >> one with the same name but different. >> > > here is a minimal change to

Re: [Pharo-dev] I need reviewer of https://pharo.fogbugz.com/default.asp?15646

2015-06-03 8:22 GMT+02:00 Marcus Denker : > Indeed… this has to do something with caching: when looking a .mcz the > system just takes the one it finds, which might be > one with the same name but different. > here is a minimal change to the way it uses the cache, or omitting the cache after the

Re: [Pharo-dev] I need reviewer of https://pharo.fogbugz.com/default.asp?15646

2015-06-03 9:56 GMT+02:00 Marcus Denker : > > On 03 Jun 2015, at 09:52, Thierry Goubier > wrote: > > > > 2015-06-03 8:22 GMT+02:00 Marcus Denker : > >> Indeed... this has to do something with caching: when looking a .mcz the >> system just takes the one it finds, which might be >> one with the sa

Re: [Pharo-dev] I need reviewer of https://pharo.fogbugz.com/default.asp?15646

> On 03 Jun 2015, at 09:52, Thierry Goubier wrote: > > > > 2015-06-03 8:22 GMT+02:00 Marcus Denker >: > Indeed… this has to do something with caching: when looking a .mcz the system > just takes the one it finds, which might be > one with the same name but diff

Re: [Pharo-dev] I need reviewer of https://pharo.fogbugz.com/default.asp?15646

2015-06-03 8:22 GMT+02:00 Marcus Denker : > Indeed... this has to do something with caching: when looking a .mcz the > system just takes the one it finds, which might be > one with the same name but different. > Unless the mcz searched for is a dependency, in which case it checks the uuid as well

Re: [Pharo-dev] I need reviewer of https://pharo.fogbugz.com/default.asp?15646

Indeed… this has to do something with caching: when looking a .mcz the system just takes the one it finds, which might be one with the same name but different. Maybe we could for real put (part of) the UID in the filename? In a disconnected system the case that the filename is the same for diffe

Re: [Pharo-dev] I need reviewer of https://pharo.fogbugz.com/default.asp?15646

I hate so much this bug Le 1/6/15 15:04, Ben Coman a écrit : Stef, I can now see all the dependent packages for the new slice, but I still have a strange error. However I'm not sure if its a bug or something unique at my end. Can someone try merging SLICE-Issue-15646-Cleaning-method-categ

Re: [Pharo-dev] I need reviewer of https://pharo.fogbugz.com/default.asp?15646

I think the problem is that, some of the packages like Kernel-StephaneDucasse.2042 DebuggerActions-StephaneDucasse.75 are in both repositories (main/inbox) but with different UUIDs. 2015-06-01 15:04 GMT+02:00 Ben Coman : > Stef, I can now see all the dependent packages for the new slice, but >

Re: [Pharo-dev] I need reviewer of https://pharo.fogbugz.com/default.asp?15646

Stef, I can now see all the dependent packages for the new slice, but I still have a strange error. However I'm not sure if its a bug or something unique at my end. Can someone try merging SLICE-Issue-15646-Cleaning-method-category-api-should-be-protocol-part-1-StephaneDucasse.1 What I see at to

Re: [Pharo-dev] I need reviewer of https://pharo.fogbugz.com/default.asp?15646

I tried, but it seems some packages are missing from the inbox. cheers -ben On Sun, May 31, 2015 at 2:19 PM, stepharo wrote: > Hi > > I continued to clean that classes have categories and method protocols > because it was not finished. > This entry is just adding protocol in the classes that were

[Pharo-dev] I need reviewer of https://pharo.fogbugz.com/default.asp?15646

Hi I continued to clean that classes have categories and method protocols because it was not finished. This entry is just adding protocol in the classes that were missing it, adding comments, and fixing some local senders It does not remove the category API but puts it in a accessing-backward