[Fedora-livecd-list] Patch review request for exception handling

Mads Kiilerich mads at kiilerich.com
Sun Jul 25 13:27:44 UTC 2010


  Bruno Wolff III wrote, On 07/25/2010 09:28 AM:
> On Sun, Jul 25, 2010 at 02:08:26 -0500,
>    Bruno Wolff III<bruno at wolff.to>  wrote:
>> Going down this road, I have something worked up that would need to be
>> tested before committing. The patch is attached.
> There is a error in the patch I used 'strerror' instead of 'errno' in one
> place. The corrected patch is attached.

Agreed, thanks, this looks like a very good patch and I have no 
objections. Handling OSError this way seems elegant and correct.

I haven't tested the patch.

/Mads



More information about the livecd mailing list