On Thu, May 24, 2012 at 12:39:37AM +0200, Tomas Pospisek wrote:
> So currently instead of failing as above I changed checkrestart to
> report an error and to continue:

That patch looks good to me.

> admin:~# diff -u /tmp/checkrestart /usr/local/bin/checkrestart
> --- /tmp/checkrestart   2012-05-24 00:37:15.147100799 +0200
> +++ /usr/local/bin/checkrestart 2012-05-24 00:24:23.000000000 +0200
> @@ -445,7 +445,8 @@
>              self.program = os.readlink('/proc/%d/exe' % self.pid)
>          except OSError, e:
>              if e.errno != errno.ENOENT:
> -                raise
> +                sys.stderr.write('ERROR: Failed to read %d' % self.pid)
> +                #raise
>          self.program = self.cleanFile(self.program)
> 
>      def cleanFile(self, f):
> 
> I wonder if there's a better way...

I think your fix is probably good enough.

> Other than that, I can report that Tollef patch works for me.

That's good to hear, thank you.

Regards

Javier

Attachment: signature.asc
Description: Digital signature

Reply via email to