-
Notifications
You must be signed in to change notification settings - Fork 2
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
fmt/1157 #11
Comments
Is this an error with PRONOM do you think @richardlehane? |
it is a bit of an odd one, but I think intended: i.e. looks for two sequences "Folio" and "v200" which both need to be near the start of the file but can be in any order |
confirmed with@Dclipsham . Failed testing DROID on this file
|
Thanks Jeremie, et al. I should have done something with this but forgot about it... Not sure I'll have a fix for a few weeks, but let me know if it does help you to have it sooner or if you can live without for this release. |
Thanks Ross, it just came up as part of DROID testing. I've created a pair of files that contain the missing 'v200' so a fix certainly isn't urgent from our perspective (am happy to share these - GitHub Issues doesn't like the file type though for file upload - let me know if you want them..). As Richard points out we cant be certain of the order in which 'Folio' and 'v200' will appear so expressed them both as distinct BOFs with offset range set |
Sounds good, if you zip them, the can be added here, and I can use them for reference in the fix! I have of course excitedly being watching references to the suite in recent testing. Big changes. They look to be shaping up well. |
Great - learning something new :) thank you! |
the latest release (v108) has a couple more signatures impacted by this: fmt/1739 and 1757 |
Wondering if I fixed this for container signatures and whether this will work here: exponential-decay/skeleton-container-test-suite-generator#14 |
fmt/1062 (https://www.nationalarchives.gov.uk/PRONOM/Format/proFormatSearch.aspx?status=detailReport&id=1868&strPageToDisplay=signatures) also seems to be impacted by this issue |
The new Folio Infobase File has overlapping beginning-of-file sequences:
Only the "Folio" bit is being generated in the test signature.
The text was updated successfully, but these errors were encountered: