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

[Deepin-Kernel-SIG] [linux-6.6.y] printk: show build commitid in the front of dmesg #381

Merged
merged 2 commits into from
Aug 20, 2024

Conversation

Avenger-285714
Copy link
Collaborator

This can help us find source code version when troubleshooting.

Banner in dmesg can be flushed if too many debug log.
We save it to deb package so even it's not installed
we can still tell which source code it is from.

To check commitid:
If installed, use apt changelog <package-name>. If not,
extract changelog.Debian.gz from package and zcat it.

Signed-off-by: wenlunpeng <[email protected]>
Signed-off-by: WangYuli <[email protected]>
This can help us find source code version when troubleshooting.

Signed-off-by: wenlunpeng <[email protected]>
Signed-off-by: WangYuli <[email protected]>
@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please ask for approval from avenger-285714. For more information see the Code Review Process.

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

@deepin-ci-robot
Copy link

deepin pr auto review

关键摘要:

  • linux_commitid_banner数组中,硬编码了Git提交ID,这可能导致在提交ID更改时需要手动更新。
  • GIT_COMMITID宏的定义使用了git log命令,这可能会导致在构建系统时出现问题,特别是当Git命令无法执行或者没有足够的权限时。
  • mkdebian脚本中的Git信息生成方式可能会导致在非Git构建系统中运行时出现问题。

是否建议立即修改:

@opsiff opsiff changed the title printk: show build commitid in the front of dmesg [Deepin-Kernel-SIG] [linux-6.6.y] printk: show build commitid in the front of dmesg Aug 20, 2024
@Avenger-285714 Avenger-285714 merged commit 68fa000 into deepin-community:linux-6.6.y Aug 20, 2024
4 of 6 checks passed
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