Re: git: cf0ff8e3b4be - main - net-mgmt/mac-telnet: Don't use chown pre-install

From: Baptiste Daroussin <bapt_at_FreeBSD.org>
Date: Wed, 21 Sep 2022 17:25:52 UTC 
On Wed, Sep 21, 2022 at 05:13:37PM +0000, Joe Marcus Clarke wrote:
> The branch main has been updated by marcus:
> 
> URL: https://cgit.FreeBSD.org/ports/commit/?id=cf0ff8e3b4be3703b5a119374a5d131384a00cbb
> 
> commit cf0ff8e3b4be3703b5a119374a5d131384a00cbb
> Author:     Joe Marcus Clarke <marcus@FreeBSD.org>
> AuthorDate: 2022-09-21 17:12:31 +0000
> Commit:     Joe Marcus Clarke <marcus@FreeBSD.org>
> CommitDate: 2022-09-21 17:12:31 +0000
> 
>     net-mgmt/mac-telnet: Don't use chown pre-install
> ---
>  net-mgmt/mac-telnet/Makefile                       |  4 ++++
>  net-mgmt/mac-telnet/files/patch-config_Makefile.am | 11 +++++++++++
>  2 files changed, 15 insertions(+)
> 
> diff --git a/net-mgmt/mac-telnet/Makefile b/net-mgmt/mac-telnet/Makefile
> index b92beb387b5b..6bf20c22e268 100644
> --- a/net-mgmt/mac-telnet/Makefile
> +++ b/net-mgmt/mac-telnet/Makefile
> @@ -20,4 +20,8 @@ GNU_CONFIGURE=	yes
>  LDFLAGS+=	-L${LOCALBASE}/lib
>  CONFIGURE_ENV+=	CRYPTO_CFLAGS="-I${OPENSSLINC}" CRYPTO_LIBS="-L${OPENSSLLIB} -lssl -lcrypto"
>  
> +post-install:
> +	@${CHMOD} 0600 ${STAGEDIR}${PREFIX}/etc/mactelnetd.users
> +	@-${CHOWN} root ${STAGEDIR}${PREFIX}/etc/mactelnetd.users

This is not nessecary at all

@(,,0600) etc/mactelnetd.users

in plist is the only thing that is needed.

Best regards,
Bapt