fix: build error on Windows#649
Conversation
![sourcery-ai[bot]](https://avatars.githubusercontent.com/in/48477?s=60&v=4){kind=small}
|
Hi @kt286. Thanks for your PR. I'm waiting for a linuxdeepin member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
|
TAG Bot New tag: 6.7.46 |
| set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -fPIC -Wextra") | ||
|
|
||
| # Export all symbols for MinGW since DTK classes lack export macros | ||
| if(WIN32 AND NOT MSVC) |
There was a problem hiding this comment.
这里目前应该不用处理吧，外部使用dtkdeclarative一般是通过qml的uri方式，目前不导出符号应该没问题，
66f9ee9 to
7ded0c7
Compare
|
/retest |
f982f20 to
fe2c812
Compare
fe2c812 to
dc19417
Compare
dc19417 to
4d096cb
Compare
Guard Linux-specific linker flags and platform assertions with appropriate #ifdefs so the project compiles on MinGW. Add DBus Qt dependency, export all symbols for MinGW, and add null checks in DSGBlendNode::render for OpenGL context/functions.
4d096cb to
0b29c1b
Compare
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: kt286 The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
1 similar comment
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: kt286 The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
use qt_add_resources for MSYS2 compatibility
6f631db to
e317e1f
Compare
4 participants
fix build error on Windows