From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:47300) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1f1dww-0004k0-KE for guix-patches@gnu.org; Thu, 29 Mar 2018 16:15:08 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1f1dws-0004ug-37 for guix-patches@gnu.org; Thu, 29 Mar 2018 16:15:06 -0400 Received: from debbugs.gnu.org ([208.118.235.43]:51185) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1f1dwr-0004uD-Vu for guix-patches@gnu.org; Thu, 29 Mar 2018 16:15:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1f1dwr-0005Mr-LB for guix-patches@gnu.org; Thu, 29 Mar 2018 16:15:01 -0400 Subject: [bug#30950] [PATCH shepherd]: Update required guile version, and remove some hacks Resent-Message-ID: From: ludo@gnu.org (Ludovic =?UTF-8?Q?Court=C3=A8s?=) References: <87lgefuk4o.fsf@zancanaro.id.au> Date: Thu, 29 Mar 2018 22:14:01 +0200 In-Reply-To: <87lgefuk4o.fsf@zancanaro.id.au> (Carlo Zancanaro's message of "Mon, 26 Mar 2018 22:55:03 +1100") Message-ID: <87605evdva.fsf@gnu.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+kyle=kyleam.com@gnu.org Sender: "Guix-patches" To: Carlo Zancanaro Cc: 30950@debbugs.gnu.org Hello! Carlo Zancanaro skribis: > I'm not very familiar with autotools, but I think I got the configure > incantation right (I stole it from Guix). Well done. :-) > From 8c812534137a5dc17dd8073706983c451d26f2db Mon Sep 17 00:00:00 2001 > From: Carlo Zancanaro > Date: Mon, 26 Mar 2018 14:44:18 +1100 > Subject: [PATCH 1/3] Update Guile dependency to 2.0.13 or later > > * README (Requirements): Change 2.x to 2.0.13 or later. > * configure.ac: Check for 2.0.13 or later if Guile 2.0 is detected. LGTM. > From e11708aba0fbafd4c83273ee1fa5147e54d1c80e Mon Sep 17 00:00:00 2001 > From: Carlo Zancanaro > Date: Mon, 26 Mar 2018 14:49:18 +1100 > Subject: [PATCH 2/3] Remove EINTR-safe, and all references to it. > > * modules/shepherd/support.scm (EINTR-safe): Remove procedure and its exp= ort. > * modules/shepherd/service.scm (system*, system*): Remove now-unnecessary > procedures. > (waitpid*): Remove references to EINTR-safe. > * modules/shepherd.scm (main): Remove references to EINTR-safe. LGTM. > From 63bc9339d88d8f1bd8a9b366774ce8e33d76dd00 Mon Sep 17 00:00:00 2001 > From: Carlo Zancanaro > Date: Mon, 26 Mar 2018 14:55:32 +1100 > Subject: [PATCH 3/3] Remove SIGALRM hack. > > * modules/shepherd.scm (main): Remove SIGALRM hack for guile <=3D 2.0.9. > --- > modules/shepherd.scm | 9 --------- > 1 file changed, 9 deletions(-) > > diff --git a/modules/shepherd.scm b/modules/shepherd.scm > index 5d97598..69fd69d 100644 > --- a/modules/shepherd.scm > +++ b/modules/shepherd.scm > @@ -207,15 +207,6 @@ > (apply format #f (gettext (cadr args)) (caddr args)) > (quit 1)))) >=20=20 > - (when (provided? 'threads) > - ;; XXX: This terrible hack allows us to make sure that signal ha= ndlers > - ;; get a chance to run in a timely fashion. Without it, after a= n EINTR, > - ;; we could restart the accept(2) call below before the correspo= nding > - ;; async has been queued. See the thread at > - ;; . > - (sigaction SIGALRM (lambda _ (alarm 1))) > - (alarm 1)) Unfortunately I think the problem remains. That=E2=80=99s one of the reaso= ns for using signalfd(2). Can you create an account on Savannah so I can add you to the group and let you push the first two patches? :-) Thank you! Ludo=E2=80=99.