Ok. I confirm that if I modify the code to use binaryReadStreamDo: and then add the MemoryFileSystemFile>>size method, it works fine.

So there were two problems :)

In any case, I opened an issue:
https://pharo.fogbugz.com/f/cases/12826/Zip-FileSystem-does-not-work

Cheers,
Doru




On Thu, Feb 6, 2014 at 3:30 PM, Max Leske <maxleske@gmail.com> wrote:
No, not my bug :)

The problem seems to be that WriteStream expects #size to be implemented but MemoryFileSystemFile only knows #fileSize. Apart from that I think it would work.

Add this to MemoryFileSystemFile (not as a final solution!):

size
^ bytes size

Cheers,
Max

On 06.02.2014, at 15:22, Tudor Girba <tudor@tudorgirba.com> wrote:

Thanks, Max.

Doru


On Thu, Feb 6, 2014 at 3:11 PM, Max Leske <maxleske@gmail.com> wrote:
On 06.02.2014, at 15:07, Tudor Girba <tudor@tudorgirba.com> wrote:

I am trying to get the ZipFileSystem to work, but I cannot quite manage it.

I am doing this:
(FileSystem zip:
(FileSystem disk workingDirectory / 'sample.zip'))
open

and I get an error.

The error comes from the fact that the code uses readStreamDo: which does not work anymore with binary streams. I tried to use binaryReadStreamDo:, but that crashes in another place.

I think this is a bug. Or am I doing something wrong?

You shouldn’t be… I’ll take a look later since that would be me who introduced the bug (probably).


Cheers,
Doru

--

"Every thing has its own flow"




--

"Every thing has its own flow"




--
www.tudorgirba.com

"Every thing has its own flow"