Re: git: 79f8f9c8f54b - main - */*: convert Makefile patches to MAKE_ARGS after "Move man pages to share/man"

From: Mathieu Arnold <mat_at_freebsd.org>
Date: Mon, 11 Mar 2024 07:42:54 UTC
On Sun, Mar 10, 2024 at 10:58:38PM +0000, Vladimir Druzenko wrote:
> The branch main has been updated by vvd:
> 
> URL: https://cgit.FreeBSD.org/ports/commit/?id=79f8f9c8f54b8de3d324d30ce9a19649bbe69e69
> 
> commit 79f8f9c8f54b8de3d324d30ce9a19649bbe69e69
> Author:     Vladimir Druzenko <vvd@FreeBSD.org>
> AuthorDate: 2024-03-10 22:49:41 +0000
> Commit:     Vladimir Druzenko <vvd@FreeBSD.org>
> CommitDate: 2024-03-10 22:49:41 +0000
> 
>     */*: convert Makefile patches to MAKE_ARGS after "Move man pages to share/man"
>     
>     Replaced patches for Makefiles with "MAKE_ARGS+=MANDIR=${PREFIX}/share/man"

Why += ? Is the variable defined before in the Makefile ? If it is, then
you should append to its previous value, if it is not, += will only
confuse people in the future, wondering why a non existing variable is
appended to instead of being set.

-- 
Mathieu Arnold