Yes, it doesn't properly work in my build though I used the latest svn of physfs.
I'm considering to use older 'zip mount patch' that worked formerly.
(But I'm not sure if that works in the latest cvs of dosbox)

I haven't tried physfs out recently but I remember that I wasn't able to view the contents of the zip's (although they mounted fine) if I used "Deflate64" instead of just "deflate" for compression. Som make sure you use "deflate". This was with 7-zip.

I'll see if I can test this again tonight but that was the only issue I ever had.

That's the same for me.
It doesn't list the contents of the zip's when using dir command, but can run executable in the archive.
(I didn't know that 7z files can be mounted as a drive.)

Nah, not .7z archive. I use 7-zip to make .zip archives since it uses the best compression.

ykhwong, it must be something strange in your build. I keep the patch up-to-date, and it works fine here. You might want to test a build using only the physfs-patch to see if it's some interaction with another patch that breaks things.

No, I've just tried to apply only your patch to compile and test my build with zip mount feature (physfs). -> Result is the same.

Thanks for your replies guys (was getting worried I was going mad for a while)

I would love to see this feature fully working as at the mo I have to have a script to unpack the archive and then manually addin any saved data then muck around with a post run script to tidy everything up where as this would solve that in one hit.

Good work moe for making the patch and hopefully yk's build will one day be able to support it 😁

I wanted to start using physfs the other day, and it wasn't working... exact same symptoms you all describe here... so, I dug into it a little and I found a bug in the physfs patch which rendered it useless in windows:

file: src/dos/drive_physfs.cpp
line: ~74

content:

1	if (strlen(basedir) > strlen(name)) strcpy(name,basedir);

should be:

1	if (strlen(basedir) > strlen(name)) {
2		strcpy(name,basedir);
3		strreplace(name,'\\','/');
4	}

Thanks, I can see a list of files/dirs in a directory mounted with zip.

Massive well done to Kronz for srting that one out just tried ykhwong's latest build and it works a treat 😀