chore(debian): adjust dde-services version requirement - #1203
Conversation
1. Change dde-services minimum version in debian/control from `>= 1.0.40` to `>> 1.0.39`. 2. `>> 1.0.39` is the standard Debian syntax for strictly-newer-than, covering 1.0.39.x and later. Log: Relax dde-services version constraint to standard Debian strict-greater-than syntax. Influence: Packaging only; no runtime behavior change. chore(debian): 调整 dde-services 版本依赖 1. 将 debian/control 中 dde-services 的最低版本约束由 `>= 1.0.40` 改为 `>> 1.0.39`。 2. `>> 1.0.39` 是 Debian 标准的严格大于语法,涵盖 1.0.39.x 及之后版本。 Log: 将 dde-services 版本约束调整为 Debian 标准严格大于语法。 Influence: 仅打包变更,不影响运行时行为。
|
Hi @MyLeeJiEun. Thanks for your PR. I'm waiting for a linuxdeepin member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Reviewer's guide (collapsed on small PRs)Reviewer's GuideAdjusts the dde-services dependency in Debian packaging to use Debian’s strict-greater-than syntax, relaxing the minimum version constraint while keeping runtime behavior unchanged. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Hey - I've left some high level feedback:
- Confirm that all dde-services version constraints across debian packaging files (e.g., control, *.install, any transitional packages) are consistently updated to the new
>> 1.0.39requirement if applicable. - If this change is intended to allow 1.0.39.x but exclude 1.0.39 itself, consider clarifying that intent in the commit message or packaging metadata to avoid confusion for future maintainers.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- Confirm that all dde-services version constraints across debian packaging files (e.g., control, *.install, any transitional packages) are consistently updated to the new `>> 1.0.39` requirement if applicable.
- If this change is intended to allow 1.0.39.x but exclude 1.0.39 itself, consider clarifying that intent in the commit message or packaging metadata to avoid confusion for future maintainers.Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: mhduiy, MyLeeJiEun The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
>= 1.0.40to>> 1.0.39.>> 1.0.39is the standard Debian syntax for strictly-newer-than, covering 1.0.39.x and later.Log: Relax dde-services version constraint to standard Debian strict-greater-than syntax.
Influence: Packaging only; no runtime behavior change.
chore(debian): 调整 dde-services 版本依赖
>= 1.0.40改为>> 1.0.39。>> 1.0.39是 Debian 标准的严格大于语法,涵盖 1.0.39.x 及之后版本。Log: 将 dde-services 版本约束调整为 Debian 标准严格大于语法。
Influence: 仅打包变更,不影响运行时行为。
Summary by Sourcery
Build: