git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH 2/3] Don't lose Error.pm if $@ gets clobbered.

From
Michael G. Schwern <schwern@pobox.com>
Date
Jul 25, 2012, 03:21 UTC
Message-ID
<1343186471-1024-3-git-send-email-schwern@pobox.com>
In-Reply-To
<1343186471-1024-1-git-send-email-schwern@pobox.com>
From: "Michael G. Schwern" <schwern@pobox.com>

In older Perls, sometimes $@ can become unset between the eval and checking $@. Its safer to check the eval directly.

Signed-off-by: Michael G Schwern <schwern@pobox.com>
---
 perl/Makefile.PL | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/perl/Makefile.PL b/perl/Makefile.PL
index 87e1f62..887fa1b 100644
--- a/perl/Makefile.PL
+++ b/perl/Makefile.PL
@@ -41,8 +41,7 @@ my %pm = (
 
 # We come with our own bundled Error.pm. It's not in the set of default
 # Perl modules so install it if it's not available on the system yet.
-eval { require Error };
-if ($@ || $Error::VERSION < 0.15009) {
+if ( !eval { require Error } || $Error::VERSION < 0.15009) {
 	$pm{'private-Error.pm'} = '$(INST_LIBDIR)/Error.pm';
 }
 
-- 
1.7.11.1
Previous: Michael G. SchwernNext: Michael G. Schwern
Message 3 of 16 in “Teach Makefile.PL to find .pm files on its own”
  1. Michael G. SchwernJul 25, 2012
  2. 1/3 Quiet warning if Makefile.PL is run with -w and no --localedirMichael G. Schwern, Jul 25, 2012
  3. 2/3 Don't lose Error.pm if $@ gets clobbered.Michael G. Schwern, Jul 25, 2012
  4. 3/3 The Makefile.PL will now find .pm files itself.Michael G. Schwern, Jul 25, 2012
  5. Jonathan NiederJul 25, 2012
  6. Jonathan NiederJul 25, 2012
  7. Michael G SchwernJul 25, 2012
  8. Jonathan NiederJul 25, 2012
  9. Junio C HamanoJul 25, 2012
  10. Michael G SchwernJul 25, 2012
  11. Junio C HamanoJul 25, 2012
  12. Michael G SchwernJul 25, 2012
  13. Junio C HamanoJul 25, 2012
  14. Michael G SchwernJul 25, 2012
  15. Junio C HamanoJul 25, 2012
  16. Eric WongJul 25, 2012

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.