[Request] Remove Packer info from submission info #774

Closed
opened 2026-01-29 16:22:04 +00:00 by claunia · 6 comments
Owner

Originally created by @Deterous on GitHub (Dec 10, 2024).

Originally assigned to: @mnadareski on GitHub.

Change options.ScanPackersForProtection to only output packer info to the protection info output text file. Packer info does not belong in submissionInfo.txt as it is not stored in redump.

Originally created by @Deterous on GitHub (Dec 10, 2024). Originally assigned to: @mnadareski on GitHub. Change options.ScanPackersForProtection to only output packer info to the protection info output text file. Packer info does not belong in submissionInfo.txt as it is not stored in redump.
claunia added the enhancement label 2026-01-29 16:22:04 +00:00
Author
Owner

@mnadareski commented on GitHub (Dec 10, 2024):

This has been band-aid fixed by hard-coding exclusion lists for packers into the current filtering.

@mnadareski commented on GitHub (Dec 10, 2024): This has been band-aid fixed by hard-coding exclusion lists for packers into the current filtering.
Author
Owner

@TonyLizard commented on GitHub (Dec 10, 2024):

Please also add the EA Online Registration to the list of elements that should not be showed in submissionInfo.txt file.

@TonyLizard commented on GitHub (Dec 10, 2024): Please also add the EA Online Registration to the list of elements that should not be showed in submissionInfo.txt file.
Author
Owner

@mnadareski commented on GitHub (Dec 10, 2024):

EA CD-key will be filtered out alongside packers and game engines now. I will leave this issue open for a couple of days in case others are found.

@mnadareski commented on GitHub (Dec 10, 2024): EA CD-key will be filtered out alongside packers and game engines now. I will leave this issue open for a couple of days in case others are found.
Author
Owner

@Deterous commented on GitHub (Dec 11, 2024):

It may be cleaner to have a list of prefixes to exclude stored somewhere, and loop through all of them in the filtering code to exclude them with FindAll(p => !p.StartsWith(x). This would make maintaining the list easier (every time BOS adds new detections).
Of course the existing, more advanced filtering to remove duplicates can remain.

@Deterous commented on GitHub (Dec 11, 2024): It may be cleaner to have a list of prefixes to exclude stored somewhere, and loop through all of them in the filtering code to exclude them with `FindAll(p => !p.StartsWith(x)`. This would make maintaining the list easier (every time BOS adds new detections). Of course the existing, more advanced filtering to remove duplicates can remain.
Author
Owner

@mnadareski commented on GitHub (Dec 11, 2024):

53c6e47fa8

@mnadareski commented on GitHub (Dec 11, 2024): https://github.com/SabreTools/MPF/commit/53c6e47fa82fd3830be01ee0f6b5816014e7e41c
Author
Owner

@mnadareski commented on GitHub (Dec 12, 2024):

All cases that needed to be covered seem to be for now.

@mnadareski commented on GitHub (Dec 12, 2024): All cases that needed to be covered seem to be for now.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: SabreTools/MPF#774