Skip to content

Added ParseVersionResourcesForEntries to support files with multiple product versions.#105

Merged
ayoubfaouzi merged 1 commit into
saferwall:mainfrom
getvictor:getvictor/multiple-versions
Nov 12, 2024
Merged

Added ParseVersionResourcesForEntries to support files with multiple product versions.#105
ayoubfaouzi merged 1 commit into
saferwall:mainfrom
getvictor:getvictor/multiple-versions

Conversation

@getvictor
Copy link
Copy Markdown
Contributor

A self-extracting archive like https://get.opera.com/ftp/pub/opera/desktop/114.0.5282.94/win/Opera_114.0.5282.94_Autoupdate_x64.exe contains the product versions for 7-Zip and the actual application.

This PR adds the ParseVersionResourcesForEntries method, which returns these separate product versions as a slice. This allows us to separate the 7-Zip version from the Opera version.

ParseVersionResources has been refactored for code reuse.

@ayoubfaouzi
Copy link
Copy Markdown
Member

Thanks @getvictor !

LGTM, as long as test passes. If you have a small binary we can test with, it will be awesome. Otherwise, I will dig my collection of file to find one.

@ayoubfaouzi
Copy link
Copy Markdown
Member

image

@ayoubfaouzi ayoubfaouzi merged commit b7e3f0e into saferwall:main Nov 12, 2024
@ayoubfaouzi
Copy link
Copy Markdown
Member

Your feature has been included in the last tag: v1.5.5. Cheers mate.

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

Successfully merging this pull request may close these issues.

2 participants