Case inconsistency in machine field definitions

Bug #1006905 reported by Robie Basak
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
flash-kernel (Debian)
Fix Released
Unknown
flash-kernel (Ubuntu)
New
Undecided
Dave Jones

Bug Description

Filed in Debian:

The README refers to Required-packages. However,
debian/flash-kernel-installer.postinst uses Required-Packages and
get_machine_field is case sensitive. This causes packages that
developers specify as required actually ending up being ignored.

Further, the existing database appears to have many entries listed with
this error.

But Bootloader-sets-root appears to be used with this capitalisation,
which is inconsistent.

Can I suggest a review of the capitalisation of these fields to make
them consistent across themselves, the documentation, the code and the
existing database?

At a minimum, the existing fields in the database should be fixed so
that debian/flash-kernel-installer.postinst can see them, and the README
should be fixed so that developers won't insert new entries into the
database in error.

Changed in flash-kernel (Debian):
status: Unknown → New
Changed in flash-kernel (Debian):
status: New → Fix Released
Revision history for this message
Dave Jones (waveform) wrote :

Required-packages vs Required-Packages appears to have been fixed some time ago; however we add Bootloader-sets-root (with its inconsistent capitalization) and it's the only field with this issue. I'll get this sorted out this cycle.

Changed in flash-kernel (Ubuntu):
assignee: nobody → Dave Jones (waveform)
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.