Issue 3693 in pharo: probable cause of 0 length mcz files in package-cache

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

Issue 3693 in pharo: probable cause of 0 length mcz files in package-cache

pharo
Status: Accepted
Owner: ----
Labels: Type-ReportDefect Importance-Low

New issue 3693 by [hidden email]: probable cause of 0 length mcz files  
in package-cache
http://code.google.com/p/pharo/issues/detail?id=3693

I was loading Seaside3.0 into Pharo1.2rc2-12324 and ran into a 'Error: file  
is too short' error. This is an error that I've seen before, but didn't  
have an idea how the "too short" file was created.

This was a fresh install of Pharo and the error occurred when I tried doing  
the load a second time ... I had interrupted the load the first time with  
an ALT-.

So I assume that the zero length file was created because I had interrupted  
Monticello while it was creating the copy in the package-cache ... which  
leads me to believe that an ensure block could be placed in the right spot  
to watch for interrupts and delete the file if the creation operation  
wasn't completed normally ...

I haven't found the spot in the code, so this is supposition on my part,  
but I think it is a good supposition:)


Reply | Threaded
Open this post in threaded view
|

Re: Issue 3693 in pharo: probable cause of 0 length mcz files in package-cache

pharo

Comment #1 on issue 3693 by [hidden email]: probable cause of 0 length  
mcz files in package-cache
http://code.google.com/p/pharo/issues/detail?id=3693

Sounds like a good idea, ifCurtailed: might be better for the purpose  
though.
Tell us if you figure out where to put it :)


Reply | Threaded
Open this post in threaded view
|

Re: Issue 3693 in pharo: probable cause of 0 length mcz files in package-cache

pharo

Comment #2 on issue 3693 by [hidden email]: probable cause of 0  
length mcz files in package-cache
http://code.google.com/p/pharo/issues/detail?id=3693

Indeed it makes sense. We should try to find where to put it.