Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix: did not get version number when gw2sharp.dll was included into .… #125

Merged
merged 1 commit into from
Sep 11, 2022
Merged

Fix: did not get version number when gw2sharp.dll was included into .… #125

merged 1 commit into from
Sep 11, 2022

Conversation

Taschenbuch
Copy link
Contributor

…exe with fody costura

  • fody costura wraps all .dlls into a single .exe. Because of that the gw2sharp.dll was not accessible. Thus the assembly.Location returned an empty string. This empty string caused .GetVersionInfo to throw a 'System.ArgumentException' in mscorlib.dll with System.ArgumentException: 'The path is not of a legal form.'
  • fody costura is used by BlishHud for example
  • try catch is probably not necessary anymore?

sources

…exe with fody costura

* fody costura wraps all .dlls into a single .exe. Because of that the gw2sharp.dll was not accessible. Thus the assembly.Location returned an empty string. This empty string caused .GetVersionInfo to throw
a 'System.ArgumentException' in mscorlib.dll with  System.ArgumentException: 'The path is not of a legal form.'
* fody costura is used by BlishHud for example
* try catch is probably not necessary anymore?

sources
* Fody/Fody#556
* https://www.llblgen.com/tinyforum/Thread/23972
@Archomeda Archomeda merged commit 574d18c into Archomeda:master Sep 11, 2022
@Archomeda Archomeda added the Type: Bug Something isn't working label Feb 28, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Type: Bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants