[Merge] ~enr0n/ubuntu-release-upgrader:drop-unused-quirks into ubuntu-release-upgrader:ubuntu/main
Julian Andres Klode
mp+467986 at code.launchpad.net
Tue Jun 25 10:24:05 UTC 2024
Review: Approve
Ship it (one comment about the pci helper).
Diff comments:
> diff --git a/DistUpgrade/DistUpgradeQuirks.py b/DistUpgrade/DistUpgradeQuirks.py
> index 166e64d..2e78137 100644
> --- a/DistUpgrade/DistUpgradeQuirks.py
> +++ b/DistUpgrade/DistUpgradeQuirks.py
> @@ -197,19 +187,6 @@ class DistUpgradeQuirks(object):
> self._install_linux_metapackage()
>
> # helpers
> - def _get_pci_ids(self):
I think this helper may be worthwhile keeping in case we need it again, but also we probably should just do it right then and walk sysfs, so idk
> - """ return a set of pci ids of the system (using lspci -n) """
> - lspci = set()
> - try:
> - p = subprocess.Popen(["lspci", "-n"], stdout=subprocess.PIPE,
> - universal_newlines=True)
> - except OSError:
> - return lspci
> - for line in p.communicate()[0].split("\n"):
> - if line:
> - lspci.add(line.split()[2])
> - return lspci
> -
> def _get_from_and_to_version(self):
> di = distro_info.UbuntuDistroInfo()
> try:
--
https://code.launchpad.net/~enr0n/ubuntu-release-upgrader/+git/ubuntu-release-upgrader/+merge/467986
Your team Ubuntu Core Development Team is subscribed to branch ubuntu-release-upgrader:ubuntu/main.
More information about the Ubuntu-reviews
mailing list