user/dev discussion of public-inbox itself
 help / color / mirror / code / Atom feed
Search results ordered by [date|relevance]  view[summary|nested|Atom feed]
thread overview below | download mbox.gz: |
* [PATCH 2/2] spawnpp: die instead of exit on exec failure
  2016-02-28 23:59  5% [PATCH 0/2] minor bugfixes Eric Wong
@ 2016-02-28 23:59  7% ` Eric Wong
  0 siblings, 0 replies; 2+ results
From: Eric Wong @ 2016-02-28 23:59 UTC (permalink / raw)
  To: meta

Perl may complain about exit not being executed, but not die.
---
 lib/PublicInbox/SpawnPP.pm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/PublicInbox/SpawnPP.pm b/lib/PublicInbox/SpawnPP.pm
index 1a402e4..288625d 100644
--- a/lib/PublicInbox/SpawnPP.pm
+++ b/lib/PublicInbox/SpawnPP.pm
@@ -25,7 +25,7 @@ sub public_inbox_fork_exec ($$$$$$) {
 			$ENV{$k} = $v;
 		}
 		exec @$cmd;
-		exit 1;
+		die "exec $cmd->[0] failed: $!\n";
 	}
 	$pid;
 }
-- 
EW


^ permalink raw reply related	[relevance 7%]

* [PATCH 0/2] minor bugfixes
@ 2016-02-28 23:59  5% Eric Wong
  2016-02-28 23:59  7% ` [PATCH 2/2] spawnpp: die instead of exit on exec failure Eric Wong
  0 siblings, 1 reply; 2+ results
From: Eric Wong @ 2016-02-28 23:59 UTC (permalink / raw)
  To: meta

Eric Wong (2):
      newswww: set Content-Type properly
      spawnpp: die instead of exit on exec failure

 lib/PublicInbox/NewsWWW.pm | 2 +-
 lib/PublicInbox/SpawnPP.pm | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)


^ permalink raw reply	[relevance 5%]

Results 1-2 of 2 | reverse | options above
-- pct% links below jump to the message on this page, permalinks otherwise --
2016-02-28 23:59  5% [PATCH 0/2] minor bugfixes Eric Wong
2016-02-28 23:59  7% ` [PATCH 2/2] spawnpp: die instead of exit on exec failure Eric Wong

Code repositories for project(s) associated with this public inbox

	https://80x24.org/public-inbox.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).