Bug#880998: debian-goodies: checkrestart should error-out when failing to call lsof

2017-11-07 Thread Axel Beckert
Hi Nicolas, Nicolas Braud-Santoni wrote: > On Mon, Nov 06, 2017 at 10:12:11PM +0100, Axel Beckert wrote: > > Nicolas Braud-Santoni wrote: > > > PS: I included the patches as signed commit in the > > > collab-maint repository, in branch bug880998. > > > > Thanks! That makes it way easier as "g

Bug#880998: debian-goodies: checkrestart should error-out when failing to call lsof

2017-11-07 Thread Nicolas Braud-Santoni
On Mon, Nov 06, 2017 at 10:12:11PM +0100, Axel Beckert wrote: > > Hi Nicolas, > > Nicolas Braud-Santoni wrote: > > PS: I included the patches as signed commit in the > > collab-maint repository, in branch bug880998. > > Thanks! That makes it way easier as "git am" choked on your bug report >

Bug#880998: debian-goodies: checkrestart should error-out when failing to call lsof

2017-11-06 Thread Axel Beckert
Control: tag -1 + pending Hi Nicolas, Nicolas Braud-Santoni wrote: > PS: I included the patches as signed commit in the > collab-maint repository, in branch bug880998. Thanks! That makes it way easier as "git am" choked on your bug report mail with the two patches. Regards,

Bug#880998: debian-goodies: checkrestart should error-out when failing to call lsof

2017-11-06 Thread Nicolas Braud-Santoni
On Mon, Nov 06, 2017 at 09:47:08PM +0100, Nicolas Braud-Santoni wrote: > checkrestart should instead error-out. > Please find enclosed patches to that effect. PS: I included the patches as signed commit in the collab-maint repository, in branch bug880998. signature.asc Description: PGP sign

Bug#880998: debian-goodies: checkrestart should error-out when failing to call lsof

2017-11-06 Thread Nicolas Braud-Santoni
Package: debian-goodies Version: 0.69.1 Severity: important Tags: stretch buster patch -BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Dear maintainers, checkrestart fails without setting an error code when lsof is not found, and reports 0 processes needing to be restarted: $ sudo check