-
Notifications
You must be signed in to change notification settings - Fork 38
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Polish support for ISO9660 file format #264
Comments
Thank you! I have a few points to make, but I'm super busy and will catch up on this soon! |
Found this in someone else's log.
|
Man, this is a rough one. I thought, back when you opened this issue, I found another ISO library for Go (seo: golang). Today, I'm only finding 3 libraries, and I seem to be using the 'best' one. None of them support Joilet file extensions, which means file names over 32 characters are out. The bugs you've run into seem to be directly in this library. I don't believe I can fix them myself. I'm also afraid the 4 GB file limitation is built into the library, but I think it may be inadvertently used for extractions when it should be used for compression. Not entirely sure yet. Question for ya @blackwind .. if I give you a spot to upload, can you send me an ISO file or two that didn't work? I'll try to engage with @kdomanski once I have a reproducible example to share with him. This is the library I'm using now: These are the other two I found:
EDIT: Found more that are 2+ years old:
If anyone find a good ISO9660 library for Go.. lemme know. |
Proper support for these files will be a huge time-saver for me, so absolutely, I'm happy to help in any way I can. The one I used in my log (X-Blades_HD-DINOByTES) is a good example of all mentioned issues and is available in the obvious places, but I'll do the legwork if you need me to for whatever reason. |
I'll try that file with a few of these libraries. Will see if anything can extract it. |
That's a UDF descriptor. |
It's funny, because the ISO9660 library I currently use just released a new version. Literally the only significant change they made was to add an error that says "UDF volumes are not supported." rip. I'm messing with this a little bit today, but I'm not very optimistic. :( |
I'm stumped at this point. The only actively maintained libraries I can find do not support 2 or more of:
I did find 1 old library that has Rock Ridge support, and 1 that supports UDF, but I don't think I found any that support Joliet. Ideally, the rock ridge support can be ported into https://github.com/kdomanski/iso9660 or https://github.com/diskfs/go-diskfs or both. |
Funny indeed. Maybe the author got a notification when you mentioned him, and saw your post. The 1 library with Rock Ridge support that you linked, it only gets the full filename from the RR data, but not timestamps. Looks like the master branch of the library you use already has RR test fixture added, so full RR support might drop any time. Supporting Joliet might then be redundant for your usecase, we'll see. As for files larger than 4GB, this requires support for multi-extent descriptors. It's not hard to implement, but it requires a bit of free time. |
Love it. Thanks for stopping by!
haha, don't be so modest. I see your recent commits (now), and am very pleased!
I hope so. Seems like rock ridge will give us what we're missing.
Give me a pointer or two? I'm willing to try if you think it might be a worthwhile use of my time. This is probably the last "hurdle." EDIT: derp moment. Just realized who I replied to earlier. haha EDIT2: and now realizing the new release you made was probably because of this issue, and that error message you quoted. Thank you :) |
ECMA-119 9.1.6: multi-extent flag. It's not very explicit, but I infer that maybe it means a multi-extent file has several consecutive Directory Records and the flag turned on. The Linux Kernel's code for this appears to interpret this flag as an indication of the given DE not being the last one for the file.
Looks like (outside of some edge cases) Linux will use RR and ignore Joliet if both are present. |
@kdomanski You're right, overall this doesn't look too hard. It's going to take me a bit to come up to speed on this, but I've got a couple hours into it now and may be able to get there. Here's where I'm at... None of the files have
Here's my "update" to do some debugging: kdomanski/iso9660@45c0c7d I ran this new code against the ISO file mention earlier in the thread. Here's the whole output: This is what it looks like mounted on my Mac: ...which makes me want to ask: Is the volume name exposed by this library yet? (the name in the title) |
Hmm, this might be a Joliet-only image. I'll look into the dump you provided.
it is now. ;-) https://github.com/kdomanski/iso9660/releases/tag/v0.3.5 |
amazing! |
Any further progress on this? Or are we blocked indefinitely? |
No one has ever extracted or created these 'advanced' format ISO images with Go apps. This is all new. kdomanski is the only person that's put together a comprehensive library that will one day provide these features. Today, it does not. I haven't had time to visit this. I have dozens of projects, and this feature is a lot of work, so it will be a while before I'm intrigued enough to spend the time required. There has been no further progress at this time. |
If you detected impatience in my tone, none was intended. I appreciate the update and all the work done on this so far. |
Sup. Release v0.4.0 can read Rock Ridge filenames. |
There's probably more I can do here, but I updated the library and pushed some updates. You can download it here https://unstable.golift.io - thanks Kamil! EDIT: Docker is ready. |
Unable to test until the Docker image is available, but it sounds like no more filename truncation, no more incorrect filename casing, but the other issues persist. I've marked the completed tasks in the first post. |
Currently just getting "UDF volumes are not supported", which I guess is an improvement over the old behavior. |
What is the image you're testing? UDF is probably another problem that needs a solution. |
Tried a few, but Stray.v1.5-Razor1911 is a well sized one for testing the 4GB issue as well. |
_unpackerred
folder instead of creating a subfolder based on ISO's filenameThe text was updated successfully, but these errors were encountered: