From: Gaelan Steele <gaelans@icloud.com>
To: Eric Wong <e@80x24.org>
Cc: meta@public-inbox.org
Subject: Re: [PATCH 2/2] syscall: use pure Perl sendmsg/recvmsg on *BSD
Date: Sat, 6 Apr 2024 01:43:40 +0100 [thread overview]
Message-ID: <0C20FAD1-1983-4C38-BB6C-03D9DEFC0D58@icloud.com> (raw)
In-Reply-To: <20240129212319.98910-3-e@80x24.org>
> On Jan 29, 2024, at 9:23 PM, Eric Wong <e@80x24.org> wrote:
>
> While syscall symbols (e.g. SYS_*) have changed on us in FreeBSD
> during the history of Sys::Syscall and this project and did bite
> us in some cases; the actual numbers don't get recycled for new
> syscalls. We're also fortunate that sendmsg and recvmsg syscalls
> and associated msghdr and cmsg structs predate the BSD forks and
> are compatible across all the BSDs I've tried.
>
> OpenBSD routes Perl `syscall' through libc; while NetBSD + FreeBSD
> document procedures for maintaining backwards compatibility.
> It looks like Dragonfly follows FreeBSD, here.
>
> Tested on i386 OpenBSD, and amd64 {Free,Net,Open,Dragonfly}BSD
>
> This enables *BSD users to use lei, -cindex and future SCM_RIGHTS-only
> features without needing Inline::C.
Unfortunately this patch broke public-inbox on Darwin:
Bareword "SIZEOF_cmsghdr" not allowed while "strict subs" in use at /tmp/public-inbox/lib/PublicInbox/Syscall.pm line 456.
BEGIN not safe after errors--compilation aborted at /tmp/public-inbox/lib/PublicInbox/Syscall.pm line 460.
Compilation failed in require at /tmp/public-inbox/lib/PublicInbox/DS.pm line 31.
BEGIN failed--compilation aborted at /tmp/public-inbox/lib/PublicInbox/DS.pm line 32.
Compilation failed in require at /tmp/public-inbox/lib/PublicInbox/Daemon.pm line 17.
BEGIN failed--compilation aborted at /tmp/public-inbox/lib/PublicInbox/Daemon.pm line 17.
Compilation failed in require at /tmp/public-inbox/script/public-inbox-httpd line 7.
BEGIN failed--compilation aborted at /tmp/public-inbox/script/public-inbox-httpd line 7.
I’m not enough of a Perl person to fully untangle this. As
best I can tell, the intent is that non-Linux/BSD OSes should
still work with Inline::C, but this doesn’t work in practice
due to a bug?
It may also be possible to use the BSD approach on Darwin -
Darwin ascribes to the BSD school of thought where libc is the
only Officially Stable interface, but if you can get away with
it on the real BSDs maybe you can get away with it on fake BSD
too.
Best wishes,
Gaelan
next prev parent reply other threads:[~2024-04-06 0:43 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-11-28 14:56 [PATCH 00/14] IT'S ALIVE! www loads cindex join data Eric Wong
2023-11-28 14:56 ` [PATCH 01/14] test_common: create_*: detect changes all parameters Eric Wong
2023-11-28 14:56 ` [PATCH 02/14] t/cindex*: require SCM_RIGHTS for these tests Eric Wong
2024-01-29 21:23 ` [PATCH 0/2] pure Perl sendmsg/recvmsg on *BSD Eric Wong
2024-01-29 21:23 ` [PATCH 1/2] syscall: update formatting to match our codebase Eric Wong
2024-01-29 21:23 ` [PATCH 2/2] syscall: use pure Perl sendmsg/recvmsg on *BSD Eric Wong
2024-04-06 0:43 ` Gaelan Steele [this message]
2024-04-08 9:48 ` [RFT] syscall: set default constants for Inline::C platforms Eric Wong
2024-04-08 12:12 ` Gaelan Steele
2024-04-08 20:11 ` Eric Wong
2023-11-28 14:56 ` [PATCH 03/14] codesearch: eliminate redundant substitutions Eric Wong
2023-11-28 14:56 ` [PATCH 04/14] solver: schedule cleanup after synchronous git->check Eric Wong
2023-11-28 14:56 ` [PATCH 05/14] xap_helper.h: move cindex endpoints to separate file Eric Wong
2023-11-28 14:56 ` [PATCH 06/14] xap_helper: implement mset endpoint for WWW, IMAP, etc Eric Wong
2023-11-28 14:56 ` [PATCH 07/14] hval: use File::Spec to make relative paths for href Eric Wong
2023-11-28 14:56 ` [PATCH 08/14] www: load and use cindex join data Eric Wong
2023-11-28 14:56 ` [PATCH 09/14] git: speed up ->git_path for non-worktrees Eric Wong
2023-11-28 14:56 ` [PATCH 10/14] cindex: require `-g GIT_DIR' or `-r PROJECT_ROOT' Eric Wong
2023-11-28 14:56 ` [PATCH 11/14] git: speed up Git->new by 5% or so Eric Wong
2023-11-28 14:56 ` [PATCH 12/14] admin: resolve_git_dir respects symlinks Eric Wong
2023-11-28 14:56 ` [PATCH 13/14] cindex: extra quit checks Eric Wong
2023-11-28 14:56 ` [PATCH 14/14] www: start working on a repo listing Eric Wong
2023-11-28 17:55 ` [PATCH 15/14] www: load cindex join data for ->ALL, too Eric Wong
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://public-inbox.org/README
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=0C20FAD1-1983-4C38-BB6C-03D9DEFC0D58@icloud.com \
--to=gaelans@icloud.com \
--cc=e@80x24.org \
--cc=meta@public-inbox.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).