build: add WineWizard - Githubissues

martyr-deepin / linglong-hub

Deprecated. Moved to https://github.com/linglongdev

https://github.com/linglongdev

GNU Lesser General Public License v3.0

41 stars 45 forks source link

build: add WineWizard #2223

Closed wjyrich closed 4 months ago

wjyrich commented 4 months ago

Modern Wine GUI

Log: add software name--WineWizard

deepin-ci-robot commented 4 months ago

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: wjyrich

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files: - **[debian/deepin/OWNERS](https://github.com/linuxdeepin/linglong-hub/blob/master/debian/deepin/OWNERS)** Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment

deepin-ci-robot commented 4 months ago

deepin pr auto review

关键摘要:

检查是否有必要包含过多的头文件，可能会影响代码的编译性能。
确认新增的 <QStyle> 头文件在代码中的作用，以及是否已经处理了相关的问题（例如 Qt 5.13 中引入的Metal 渲染支持）。
验证 WaitForm 类的构造函数中是否有必要指定 parent 参数，以及是否有其他代码依赖于这个改变。

是否建议立即修改:

是，特别是关于性能和潜在的兼容性问题，应该立即进行评估和优化。

kamiyadm commented 4 months ago

不太能用，这只是个前端界面，需要配合wine模拟器才能工作