-
-
Notifications
You must be signed in to change notification settings - Fork 133
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
GUI hands when applying Merge modules via ILMerge #170
Comments
Thanks for the report! I think there is a problem for "Generate debug info" option. Unfortunately I'm not sure when I personally be ready to fix something because of https://twitter.com/GitHub3F/status/1302228682870841345 For 1.7.3 try to temporarily uncheck this option. Or please open PR. I can try to coordinate some workflow for new release later. |
Thanks, that worked! Though, do you know some trick to actually enable the debug info (if that's what I think it is, this is merging of .pdbs)? Couse just removing Or, since you look like you have some insight into the problem, maybe you may tell where you think is the issue and what I might need to fix the original option (as I have zero knowlage of this codebase). Anyway, thanks for help. |
/ndebug had a bit problematic implementation in ilmerge as I remember. You need either to specify different output path or remove existing .pdb before new processing. Also unsupported portable-pdb type: either pdbonly or full. And related. You can also look at manual configuring via msbuild #40 (comment) Other, follow the changes after #146 |
Please test the fix before merge:
|
* Fixed #170. GUI hands when applying Merge modules via ILMerge * Updated Id for Pre/PostProc gears * Fixed possible UI threads problems at Pre/PostProc features
* FIXED: Basic project integration stuck on adding property message. Issue #175. Modern VS/MSBuild 16.8+ * FIXED: Fixed integration with Conari and merging modules via ILMerge. Issue #170. * FIXED: double.NaN leads to errors for non exported methods. Issue #174. ``` syntax error at token '-' in: IL_0029: ldc.r8 -nan(ind) ``` * NEW: Manager. Implemented `-no-mgr` key: - Do not use manager for automatic restore the remote package. * CHANGED: FIPS compliant algorithms. Unable to install when FIPS enabled. Issue #171. * CHANGED: Improved the searching paths logic for RootPath and PkgPath, to be more loyal when no keys for some reason. Part of #175. * CHANGED: Updated MvsSln 2.5.3 https://github.com/3F/MvsSln/releases/tag/2.5.3 * CHANGED: Use special edition for DllExport project. Part of issue #171. https://github.com/3F/MvsSln branch: edition/DllExport
When I select Pre-Processing > Merge modules via ILMerge and click Apply the GUI hangs infinitely (this is, I waited few minutes).
DllExport -version
: 1.7.3+9a4bc51Information from
Data
tab or log data:stdout after hitting Apply:
The text was updated successfully, but these errors were encountered: