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 3/7] Makefile.PL: depend on autodie, at least for tests
  2023-08-30  5:10  5% [PATCH 0/7] various build fixes + OpenBSD compat Eric Wong
@ 2023-08-30  5:10  7% ` Eric Wong
  0 siblings, 0 replies; 2+ results
From: Eric Wong @ 2023-08-30  5:10 UTC (permalink / raw)
  To: meta

While using autodie everywhere is not appropriate[*], many of
our tests and FS access code can be easier-to-write and more
readable using autodie as we've started doing in XapHelperCxx.pm
and xap_helper.t

[*] - EAGAIN on non-blocking I/O shouldn't die, nor should
      certain cases of opening maybe-missing files for reading
---
 INSTALL     | 1 +
 Makefile.PL | 1 +
 2 files changed, 2 insertions(+)

diff --git a/INSTALL b/INSTALL
index f5e14ebe..5f080f28 100644
--- a/INSTALL
+++ b/INSTALL
@@ -201,6 +201,7 @@ library.  Debian-based distros put them in "libperl5.$MINOR" or
 "perl-modules-5.$MINOR"; and FreeBSD puts them in "perl5".
 RPM-based distros split them out into separate packages:
 
+* autodie                          rpm: perl-autodie
 * Digest::SHA                      rpm: perl-Digest-SHA
 * Data::Dumper                     rpm: perl-Data-Dumper
 * Encode                           rpm: perl-Encode
diff --git a/Makefile.PL b/Makefile.PL
index d2309336..5a5628ba 100644
--- a/Makefile.PL
+++ b/Makefile.PL
@@ -150,6 +150,7 @@ WriteMakefile(
 
 		# perl-modules-5.xx or libperl5.xx in Debian-based
 		# part of "perl5" on FreeBSD
+		'autodie' => 0,
 		'Compress::Raw::Zlib' => 0,
 		'Compress::Zlib' => 0,
 		'Data::Dumper' => 0,

^ permalink raw reply related	[relevance 7%]

* [PATCH 0/7] various build fixes + OpenBSD compat
@ 2023-08-30  5:10  5% Eric Wong
  2023-08-30  5:10  7% ` [PATCH 3/7] Makefile.PL: depend on autodie, at least for tests Eric Wong
  0 siblings, 1 reply; 2+ results
From: Eric Wong @ 2023-08-30  5:10 UTC (permalink / raw)
  To: meta

7/7 had me shaking my head for a bit, but I'd rather pay
that price than introduce weirdness with C++ templates.

Eric Wong (7):
  treewide: drop MSG_EOR with AF_UNIX+SOCK_SEQPACKET
  Makefile.PL: fix syntax for ASan and valgrind targets
  Makefile.PL: depend on autodie, at least for tests
  t/kqnotify: improve test reliability on OpenBSD
  xap_helper.h: don't compress debug sections on OpenBSD
  xap_helper.h: limit stderr assignment to glibc+FreeBSD
  xap_helper.h: fix double-free on OpenBSD hdestroy

 INSTALL                          |  1 +
 Makefile.PL                      | 10 +++--
 lib/PublicInbox/CodeSearchIdx.pm |  8 ++--
 lib/PublicInbox/IPC.pm           | 10 ++---
 lib/PublicInbox/LEI.pm           | 28 ++++++-------
 lib/PublicInbox/PktOp.pm         |  4 +-
 lib/PublicInbox/WQBlocked.pm     |  3 +-
 lib/PublicInbox/XapClient.pm     |  4 +-
 lib/PublicInbox/XapHelperCxx.pm  |  4 +-
 lib/PublicInbox/xap_helper.h     | 68 ++++++++++++++++++++++++++++----
 script/lei                       | 10 ++---
 t/check-www-inbox.perl           |  4 +-
 t/cmd_ipc.t                      |  8 ++--
 t/kqnotify.t                     | 55 +++++++++++++++++++-------
 t/lei-daemon.t                   |  4 +-
 t/xap_helper.t                   | 11 +++---
 16 files changed, 158 insertions(+), 74 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 --
2023-08-30  5:10  5% [PATCH 0/7] various build fixes + OpenBSD compat Eric Wong
2023-08-30  5:10  7% ` [PATCH 3/7] Makefile.PL: depend on autodie, at least for tests 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).