Skip to content
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

DLC sometimes comes as non-v0 and are handled incorrectly #105

Closed
jockebr opened this issue Jan 13, 2022 · 1 comment
Closed

DLC sometimes comes as non-v0 and are handled incorrectly #105

jockebr opened this issue Jan 13, 2022 · 1 comment

Comments

@jockebr
Copy link

jockebr commented Jan 13, 2022

It happens rarely, but sometimes DLC have updated versions (v0 and v65536 or higher) and when using --rm-old-version the detection logic seems to incorrectly deem them to be updates and error out.

Using command line: nsz.py -C --verify --parseCnmt --rm-old-version --rm-source *.nsp

Example error:

Error while compressing file: /path/Some DLC [0100xxxxxxxxx011] [v65536].nsp
Traceback (most recent call last):
  File "/path/nsz/nsz/__init__.py", line 161, in main
    if not AllowedToWriteOutfile(filePath, ".nsz", targetDictNsz[outFolder], args):
  File "/path/nsz/nsz/FileExistingChecks.py", line 121, in AllowedToWriteOutfile
    raise ValueError("rm-old-version: A titleID containing updates should never have any version v0 with the same titleID!")
ValueError: rm-old-version: A titleID containing updates should never have any version v0 with the same titleID!

Note there exists both a v0 and v65536 of this DLC title id in this directory.

Omitting --rm-old-version works around the problem.

@nicoboss
Copy link
Owner

nicoboss commented Sep 4, 2022

Thanks a lot for reporting this. I wasn’t aware that there are v0 DLCs. I now removed the safety check that prevents v0 titles to be deleted if there is an update for it. The original purpose of this safety check was to protect base titles in the unlikely case there is a titleID naming issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants