mach-kernel / cadius

A maintained fork of BrutalDeluxe's Cadius ProDOS disk imaging utility (used for making Apple II disk images).
GNU General Public License v3.0
30 stars 9 forks source link

REPLACEFILE errors despite success #16

Closed sicklittlemonkey closed 6 years ago

sicklittlemonkey commented 6 years ago

Hi, thanks for your updates to this handy tool. I was just putting a simple cross-dev script together and this puzzled me a bit:

cadius.exe v 1.4.0 (c) Brutal Deluxe 2011-2013.

  • Replacing file '/MERLIN16/M2#061000' : Error : Can't get file from Image, File not found. Error : Invalid Prodos File path '/MERLIN16/M2'.

That's a REPLACEFILE when the destination file (M2) doesn't exist. I haven't looked at the code yet but I presume the error is from the delete. It actually adds the file and returns a success code, but it would be nicer without errors.

Cheers, Nick.

mach-kernel commented 6 years ago

Hi Nick!

I apologize for being lazy, but that snippet of code is the reason why the error happens (I'm just reusing the existing delete functions but there should be an option to do so silently as required by this use case).

I'll get to it, thanks for writing in :+1:

mach-kernel commented 6 years ago

@sicklittlemonkey Would you be interested in helping me maintain this? Extra hands are always useful, but especially so for code reviews & testing before release. Block device stuff is coming soon and I want to look at testing it more closely.

sicklittlemonkey commented 6 years ago

I can't "commit" to anything at the moment, sorry, but I'll keep an eye on things when I can. There's stuff I'd like to be doing in AppleWin that I don't have time for. Kids etc come first.