Code review comment for lp:~persia/casper/trunk

Revision history for this message
Emmet Hikory (persia) wrote :

> I'm a bit concerned that syntactic errors mean that this hasn't been tested -
> perhaps you could build a test image with this and run it through before we merge,
> to make sure that it really does the right thing?

Right. No more attempts to go faster, as it seems to both take longer, and waste other people's time as well.

Test procedure was as follows:

update files
copy changes to loop-mounted image
regenerate initramfs
extract to /casper/initrd.gz on image
boot result
repeat

    I didn't go so far as to rebuild casper, and install the rebuilt casper into the image prior to update.

« Back to merge proposal