Skip to content

feat(grub-gfx): 集成并优化 grub-gfx 模块的图形检测逻辑#1037

Draft
electricface wants to merge 1 commit intolinuxdeepin:masterfrom
electricface:swt/fix-grub-theme
Draft

feat(grub-gfx): 集成并优化 grub-gfx 模块的图形检测逻辑#1037
electricface wants to merge 1 commit intolinuxdeepin:masterfrom
electricface:swt/fix-grub-theme

Conversation

@electricface
Copy link
Member

  • 新增 grub-gfx 模块并加载到 session-daemon 中
  • 优化 grub2 中的 gfxmode 检测流程,增加完成检测的判断
  • 调整 grub_common 包中 deepin gfxmode 相关代码,修正路径和默认分辨率
  • 修正 gfxmode 检测失败时的处理逻辑,增强稳定性
  • 对 parseBootArgDeepinGfxmode 测试用例进行跳过处理,防止因 hack 影响测试
  • 重构部分 grub_common 代码逻辑,提升代码规范性和可读性

Bug: PMS-277733

Copy link

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry @electricface, you have reached your weekly rate limit of 500000 diff characters.

Please try again later or upgrade to continue using Sourcery

@deepin-ci-robot
Copy link

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: electricface

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

Details 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

- 新增 grub-gfx 模块并加载到 session-daemon 中
- 优化 grub2 中的 gfxmode 检测流程,增加完成检测的判断
- 调整 grub_common 包中 deepin gfxmode 相关代码,修正路径和默认分辨率
- 修正 gfxmode 检测失败时的处理逻辑,增强稳定性
- 对 parseBootArgDeepinGfxmode 测试用例进行跳过处理,防止因 hack 影响测试
- 重构部分 grub_common 代码逻辑,提升代码规范性和可读性

PMS: BUG-277733, BUG-291909
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.

2 participants