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