+  * New patches:
+    - d/p/0002-collect-refactor-_format_version.patch
+    - d/p/0003-pacemaker-Use-pep440-formatted-version-on-comparison.patch
+    - d/p/0004-cirrus-Run-tests-on-latest-daily-builds-for-ubuntu.patch
+    - d/p/0005-processor-handle-msr-module-loading-via-predicate.patch

Why, when this is an upstream release for which Canonical is the
upstream, are there local Debian patches included here?

+  * d/control:
+    - Add 'python3-packaging' as part of the runtime depends.
+    - Add 'python3-packaging' as part of the build depends.
+    - Add 'python3-yaml' as part of the build depends.

When making such packaging changes in an SRU, it should be documented in
the changelog why they are necessary and justified.  I can infer a
reason for the python3-packaging addition from the debdiff, but
shouldn't have to: the rationale should be documented so that there is
no misunderstanding.

And it is not at all obvious from the debdiff why python3-yaml is added
to the build-dependencies: it was already listed in setup.py before as a
requirement but apparently didn't cause a build failure.  So explanation
is needed here.

+    - Add 'X-Python3-Version: >= 3.6' to ensure we use the python
+      revision that is supported. (LP #2038648)

Well this is pointless, since python3 is >= 3.6 in all releases after
bionic and we don't do SRUs for bionic anymore.

+    - Add 'python3-boto3' to Suggests

This is also pretty pointless as nothing installs package Suggests: by
default and users almost never look at this.

** Changed in: sosreport (Ubuntu Noble)
       Status: In Progress => Incomplete

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/2054395

Title:
  [sru] sos upstream 4.7.0

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu-pro/+bug/2054395/+subscriptions


-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to