-
Notifications
You must be signed in to change notification settings - Fork 6.4k
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
wxwidgets installed but not detected by FindwxWidgets.cmake #4756
Comments
I created a slightly modified It works when modifying I tried modifying the this is what I tried to add in the https://github.com/Microsoft/vcpkg/blob/master/ports/wxwidgets/portfile.cmake : |
Same issue here. |
No, I ended up putting The whole wxwidgets build system is a mess. |
The same issue, @markusobi can you please share the modified vcpkg aware version of FindwxWidgets.cmake ? |
My solution: |
@zelid |
It seems that I have the same issue. |
I'm having this issue too. I'm making a cross-platform project and I'd like to be able to use cmake on all platforms (Linux, Windows, Mac). Has anyone found a solution to this problem? |
I just stumbled into this too :-( |
Any update on this issue ? vcpkg is really useful but if we can't use it with wxWidgets there is no point in using it in our projects. |
Could you share with us the modified FindwxWidgets.cmake ? |
@jmkerloch I am using this version, slightly modified from what the original author did, see the revisions on the gist. https://gist.github.com/nickoe/d3c224a2587eff8ea959bc383a993520/ I am not exactly sure why this special findwxwidgets cmake file is needed, it would be nice if a fix could be upstreamed in cmake or vcpkg. |
@nickoe I just tried your file and now I can find wxWidgets from VCPKG. Thanks ! |
I tried that but I got lots of linker errors: https://pastebin.com/xCGEJLLc |
@SeanOMik How did you link to wxWidgets in your CMakeList.txt ? |
This is how I find it: find_package(wxWidgets 3.1 COMPONENTS core base REQUIRED) And link: target_link_libraries(discord_client PRIVATE ${wxWidgets_LIBRARY}) |
I've updated the patch @nickoe did for the latest version of FindwxWidgets.cmake (CMake 3.17.2). https://gist.github.com/brad-anderson/876a8c581abf285dd20b3c532bf48bd2 |
@brad-anderson how do you integrate your FindwxWidgets.cmake on your CMakeList.txt ? The file is working for a release version in Windows but for debug version the path to the .lib are not correct For example here is the lib include for net module Does anyone have an idea how to get the correct path ? |
For the debug version I wasn't selecting the correct version in Visual Studio... |
Hi, I also have a similar issue. Is there any progress on this issue? |
I will confirm and fix this issue soon. |
can confirm that this is still broken at this point in time. |
Also broken for me. |
Any temporary fix for this please? Running Windows |
Using this patch in CMAKE https://gist.github.com/brad-anderson/876a8c581abf285dd20b3c532bf48bd2 I am able to find wxWidgets but get loads of linking errors on Windows |
Did anyone find out how to do it?, or a solution will be merged to master? |
PR #13361 would address this issue. |
* [wxwidgets] Fix #4756 * [wxwidgets] Avoid replacing FindwxWidgets from CMake * [wxwidgets] Provide usage information * [gppanel] Use built-in CMake module * [wxchartdir] Use built-in CMake module * [wxwidgets] Fixup for applocal-ing * [wxwidgets] Fix osx builds * Update port_versions * [gppanel][wxchartdir][wxwidgets] Change versioning schemas Co-authored-by: Robert Schumacher <[email protected]> Co-authored-by: NancyLi1013 <[email protected]>
This issue seems closed as of today, but I can't still build a basic wxWidgets project with CMake under Linux. Just for the record, the same CMake project builds ok when using the wxWidgets libraries from the Linux distro (Ubuntu 20.04) instead of vcpkg. The only catch is that this line is required in include(${wxWidgets_USE_FILE}) This is not indicated in the message provided at the end of the installation of wxWidgets in vcpkg, which reads: > ./vcpkg install wxwidgets
...
The package wxwidgets provides CMake integration:
find_package(wxWidgets REQUIRED)
target_include_directories(main PRIVATE ${wxWidgets_INCLUDE_DIRS})
target_link_libraries(main PRIVATE ${wxWidgets_LIBRARIES}) |
A note for those who are stuck: make sure your CMake build matches the |
This goes without saying, unrelated to wxWidgets. There have been significant improvements in the past months, including basic CI testing of |
Running into this same issue. CMake isn't able to locate a
I'm building a small executable utility so I don't know how I can match the |
@EvanBalster Please open a new new issue, following the current issue templates. There is just too much change in wxwidgets between 2018 and 2022. Again: CMake usage (from a user perspective) is tested in vcpkg CI. |
The present of September 2022
|
## 1.11.0 (2024-01-11) ### Features Added - Added 'OPTIONS' HTTP method to `Azure::Core::Http::HttpMethod` enum. - Added TLS 1.3 support to WinHTTP transport. - Environment Log Level Listener now logs the ThreadID for the thread originating the trace. - [[microsoft#4983]](Azure/azure-sdk-for-cpp#4983) Added support for setting `CURLOPT_CAPATH` libcurl option on Linux. (A community contribution, courtesy of _[phoebusm](https://github.com/phoebusm)_) ### Bugs Fixed - [[microsoft#5172]](Azure/azure-sdk-for-cpp#5172) `Azure::Nullable::Emplace()` does not set `HasValue()` to `true`. - [[microsoft#5130]](Azure/azure-sdk-for-cpp#5130) `Url::AppendPath()` and `Url::SetPath()` may end up with a double slash at the beginning of a path. - [[microsoft#5007]](Azure/azure-sdk-for-cpp#5007) Some versions of GCC no longer include stdint.h in cstdint. ### Other Changes - [[microsoft#4756]] (Azure/azure-sdk-for-cpp#4756) `BearerTokenAuthenticationPolicy` now uses shared mutex lock for read operations. ### Acknowledgments Thank you to our developer community members who helped to make Azure Core better with their contributions to this release: - Phoebus Mak _([GitHub](https://github.com/phoebusm))_
* [azure-messaging-eventhubs-cpp] Update to 1.0.0-beta.5 ## 1.0.0-beta.5 (2024-01-11) ### Breaking Changes - EventHub `ConsumerClient` and `ProcessorClient` objects now return pointers to `EventData` objects instead of `EventData` objects by value. * [azure-core-amqp-cpp] Update to 1.0.0-beta.6 ## 1.0.0-beta.6 (2024-01-11) ### Features Added - AMQP Value reference counts are now atomic, this fixes several AMQP related crashes. ### Breaking Changes - `MessageReceiver` returns a pointer to the received message instead of a copy. ### Bugs Fixed - Fixed several memory leaks. - AMQP Link Credits now work as expected. - Integrated the fix for NVD - CVE-2024-21646. * [azure-core-cpp] Update to 1.11.0 ## 1.11.0 (2024-01-11) ### Features Added - Added 'OPTIONS' HTTP method to `Azure::Core::Http::HttpMethod` enum. - Added TLS 1.3 support to WinHTTP transport. - Environment Log Level Listener now logs the ThreadID for the thread originating the trace. - [[#4983]](Azure/azure-sdk-for-cpp#4983) Added support for setting `CURLOPT_CAPATH` libcurl option on Linux. (A community contribution, courtesy of _[phoebusm](https://github.com/phoebusm)_) ### Bugs Fixed - [[#5172]](Azure/azure-sdk-for-cpp#5172) `Azure::Nullable::Emplace()` does not set `HasValue()` to `true`. - [[#5130]](Azure/azure-sdk-for-cpp#5130) `Url::AppendPath()` and `Url::SetPath()` may end up with a double slash at the beginning of a path. - [[#5007]](Azure/azure-sdk-for-cpp#5007) Some versions of GCC no longer include stdint.h in cstdint. ### Other Changes - [[#4756]] (Azure/azure-sdk-for-cpp#4756) `BearerTokenAuthenticationPolicy` now uses shared mutex lock for read operations. ### Acknowledgments Thank you to our developer community members who helped to make Azure Core better with their contributions to this release: - Phoebus Mak _([GitHub](https://github.com/phoebusm))_
* [azure-messaging-eventhubs-cpp] Update to 1.0.0-beta.5 ## 1.0.0-beta.5 (2024-01-11) ### Breaking Changes - EventHub `ConsumerClient` and `ProcessorClient` objects now return pointers to `EventData` objects instead of `EventData` objects by value. * [azure-core-amqp-cpp] Update to 1.0.0-beta.6 ## 1.0.0-beta.6 (2024-01-11) ### Features Added - AMQP Value reference counts are now atomic, this fixes several AMQP related crashes. ### Breaking Changes - `MessageReceiver` returns a pointer to the received message instead of a copy. ### Bugs Fixed - Fixed several memory leaks. - AMQP Link Credits now work as expected. - Integrated the fix for NVD - CVE-2024-21646. * [azure-core-cpp] Update to 1.11.0 ## 1.11.0 (2024-01-11) ### Features Added - Added 'OPTIONS' HTTP method to `Azure::Core::Http::HttpMethod` enum. - Added TLS 1.3 support to WinHTTP transport. - Environment Log Level Listener now logs the ThreadID for the thread originating the trace. - [[microsoft#4983]](Azure/azure-sdk-for-cpp#4983) Added support for setting `CURLOPT_CAPATH` libcurl option on Linux. (A community contribution, courtesy of _[phoebusm](https://github.com/phoebusm)_) ### Bugs Fixed - [[microsoft#5172]](Azure/azure-sdk-for-cpp#5172) `Azure::Nullable::Emplace()` does not set `HasValue()` to `true`. - [[microsoft#5130]](Azure/azure-sdk-for-cpp#5130) `Url::AppendPath()` and `Url::SetPath()` may end up with a double slash at the beginning of a path. - [[microsoft#5007]](Azure/azure-sdk-for-cpp#5007) Some versions of GCC no longer include stdint.h in cstdint. ### Other Changes - [[microsoft#4756]] (Azure/azure-sdk-for-cpp#4756) `BearerTokenAuthenticationPolicy` now uses shared mutex lock for read operations. ### Acknowledgments Thank you to our developer community members who helped to make Azure Core better with their contributions to this release: - Phoebus Mak _([GitHub](https://github.com/phoebusm))_
* [azure-messaging-eventhubs-cpp] Update to 1.0.0-beta.5 ## 1.0.0-beta.5 (2024-01-11) ### Breaking Changes - EventHub `ConsumerClient` and `ProcessorClient` objects now return pointers to `EventData` objects instead of `EventData` objects by value. * [azure-core-amqp-cpp] Update to 1.0.0-beta.6 ## 1.0.0-beta.6 (2024-01-11) ### Features Added - AMQP Value reference counts are now atomic, this fixes several AMQP related crashes. ### Breaking Changes - `MessageReceiver` returns a pointer to the received message instead of a copy. ### Bugs Fixed - Fixed several memory leaks. - AMQP Link Credits now work as expected. - Integrated the fix for NVD - CVE-2024-21646. * [azure-core-cpp] Update to 1.11.0 ## 1.11.0 (2024-01-11) ### Features Added - Added 'OPTIONS' HTTP method to `Azure::Core::Http::HttpMethod` enum. - Added TLS 1.3 support to WinHTTP transport. - Environment Log Level Listener now logs the ThreadID for the thread originating the trace. - [[microsoft#4983]](Azure/azure-sdk-for-cpp#4983) Added support for setting `CURLOPT_CAPATH` libcurl option on Linux. (A community contribution, courtesy of _[phoebusm](https://github.com/phoebusm)_) ### Bugs Fixed - [[microsoft#5172]](Azure/azure-sdk-for-cpp#5172) `Azure::Nullable::Emplace()` does not set `HasValue()` to `true`. - [[microsoft#5130]](Azure/azure-sdk-for-cpp#5130) `Url::AppendPath()` and `Url::SetPath()` may end up with a double slash at the beginning of a path. - [[microsoft#5007]](Azure/azure-sdk-for-cpp#5007) Some versions of GCC no longer include stdint.h in cstdint. ### Other Changes - [[microsoft#4756]] (Azure/azure-sdk-for-cpp#4756) `BearerTokenAuthenticationPolicy` now uses shared mutex lock for read operations. ### Acknowledgments Thank you to our developer community members who helped to make Azure Core better with their contributions to this release: - Phoebus Mak _([GitHub](https://github.com/phoebusm))_
wxWidgets installed with simple
vcpkg install wxwidgets:x64-windows-static
without problems.Simple minimal CMake project
generates error:
because libraries are installed in non-standard directory (${WX_ROOT_DIR}/lib in place of ${WX_ROOT_DIR}/lib/vc141...). See
https://github.com/Kitware/CMake/blob/be9ad8279a29893943b342cc5ffdbf9868fb1b4c/Modules/FindwxWidgets.cmake#L544
The text was updated successfully, but these errors were encountered: