-
Notifications
You must be signed in to change notification settings - Fork 41
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
build cilacc.dll as release #1928
Conversation
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.
sad... Ideally adding windows debug libs to conda would be better.
As far as I know they are not included the redistributable folder and therefore license so I'm not sure we could. I don't see what we lose with this. Anyone needing to debug should be able to build whichever they want locally. |
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.
The CMAKE_BUILD_TYPE
has no effect with visual compilers such as VS. At compile time you need to pass the --config RelWithDebInfo
. CMake Docs read that CMAKE_BUILD_TYPE
Specifies the build type on single-configuration generators (e.g. Makefile Generators or Ninja).
If I'm not mistaken the default configuration of VS is Debug
not RelWithDebInfo
.
Hmm I'm mostly sure I looked at the dependencies with |
@paskino you are correct, Building:
Path of loaded dll:
Dependencies of loaded dll:
|
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.
yay
Changes
Added:
--config RelWithDebInfo
specified the generator and toolkit in the windows build script to keep consistency between builders.
OUTDATED
Need to build the shared library as 'Release'. Previous build 'RelWithDebug' requires debug dlls which are not included in the VC redistributable installed in the conda environment.
Built as 'RelWithDebug':
Testing you performed
Related issues/links
Needed to address for #1880
Checklist
Contribution Notes
Please read and adhere to the developer guide and local patterns and conventions.