Re: git: 2aedfff8a362 - main - sysutils/tuptime: Fix cron file

From: Mina_Galić <freebsd_at_igalic.co>
Date: Mon, 08 May 2023 10:03:42 UTC
On 08/05/2023 08:36, Fernando Apesteguía wrote:
> The branch main has been updated by fernape:
>
> URL: https://cgit.FreeBSD.org/ports/commit/?id=2aedfff8a362c432d6f6ee89587bd8f5470f9687
>
> commit 2aedfff8a362c432d6f6ee89587bd8f5470f9687
> Author:     Jeremy Johnston <jeremy@smart-serv.net>
> AuthorDate: 2023-05-08 06:28:09 +0000
> Commit:     Fernando Apesteguía <fernape@FreeBSD.org>
> CommitDate: 2023-05-08 07:32:03 +0000
>
>      sysutils/tuptime: Fix cron file
>      
>      Fix cron file to use -q file to grep aswell as switch to checking service
>      tuptime rcvar as this includes other locations such as
>      /usr/local/etc/rc.conf.d/tuptime

After the changes made during the discussion in the PR, this commit 
message is now a bit confusing.

-q was there before.

We're not (manually) checking rcvar; we're "simply" asking service to do 
this task for us.

>      
>      PR:             271243
>      Reported by:    jeremy@smart-serv.net (maintainer)
> ---
>   sysutils/tuptime/Makefile   | 2 +-
>   sysutils/tuptime/files/cron | 2 +-
>   2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/sysutils/tuptime/Makefile b/sysutils/tuptime/Makefile
> index 1e9d1e52a6ef..914069786286 100644
> --- a/sysutils/tuptime/Makefile
> +++ b/sysutils/tuptime/Makefile
> @@ -1,6 +1,6 @@
>   PORTNAME=	tuptime
>   DISTVERSION=	5.2.2
> -PORTREVISION=	1
> +PORTREVISION=	2
>   CATEGORIES=	sysutils
>   
>   MAINTAINER=	jeremy@smart-serv.net
> diff --git a/sysutils/tuptime/files/cron b/sysutils/tuptime/files/cron
> index 833de2151cab..d8eabe5dd6ff 100644
> --- a/sysutils/tuptime/files/cron
> +++ b/sysutils/tuptime/files/cron
> @@ -1 +1 @@
> -*/5	*	*	*	*	_tuptime	(grep '^tuptime_enable="YES"' /etc/rc.conf) && /usr/local/bin/tuptime -q
> +*/5	*	*	*	*	_tuptime	(/usr/sbin/service tuptime enabled) && /usr/local/bin/tuptime -q


Kind regards,


Mina