Re: git: 14821d60aad2 - main - databases/clickhouse-cpp: Add new port

From: Guido Falsi <madpilot_at_FreeBSD.org>
Date: Wed, 31 Dec 2025 17:10:49 UTC
On 12/31/25 08:47, Guido Falsi wrote:
> On 12/31/25 08:32, Daniel Engberg wrote:
>> On 2025-12-30 19:50, Guido Falsi wrote:
>>> The branch main has been updated by madpilot:
>>>
>>> URL: https://cgit.FreeBSD.org/ports/commit/? 
>>> id=14821d60aad2335e18bd68eec7cdfd9c3299ca9a
>>>
>>> commit 14821d60aad2335e18bd68eec7cdfd9c3299ca9a
>>> Author:     Guido Falsi <madpilot@FreeBSD.org>
>>> AuthorDate: 2025-12-30 18:48:29 +0000
>>> Commit:     Guido Falsi <madpilot@FreeBSD.org>
>>> CommitDate: 2025-12-30 18:48:29 +0000
>>>
>>>      databases/clickhouse-cpp: Add new port
>>>      clickhouse-cpp is the official C++ client library for ClickHouse,
>>>      providing a fast and type-safe interface to ClickHouse using its
>>>      native binary protocol.
>>> ---
>>>   databases/Makefile                                 |  1 +
>>>   databases/clickhouse-cpp/Makefile                  | 41 +++++++++++ 
>>> + ++++++++
>>>   databases/clickhouse-cpp/distinfo                  |  3 ++
>>>   .../files/patch-clickhouse_base_platform.h         | 20 ++++++++++
>>>   .../files/patch-clickhouse_base_socket.cpp         | 11 ++++++
>>>   .../files/patch-clickhouse_base_socket.h           | 13 +++++++
>>>   databases/clickhouse-cpp/pkg-descr                 |  3 ++
>>>   databases/clickhouse-cpp/pkg-plist                 | 45 +++++++++++ 
>>> + ++++++++++
>>>   8 files changed, 137 insertions(+)
>>>
>>> ...
>>>
>>> +.if ${OPSYS} == FreeBSD
>>> +USES+=    llvm:min=21
>>> +CFLAGS+=-Wno-error=int-conversion
>>> +.endif
>>>
>>> Hi,
>>>
>>> What is USES supposed to fix? :-)
> 
> Actually I copied this from databases/clickhouse in early stages of 
> porting and, I confess, forgot about this. I did assume that the version 
> of clang being used was relevant, but I could be wrong.
> 
> Now that you mention it, I need to do more testing without that 
> workaround. OTOH maybe databases/clickhouse too need to revise that stanza?
> 

Removed the whole condition, as it really makes no difference. Thanks 
for reporting!

-- 
Guido Falsi <madpilot@FreeBSD.org>