Comment 15 for bug 1037607

Revision history for this message
Richard Hansen (rhansen) wrote :

> However I'd suggest that rather than patching dapper.py, it would be
> better to do modify the quantal.py subclass (if that's the first
> release which requires /proc to be mounted). Otherwise you're going
> to change the build behaviour for all previous releases too.

Thanks for the patch; it works like a charm.

Question: Wouldn't it be better to do this:

    self.run_in_target('umount', '/proc')

instead of this:

    run_cmd('umount', '%s/proc' % self.context.chroot_dir)

Then you wouldn't have to import VMBuilder.util.run_cmd. (I haven't
tried it; just an observation.)