public inbox for git@vger.kernel.org
 help / color / mirror / Atom feed
From: Brad Smith <brad@comstyle.com>
To: Junio C Hamano <gitster@pobox.com>
Cc: git@vger.kernel.org
Subject: Re: [PATCH] config.mak.uname: update settings for OpenBSD
Date: Mon, 2 Jun 2025 01:05:13 -0400	[thread overview]
Message-ID: <3ef13594-41a7-4455-bec5-3da6ff517341@comstyle.com> (raw)
In-Reply-To: <xmqqldqavis7.fsf@gitster.g>

On 2025-06-02 1:01 a.m., Junio C Hamano wrote:
> Brad Smith <brad@comstyle.com> writes:
>
>> OpenBSD requires DIR_HAS_BSD_GROUP_SEMANTICS.
>>
>> OpenBSD has never had the BSD sysctl KERN_PROC_PATHNAME nor
>> does it support or use the /proc filesystem.
>>
>> OpenBSD has had strcasestr() since 3.8. OpenBSD has had memmem()
>> since 5.4.
> ;-) Thanks.
>
> 5.4 was from Nov 2013 if I can trust
> https://undeadly.org/cgi?action=article;sid=20131101142807 and it is
> about time we adjust ourselves to the more recent world order.
>
> Will apply.  Thanks.

Yup, that is correct. For us that is 23 releases and over 11+ years old. 
I think
we're good.

>> Signed-off-by: Brad Smith <brad@comstyle.com>
>> ---
>>   config.mak.uname | 5 +----
>>   1 file changed, 1 insertion(+), 4 deletions(-)
>>
>> diff --git a/config.mak.uname b/config.mak.uname
>> index df172d5871..5a0b4200c3 100644
>> --- a/config.mak.uname
>> +++ b/config.mak.uname
>> @@ -301,17 +301,14 @@ ifeq ($(uname_S),FreeBSD)
>>   	FILENO_IS_A_MACRO = UnfortunatelyYes
>>   endif
>>   ifeq ($(uname_S),OpenBSD)
>> -	NO_STRCASESTR = YesPlease
>> -	NO_MEMMEM = YesPlease
>> +	DIR_HAS_BSD_GROUP_SEMANTICS = YesPlease
>>   	USE_ST_TIMESPEC = YesPlease
>>   	NEEDS_LIBICONV = YesPlease
>>   	BASIC_CFLAGS += -I/usr/local/include
>>   	BASIC_LDFLAGS += -L/usr/local/lib
>>   	HAVE_PATHS_H = YesPlease
>>   	HAVE_BSD_SYSCTL = YesPlease
>> -	HAVE_BSD_KERN_PROC_SYSCTL = YesPlease
>>   	CSPRNG_METHOD = arc4random
>> -	PROCFS_EXECUTABLE_PATH = /proc/curproc/file
>>   	FREAD_READS_DIRECTORIES = UnfortunatelyYes
>>   	FILENO_IS_A_MACRO = UnfortunatelyYes
>>   endif

      reply	other threads:[~2025-06-02  5:05 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-06-02  3:14 [PATCH] config.mak.uname: update settings for OpenBSD Brad Smith
2025-06-02  4:45 ` Carlo Marcelo Arenas Belón
2025-06-02  5:01 ` Junio C Hamano
2025-06-02  5:05   ` Brad Smith [this message]

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

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=3ef13594-41a7-4455-bec5-3da6ff517341@comstyle.com \
    --to=brad@comstyle.com \
    --cc=git@vger.kernel.org \
    --cc=gitster@pobox.com \
    /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