libparted2 package dependencies (dmidecode) differ between arm64 and amd64

[Philip Roche]

Hi,

While working on new arm64 mantic minimal cloud images (http://cloud-images.ubuntu.com/minimal/daily/mantic/) we discovered that `dmidecode` was not being installed in the arm64 images.

In debugging, I found that the package dependencies of the libparted2 package differ between arm64 and amd64. arm64 `libparted2` does not depend on `dmidecode` but on amd64 it does.

#### arm64
```
ubuntu@cloudimg:~$ apt show libparted2
Package: libparted2
Version: 3.6-3
Priority: standard
Section: libs
Source: parted
Origin: Ubuntu
Maintainer: Ubuntu Developers <email address hidden>
Original-Maintainer: Parted Maintainer Team <email address hidden>
Bugs: https://bugs.launchpad.net/ubuntu/+filebug
Installed-Size: 397 kB
Provides: libparted
Depends: libblkid1 (>= 2.17.2), libc6 (>= 2.34), libdevmapper1.02.1 (>= 2:1.02.97), libuuid1 (>= 2.16)
Suggests: parted, libparted-dev, libparted-i18n (= 3.6-3)
Homepage: https://www.gnu.org/software/parted
Task: standard, cloud-minimal
Download-Size: 150 kB
APT-Manual-Installed: no
APT-Sources: http://ports.ubuntu.com/ubuntu-ports mantic/main arm64 Packages
Description: disk partition manipulator - shared library
 GNU Parted is a program that allows you to create, destroy, resize,
 move, and copy disk partitions. This is useful for creating space
 for new operating systems, reorganizing disk usage, and copying data
 to new hard disks.
 .
 This package contains the shared library.

```

#### amd64
```
buntu@cloudimg:~$ apt show libparted2
Package: libparted2
Version: 3.6-3
Priority: standard
Section: libs
Source: parted
Origin: Ubuntu
Maintainer: Ubuntu Developers <email address hidden>
Original-Maintainer: Parted Maintainer Team <email address hidden>
Bugs: https://bugs.launchpad.net/ubuntu/+filebug
Installed-Size: 385 kB
Provides: libparted
Depends: libblkid1 (>= 2.17.2), libc6 (>= 2.34), libdevmapper1.02.1 (>= 2:1.02.97), libuuid1 (>= 2.16), dmidecode
Suggests: parted, libparted-dev, libparted-i18n (= 3.6-3)
Homepage: https://www.gnu.org/software/parted
Task: standard, cloud-minimal
Download-Size: 151 kB
APT-Manual-Installed: no
APT-Sources: http://archive.ubuntu.com/ubuntu mantic/main amd64 Packages
Description: disk partition manipulator - shared library
 GNU Parted is a program that allows you to create, destroy, resize,
 move, and copy disk partitions. This is useful for creating space
 for new operating systems, reorganizing disk usage, and copying data
 to new hard disks.
 .
 This package contains the shared library.

```

Is this a bug, or is there reasoning why this is the case?

I note the change to dependencies `, dmidecode [amd64 i386]` was added as part of changes in version 3.2-21 in Apr 2018 to address bug https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=840709

Should I file this bug upstream @ https://packages.debian.org/bookworm/parted ?

[Dimitri John Ledkov (xnox)]

Yes, please file it upstream:

```
#if (defined(__i386__) || defined(__x86_64__)) && defined(__linux__)
# define USE_DMI
#endif

#define APPLE_DMI "Apple Computer, Inc."
#define APPLE_DMI_2 "Apple Inc."
static int is_apple = 0;

static char *
dmi_system_manufacturer (void)
{
#ifdef USE_DMI
  FILE *dmidecode;
  char *manufacturer = NULL;
  size_t manufacturer_len = 0;

  dmidecode = popen ("dmidecode -s system-manufacturer 2>/dev/null", "r");
  if (getline (&manufacturer, &manufacturer_len, dmidecode) < 0) {
    /* ignore; will return NULL */
  }
  pclose (dmidecode);
  if (manufacturer) {
    char *newline = strchr (manufacturer, '\n');
    if (newline)
      *newline = '\0';
  }
  return manufacturer;
#else /* !USE_DMI */
  return NULL;
#endif /* USE_DMI */
}

```

As indeed this should be improved / ported to do arm64 too. To have arm64 ifdef, and depend on dmidecode on arm too.

[Philip Roche (philroche)]

Now filed upstream too @ https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1052462

[Philip Roche (philroche)]

As per the response on upstream bug @ https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1052462#10 this can be marked as invalid as it is by design that the dependencies differ.

> I don't personally see why this needs to be extended
to arm64, although of course if somebody comes forward and explicitly
confirms that it's relevant to partitioning on Apple Silicon or whatever
then we could do that - but I wouldn't do it just for architectural
symmetry.