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

chore:[tag] new version 6.5.2 #120

Merged
merged 1 commit into from
Sep 27, 2024
Merged

Conversation

re2zero
Copy link
Contributor

@re2zero re2zero commented Sep 27, 2024

New version 6.5.2, and slimming down the linglong layer package.

Log: v6.5.2 release.

Copy link

TAG Bot

TAG: 6.5.2
EXISTED: no
DISTRIBUTION: unstable

New version 6.5.2, and slimming down the linglong layer package.

Log: v6.5.2 release.
@deepin-ci-robot
Copy link

deepin pr auto review

关键摘要:

  • debian/changelog文件中,版本号更新不一致,应保持与linglong.yaml中的版本号格式一致。
  • deploy_dep脚本中使用了awk命令,但没有考虑到 awk 命令可能需要使用不同的换行符,建议使用gawkmawk来确保兼容性。
  • deploy_dep脚本中的正则表达式[^/]*\.so可能无法正确匹配所有以.so结尾的动态库,建议使用更精确的匹配模式。
  • deploy_dep脚本中使用了sort -r来排序依赖库,但没有考虑到库的依赖关系顺序可能会变化,这可能会影响依赖关系的正确性。
  • deploy_dep脚本中使用了通配符${BASE_LIB_NAME%.*}来获取基本库名,但没有考虑到库名可能不包含点号,这可能会导致错误。
  • deploy_dep脚本中使用了echo '^'${PREFIX}'/'${TRIPLET}'/.+(?<!\.a)$'来排除静态链接库,但没有考虑到排除的库名可能不包含点号,这可能会导致错误。

是否建议立即修改:

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: rb-union, re2zero

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

@re2zero
Copy link
Contributor Author

re2zero commented Sep 27, 2024

/forcemerge

@deepin-bot
Copy link
Contributor

deepin-bot bot commented Sep 27, 2024

This pr force merged! (status: blocked)

@deepin-bot deepin-bot bot merged commit 90194df into linuxdeepin:master Sep 27, 2024
16 of 18 checks passed
@re2zero re2zero deleted the ll-v25 branch September 27, 2024 07:09
@Zeno-sole
Copy link

/topic cdapp-20240927

@deepin-ci-robot
Copy link

Add topic: cdapp-20240927 successed.

@Zeno-sole
Copy link

/integr-topic cdapp-20240927

@deepin-ci-robot
Copy link

Integrated with pr deepin-community/Repository-Integration#2053

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

4 participants