-
Notifications
You must be signed in to change notification settings - Fork 576
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
Revert "Strip static and shared libraries of iwasm to reduce the bin..." #2669
Conversation
…ry size (bytecodealliance#2431)" This reverts commit 7e3a29d. * it doesn't work for macOS. ("unrecognized option: --strip-all") * it doesn't work for ubuntu. bytecodealliance#2665
A little confused here, do we need a stripped release or a symbolized release? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@cngzhnp please be aware of it. |
@lum1n0us Thanks for the warning. However, I do not understand that why we need to revert it. If does not work with macOS which is not possible by the way, only we need to adapt CMakeFiles.txt for this, not to revert it. Someone needs to see symbols in the static library or shared library should generate binary in DEBUG mode, not release mode in my opinion. Could you please explain it to me more @yamt ? |
as far as tested, it produced unlinkable library on ubuntu. |
Do you have reproduce steps how to test that it was unlinkable library? |
have you looked at #2665 ? |
…ry size (bytecodealliance#2431)" (bytecodealliance#2669) This reverts commit 7e3a29d: * it doesn't work for macOS. ("unrecognized option: --strip-all") * it doesn't work for ubuntu. bytecodealliance#2665
This reverts commit 7e3a29d.
it doesn't work for macOS. ("unrecognized option: --strip-all")
it doesn't work for ubuntu. Problem building WAMR 1.2.3 #2665