Re: git: f155b5e0d9f3 - main - net-mgmt/monitoring-plugins: Add CURL option to build check_curl

Am 22.08.26 um 06:19 schrieb Matthias Fechner:
> Hi Jochen,
> 
> Am 22.08.26 um 00:59 schrieb Jochen Neumeister:
>>      Upstream deprecated check_http in 3.0.0 and declared check_curl its
>>      stable replacement, so make the plugin available here as an option.
>>      It is off by default, like every other option in this port that 
>> pulls
>>      an external library.
> at first thanks for this!
> 
> I vote to enable CURL be default.
> 
> If check_http is deprecated we should give the user time to be able to 
> migrate to check_curl.
> But to be able to do this, the command must exist in the pkg.
> 
> Thanks
> Matthias
> 
> 

Moin Matthias,

I'd rather keep it off for now.

Upstream tracks the removal in issue #1806: open since November 2022, 
milestone 4.0.0, no date, and 3.1.0 still in between. So there's no 
deadline we could miss, and turning the default on later is one line.

Until then it puts curl plus six more packages on every box that only 
runs check_disk and check_load over nrpe. Every other option in this 
port that needs a library is off for the same reason.

I'd flip it once 4.0.0 comes into sight. But the port has no maintainer, 
so it's not mine to decide alone. If you still prefer on by default, 
just change it, I won't argue.

Cheers
Jochen