[Merge] lp:~semiosis/livecd-rootfs/fix-for-1565985 into lp:livecd-rootfs

Louis Zuckerman launchpad at louiszuckerman.com
Fri Jun 24 13:19:25 UTC 2016


Louis Zuckerman has proposed merging lp:~semiosis/livecd-rootfs/fix-for-1565985 into lp:livecd-rootfs.

Requested reviews:
  Ubuntu Core Development Team (ubuntu-core-dev)
Related bugs:
  Bug #1565985 in cloud-images: "vagrant vb ubuntu/xenial64 cannot mount synced folders"
  https://bugs.launchpad.net/cloud-images/+bug/1565985

For more details, see:
https://code.launchpad.net/~semiosis/livecd-rootfs/fix-for-1565985/+merge/298305

Several problems with the official ubuntu vagrant boxes for Xenial:

- Standard packages were missing (virtualbox-guest stuff, config management stuff)
- Box name was statically assigned, limiting to only one instance per host
- Default synced folder was disabled

This change fixes all of the above issues.

Also cleaned up some inconsistent indentation.

Last of all, had to change the shebang to not use -u.  This is because clean_loops (from functions) gets called twice and the second time (by trap) has unset variables so the build would die.

As a side note, a solution to allow clean_loops to run twice under bash -u would be to use ${var_name-} instead of ${var_name} in the two variable test lines, but that was out of scope for my issues.  Just FYI.

Thank you!
-- 
Your team Ubuntu Core Development Team is requested to review the proposed merge of lp:~semiosis/livecd-rootfs/fix-for-1565985 into lp:livecd-rootfs.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: review-diff.txt
Type: text/x-diff
Size: 4943 bytes
Desc: not available
URL: <https://lists.ubuntu.com/archives/ubuntu-reviews/attachments/20160624/692eac1a/attachment.diff>


More information about the Ubuntu-reviews mailing list