On Tue, Apr 22, 2014 at 12:33 PM, Gene Czarcinski <gene@czarc.net> wrote:
+ first_time = 0

This should be "False".

for image in self.images: args = Arguments() args.update(["root=%s" % image.device.fstabSpec, "ro"]) @@ -2212,6 +2213,12 @@ class EXTLINUX(BootLoader): args.update(["ro"]) args.update(self.boot_args) log.info("bootloader.py: used boot args: %s ", args) + if first_time == 0:

if first_time:


+ first_time = 1

first_time = True

+ pre_stanza = ("default %(label)s (%(version)s)\n\n" + % {"label": self.image_label(image), + "version": image.version}) + config.write(pre_stanza)

But you just seem to be adding code; won't this conflict with the
existing code in write_config_header() which writes a default?

Note also this conflicts with my outstanding (but approved)
patch to make the default stanza optional, because OSTree
takes care of updating the bootloader configuration itself.