-
Notifications
You must be signed in to change notification settings - Fork 177
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
add desktop plugin definitions and improve plugin loading #2542
Conversation
- Added required dependencies in pkg-config file: * Added Qt6, DTK6 and DFM6 components to Requires field - Enhanced CMake configuration: * Added CMakeFindDependencyMacro for downstream projects * Added proper dependency declarations in Config.cmake * Updated target installation to use modern CMake approach * Improved include directory handling with generator expressions - Re-enabled video preview plugin build - Fixed Qt6 compatibility issues: * Updated palette color role (Foreground -> WindowText) * Replaced deprecated QDesktopWidget with QScreen API Log:
- Added desktop plugin definitions in dfm_plugin_defines.h: * Added Core::kDesktop for core desktop plugins * Added Edge::kDesktop for edge desktop plugins * Added desktopCorePlugins() and desktopAllPlugins() utility functions - Improved plugin loading mechanism: * Replaced static lazy load list with dynamic filter function * Added plugin filter based on desktop core plugins list * Removed hardcoded plugin names from lazy loading list - Updated package dependencies: * Removed qt5dxcb-plugin and dcc-wallpapersetting-plugin from Recommends Log:
Warning
|
deepin pr auto review代码审查意见:
以上是针对代码审查意见的总结,希望能够对您有所帮助。 |
Note
详情{
"debian/control": [
{
"line": "Homepage: http://www.deepin.org",
"line_number": 69,
"rule": "S35",
"reason": "Url link | 6fe814dfb7"
}
]
} |
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: Johnson-zs, lzwind, max-lvs The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/forcemerge |
This pr force merged! (status: blocked) |
refactor: add desktop plugin definitions and improve plugin loading
refactor: improve cmake config for dfm6-base