Date: Wed, 16 Jul 2014 12:34:11 +0200 From: Baptiste Daroussin <bapt@FreeBSD.org> To: Mateusz Guzik <mjguzik@gmail.com> Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org Subject: Re: svn commit: r268745 - in head/usr.bin: . timeout Message-ID: <20140716103411.GH48710@ivaldir.etoilebsd.net> In-Reply-To: <20140716102908.GA17076@dft-labs.eu> References: <201407160955.s6G9taro084054@svn.freebsd.org> <20140716102908.GA17076@dft-labs.eu>
next in thread | previous in thread | raw e-mail | index | archive | help
--YrQNB5Deg1WGKZi3 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Wed, Jul 16, 2014 at 12:29:08PM +0200, Mateusz Guzik wrote: > On Wed, Jul 16, 2014 at 09:55:36AM +0000, Baptiste Daroussin wrote: > > Author: bapt > > Date: Wed Jul 16 09:55:36 2014 > > New Revision: 268745 > > URL: http://svnweb.freebsd.org/changeset/base/268745 > >=20 > > Log: > > New BSDL timeout(1) utility compatible with GNU timeout > > =20 > > it fully passes the GNU timeout regression tests, it is written in a = mostly > > portable way (only signal parsing is relying on non portable structur= es) > > =20 > > Phabric: D377 > >=20 > > +static sig_atomic_t sig_chld =3D 0; > > +static sig_atomic_t sig_term =3D 0; > > +static sig_atomic_t sig_alrm =3D 0; > > +static sig_atomic_t sig_ign =3D 0; > > + >=20 > No reason to se these explicitely to 0. >=20 > > +static void > > +usage(void) > > +{ > > + fprintf(stderr, "Usage: %s [--signal sig | -s sig] [--preserve-status= ]" > > + " [--kill-after time | -k time] [--foreground] <duration> <comman= d>" > > + " <arg ...>\n", getprogname()); > > + >=20 > Missing newline at the begnning. New line at the beginning? >=20 > > + exit(EX_USAGE); > > +} > > + > > + switch(signo) { > > + case 0: >=20 > sig 0? I doubt it is ever delivered. one of the tests from the GNU testsuite is passing 0 signal >=20 > > + if (timedout && !preserve) > > + pstat =3D EXIT_TIMEOUT; > > + >=20 > Bad indentation. right I'll fix regards, Bapt --YrQNB5Deg1WGKZi3 Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iEYEARECAAYFAlPGVSMACgkQ8kTtMUmk6Ey5XQCdFEqvPHJegoXlHyz+edknwxbz 8IgAn1SfQxtFaOhq9AmItPVHKUoSoW77 =Grex -----END PGP SIGNATURE----- --YrQNB5Deg1WGKZi3--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20140716103411.GH48710>