On Sat, Jul 23, 2016 at 2:29 AM, Dale Henrichs <dale.henri...@gemtalksystems.com> wrote: > > > On 07/22/2016 11:19 AM, Ben Coman wrote: >>> >>> On 07/21/2016 08:40 PM, Ben Coman wrote: >>>> >>>> Just a general observation and first impression of installing a github >>>> hosted project. >>>> >>>> After installing OSSubprocess from the Catalog Browser on Pharo 50760, >>>> which uses repository... >>>> github://marianopeck/OSSubProcess:v0.2.4/repository >>>> >>>> ...I see file >>>> "github--marianopeckOSSubprocessv0241266307023333901887488330764830.zip" >>>> sitting next to my .image file. That would seem to dirty up that area >>>> when I start to use git more often. I'd expect that file to be placed >>>> in >>>> one of the cache folders. >>>> >>>> cheers -ben >>> >>> >>> >> On Sat, Jul 23, 2016 at 1:38 AM, Dale Henrichs >> <dale.henri...@gemtalksystems.com> wrote: >>> >>> Ben, >>> >>> I think I've found the bug ... Metacello intends that the zip file be >>> downloaded to /tmp, but the code as implemented 3 years ago for Pharo was >>> not actually downloading the file to /tmp ... thanks for pointing this >>> out, >>> I've created a Metacello issue for this[1] ... I will probably fix this >>> shortly, but I'm not sure when (or if) my fix will be integrated into the >>> Pharo-5.0 release. >> >> Cool. Thanks Dale. I created issues for Pharo 5 & 6 to track this our >> end. >> https://pharo.fogbugz.com/default.asp?18804 >> https://pharo.fogbugz.com/default.asp?18805 >> >> cheers -ben >> >> P.S.@All curious why "Area=divers" for "Project=Metacello" >> >> > Ben, > > Sorry ... I gave you the wrong issue # for this one ... I opened 3 new > issues for this message and only 2 of them applied directly '#405: Github > .zip download not downloaded to `/tmp` as expected ... Pharo-5.0' and '#406: > Github .zip download file should be deleted after archive is extracted' ... > #404 is not really an issue, since the pid is meant for disambiguation of > the file name, which is exactly the same thing that is supposed to be done > by FileReference class>>newTempFilePrefix:suffix: ... > > Too many windows floating around:) > > Dale >
@All, btw in the pharo-project swimlane here [1] it would be nice to have a branch tagged "Pharo5.0" showing the point that was integrated into Pharo 5. cheers -ben