[Bug 2063887] Re: debsums warning after upgrade to Ubuntu 24.04 (noble): changed file /etc/X11/xorg.conf.d/99-v3d.conf (from ubuntu-raspi-settings-desktop package)

[Dave Jones]

** Tags added: foundations-todo

-- 
You received this bug notification because you are a member of Ubuntu
Foundations Bugs, which is subscribed to ubuntu-raspi-settings in
Ubuntu.
https://bugs.launchpad.net/bugs/2063887

Title:
  debsums warning after upgrade to Ubuntu 24.04 (noble): changed file
  /etc/X11/xorg.conf.d/99-v3d.conf (from ubuntu-raspi-settings-desktop
  package)

Status in ubuntu-raspi-settings package in Ubuntu:
  Triaged

Bug description:
  System: Raspberry Pi 4 Model B Rev 1.4

  After upgrade to Ubuntu 24.04 (noble) I'm getting this debsums
  warning:

  # debsums -sa ubuntu-raspi-settings-desktop
  debsums: changed file /etc/X11/xorg.conf.d/99-v3d.conf (from ubuntu-raspi-settings-desktop package)

  The difference between the file installed and the file from the
  ubuntu-raspi-settings-desktop package is just the indentation:

  - The file installed is indented with two spaces:
  Section "OutputClass"
    Identifier "vc4"
    MatchDriver "vc4"
    Driver "modesetting"
    Option "PrimaryGPU" "true"
  EndSection

  - The file from the package is indented with one tabulator:
  Section "OutputClass"
        Identifier "vc4"
        MatchDriver "vc4"
        Driver "modesetting"
        Option "PrimaryGPU" "true"
  EndSection

  If I replace the file installed with the one from the package in order
  to avoid the debsums warning, the file is auto-replaced again during
  boot by the systemd update-v3d-config.service.

  This service runs the script /usr/bin/update-v3d-config; when the
  system is a Raspberry Pi 5 this script replaces "Identify" and
  "MatchDriver" but for the rest of Raspberry Pi models it should leave
  the file as originally was. The problem is that the script always
  replaces the file with two spaces indentation instead of one tabulator
  causing the debsums mismatch:

  ...
      sed -e "s/IDENTIFIER/${IDENTIFIER}/; s/DRIVER/${DRIVER}/" << EOF > "$TMPFILE"
  Section "OutputClass"
    Identifier "IDENTIFIER"
    MatchDriver "DRIVER"
    Driver "modesetting"
    Option "PrimaryGPU" "true"
  EndSection
  EOF
      mv "$TMPFILE" "/etc/X11/xorg.conf.d/99-v3d.conf"
  ...

  I understand that this fix is done in every boot because we (users)
  should use the same OS installed media  with different Raspberry Pi
  models.

  Well... a possible solution is to change the package file to have two
  spaces of indentation instead of a tabulator, or to change the script
  to use a tabulator instead of two spaces. The objective is having both
  files identical to avoid the debsums warning... But unfortunately this
  "fix" only avoids the debsums warning for the systems that are not a
  Raspberry Pi 5 because in that case the warning is unavoidable as the
  result installed file will be always different from the one of the
  package. So this isn't a perfect solution.

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/ubuntu-raspi-settings/+bug/2063887/+subscriptions