mock patch proposal: unmount everything mounted under chroot
Mike McLean
mikem at redhat.com
Tue Mar 16 19:33:59 UTC 2010
On 03/16/2010 02:50 PM, Clark Williams wrote:
>>> + except mock.exception.Error, e:
>>> + self.root_log.warning("'%s' failed." % cmd)
>>
>> It might be better to log the exception string rather than this less
>> informative warning.
>>
>
> Agreed. Maybe:
> self.root_log.warning("'%s': %s" % (cmd, e))
All the exception strings from do() already include the command text. I
was thinking simply:
self.root_log.warning(str(e))
More information about the buildsys
mailing list