Re: git: ce3687b1f659 - main - sysutils/sensors: new port, show sensors/temperatures info

From: Daniel Engberg <diizzy_at_FreeBSD.org>
Date: Sat, 02 Aug 2025 12:42:22 UTC
On 2025-08-02 13:53, Älven wrote:
> The branch main has been updated by alven:
>
> URL:https://cgit.FreeBSD.org/ports/commit/?id=ce3687b1f6590796c17405040d70fdf6f4977e9f
>
> commit ce3687b1f6590796c17405040d70fdf6f4977e9f
> Author:     Älven<alven@FreeBSD.org>
> AuthorDate: 2025-08-02 11:46:40 +0000
> Commit:     Älven<alven@FreeBSD.org>
> CommitDate: 2025-08-02 11:52:36 +0000
>
>      sysutils/sensors: new port, show sensors/temperatures info
>      
>      The sensors(8) tools shows temperatures and sensors info on FreeBSD UNIX systems.
>      
>      More details here:
>      -https://github.com/vermaden/sensors
>      -https://vermaden.wordpress.com/2022/02/15/sensors-information-on-freebsd/
>      
>      PR:             288500
>      Approved by:    yuri@ (Mentor)
> ...
> --- /dev/null
> +++ b/sysutils/sensors/distinfo
> @@ -0,0 +1,5 @@
> +TIMESTAMP = 1753639972
> +SHA256 (sensors/sensors-0.6.tar.gz) = 6304f7f6da342c1a856deebb16784437d6a7001e027c1fab6be38d238a5f5b55
> +SIZE (sensors/sensors-0.6.tar.gz) = 3257
> +SHA256 (sensors/LICENSE) = b4bb8570bcaaf370e936763a1cf5dade6994c8fceb93bff22c9b768b8184f14f
> +SIZE (sensors/LICENSE) = 1356

Hi,

Yuri, can you please review more carefully?

This will cause clashes because LICENSE is unversioned. Given that 
submitter is maintainer why not ask to fix the repo (use tags and don't 
add binaries to repo) or create proper release assets?Best regards, Daniel