Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.debian.bugs.dist > #814988

Bug#858555: chiark-utils-bin: [patch] add timeout option to with-lock-ex

From Matthew Vernon <matthew@debian.org>
Newsgroups linux.debian.bugs.dist
Subject Bug#858555: chiark-utils-bin: [patch] add timeout option to with-lock-ex
Date 2017-03-23 18:30 +0100
Message-ID <toeIG-5py-13@gated-at.bofh.it> (permalink)
References <toa2m-28N-23@gated-at.bofh.it> <tocGS-3Wl-9@gated-at.bofh.it> <tocGS-3Wl-7@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw

[Multipart message — attachments visible in raw view] - view raw

Hi,

Revised patch attached. My commit message:

Use getopt for CLI parsing; simplify SIGALRM handling

Using getopt for option parsing means that the ordering of -t and -f / 
-q doesn't matter, and we can put more error-handling of user input into 
the same place.

We now assume:
i) SIGALRM hasn't come from some other source
ii) SIGALRM was set to SIG_DFL on entry, and will be reset thus by execvp

...and this means we can simplify the code somewhat (no need to track 
elapsed time or store the old sigaction).

I'll point you at a git branch out-of-band.

Regards,

Matthew

Back to linux.debian.bugs.dist | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

Bug#858555: chiark-utils-bin: [patch] add timeout option to with-lock-ex Matthew Vernon <matthew@debian.org> - 2017-03-23 13:30 +0100
  Bug#858555: chiark-utils-bin: [patch] add timeout option to with-lock-ex Ian Jackson <ijackson@chiark.greenend.org.uk> - 2017-03-23 16:20 +0100
    Bug#858555: chiark-utils-bin: [patch] add timeout option to with-lock-ex Ian Jackson <ijackson@chiark.greenend.org.uk> - 2017-03-23 16:20 +0100
      Bug#858555: chiark-utils-bin: [patch] add timeout option to with-lock-ex Matthew Vernon <matthew@debian.org> - 2017-03-23 18:30 +0100
        Bug#858555: chiark-utils-bin: [patch] add timeout option to with-lock-ex Ian Jackson <ijackson@chiark.greenend.org.uk> - 2017-03-23 19:40 +0100
          Bug#858555: chiark-utils-bin: [patch] add timeout option to with-lock-ex Matthew Vernon <matthewv@chiark.greenend.org.uk> - 2017-03-24 14:40 +0100
    Bug#858555: chiark-utils-bin: [patch] add timeout option to with-lock-ex Ian Jackson <ijackson@chiark.greenend.org.uk> - 2017-03-23 17:00 +0100
    Bug#858555: chiark-utils-bin: [patch] add timeout option to with-lock-ex Matthew Vernon <matthew@debian.org> - 2017-03-23 17:10 +0100

csiph-web