omex complains about missing files in archive

[fbergmann]

I've noticed that when importing a omex archive in pymetadata, that it complains on windows about entries missing in the manifest, even though they are present.

Looking through the code, this is due to

pymetadata/src/pymetadata/omex.py

Line 576 in d09ca89

file_path = os.path.join(root, file)

which on windows will create a string with a backslash as separator, if the file in the manifest is encoded with a slash, then the entry will not be able to be properly resolved.

[matthiaskoenig]

@fbergmann Thanks for the report. I will fix this asap.

• fix bug
• add test
• add windows to CI for testing

[fbergmann]

thanks! locally i've solved it with just replacing the backslashes with forward ones and was able to get it running

       for root, _dirs, files in os.walk(str(directory)):
            for file in files:
                file_path = os.path.join(root, file)
                location = f"./{os.path.relpath(file_path, directory)}"
                if location == "./manifest.xml":
                    # manifest is created from the internal manifest entries
                    continue

                # replace backslashes with forward slashes  <--------
                location = location.replace("\\", "/")
                ...