Skip to content
This repository has been archived by the owner on Oct 11, 2024. It is now read-only.

build: add app KKEditQT #2155

Closed
wants to merge 1 commit into from
Closed

Conversation

Van020530
Copy link
Contributor

KKEditQT Source Code Editor.

log: add app

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Van020530

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

KKEditQT Source Code Editor.

log: add app
@deepin-ci-robot
Copy link

deepin pr auto review

关键摘要:

  • buildplugs.sh脚本中,使用了${PREFIX}来构建路径,但未验证PREFIX变量是否正确设置。
  • qmake命令的参数中使用了PREFIX=${PREFIX},但应确保PREFIX环境变量已正确设置。
  • make命令使用了${JOBS},但未验证JOBS环境变量是否已正确设置。

是否建议立即修改:

  • 是,需要确保PREFIXJOBS环境变量被正确设置,并且路径拼接的使用是安全的。同时,建议增加错误检查和日志记录,以便在出现问题时能够追踪和调试。

@kamiyadm
Copy link
Contributor

无法启动

@kamiyadm kamiyadm closed this May 29, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants