[Merge] lp:~codyshepherd/livecd-rootfs/snaps-manifest into lp:livecd-rootfs

Tobias Koch tobias.koch at canonical.com
Fri Nov 9 07:02:22 UTC 2018


I don't want to be overly nit-picky, but I find this slightly unintuitive. If you don't know what it does, you will first suspect that filesystem.packages is an input file. Personally, I would prefer, if this behaved more like any other *nix CLI tool, for example

    snap-seed-parse --root="chroot/" -o "binary/${INITFS}/filesystem.packages"

and the default could be to print to stdout, if "-o" is not specified.


-- 
https://code.launchpad.net/~codyshepherd/livecd-rootfs/snaps-manifest/+merge/358530
Your team Ubuntu Core Development Team is requested to review the proposed merge of lp:~codyshepherd/livecd-rootfs/snaps-manifest into lp:livecd-rootfs.



More information about the Ubuntu-reviews mailing list