Skip to content
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

fix: fix the issue of incomplete search results #519

Merged
merged 1 commit into from
Dec 23, 2024

Conversation

wangrong1069
Copy link
Contributor

dmanHelper uses the directories in XDG_DATA_DIRS to create a search database. Before the fix, dmanHelper found very few application help. In order to search for help information for host applications and linglong applications, let's add the relevant paths.

Log: Adapt linglong
Bug: https://pms.uniontech.com/bug-view-290335.html

dmanHelper uses the directories in XDG_DATA_DIRS to create a search
database. Before the fix, dmanHelper found very few application help.
In order to search for help information for host applications and
linglong applications, let's add the relevant paths.

Log: Adapt linglong
Bug: https://pms.uniontech.com/bug-view-290335.html
@deepin-ci-robot
Copy link

deepin pr auto review

关键摘要:

  • 在新增代码中,对环境变量XDG_DATA_DIRS的修改可能会导致路径拼接错误,应该使用适当的路径分隔符。
  • 新增代码中直接修改环境变量可能不是线程安全的,建议使用线程局部存储或同步机制。

是否建议立即修改:

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: lzwind, wangrong1069

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

@wangrong1069
Copy link
Contributor Author

/merge

@deepin-bot deepin-bot bot merged commit 504d895 into linuxdeepin:master Dec 23, 2024
14 checks passed
@wangrong1069 wangrong1069 deleted the bug-1220 branch December 23, 2024 02:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants