Skip to content
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

Added Pragmas for the Debug Libs from Openssl #4408

Closed
wants to merge 5 commits into from

Conversation

KevDi
Copy link

@KevDi KevDi commented Aug 25, 2023

If Openssl is build from sources, the Debug Version of the Library ends with an d. Added an additional check to set the Pragma to the Correct Library Files.

If Openssl is build from sources, the Debug Version of the Library ends with an `d`.
Added an additional check to set the Pragma to the Correct Library Files.
@cla-assistant
Copy link

cla-assistant bot commented Aug 25, 2023

CLA assistant check
All committers have signed the CLA.

@KevDi
Copy link
Author

KevDi commented Aug 25, 2023

I'm not sure if this is the best way to solve this, or if this breaks some of the Packaging Tools (VCPKG) because i was only able to test it with conan and there it works with that fix. So i'm open to any recommendations on this.

@KevDi
Copy link
Author

KevDi commented Nov 10, 2023

@emasab any news on that? I also added a PR on the Conan Side to fix this directly in Conan. (Conan PR) the question is what would be the best solution here?

@KevDi
Copy link
Author

KevDi commented Jan 31, 2024

@emasab are there any news on that?

@KevDi
Copy link
Author

KevDi commented Feb 23, 2024

PR Could be closed Issue was solved on Conan Side by changing the Pragmas

@KevDi KevDi closed this Feb 23, 2024
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.

1 participant