vmbuilder EC2: /etc/fstab missing trailing newline when built with --part

Bug #301499 reported by Eric Hammond
2
Affects Status Importance Assigned to Milestone
vm-builder (Ubuntu)
Triaged
Wishlist
Unassigned

Bug Description

When I use the --part option in vmbuilder --ec2 it builds an /etc/fstab without a trailing newline on the last record.

This is probably just a minor nuisance and shouldn't affect much in the way of functionality, but it also should be easy to fix.

I'm currently working off of this branch (with the fixes from ec2exec-fixes)

  https://code.launchpad.net/~nijaba/vmbuilder/ec2exec

I used the following file with --part:

cat >vmbuilder-ec2-32.part <<EOM
root 4096 a1
/mnt 0 a2
swap 1024 a3
/sys 0
EOM

Revision history for this message
Eric Hammond (esh) wrote :

Turns out this is already a problem and should be increased in priority.

When walking through the following tutorial,

  http://developer.amazonwebservices.com/connect/entry.jspa?categoryID=100&externalID=1663

the missing newline breaks a command like:

  echo "/dev/sdh /vol xfs noatime 0 0" >> /etc/fstab

I don't think anybody would be considered strange if they expected /etc/fstab to end in a newline.

Revision history for this message
Chuck Short (zulcss) wrote :

Relatively easy to fix. Thanks for the bugreport eric.

Regards
chuck

Chuck Short (zulcss)
Changed in vm-builder (Ubuntu):
status: New → Triaged
importance: Undecided → Wishlist
assignee: nobody → Chuck Short (zulcss)
Revision history for this message
Loïc Minier (lool) wrote :

Is this still an issue? I looked at the templates for fstab generation, and it wasn't immediately obvious to me that the output would miss a newline (the templates themselves already have a newline!)

Chuck Short (zulcss)
Changed in vm-builder (Ubuntu):
assignee: Chuck Short (zulcss) → nobody
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.