svn commit: r443793 - head/lang/gcc7-devel

[Jan Beich]

Gerald Pfeifer <gerald at pfeifer.com> writes:

> On Sun, 18 Jun 2017, Jan Beich wrote:
>
>>>   When we set STAGE1_CXXFLAGS / CXXFLAGS in MAKE_FLAGS for arm platforms,
>>>   do so incrementally (with += instead of =).
>> Did you mean to use MAKE_ENV instead of +=?
>
> I'm not sure. ;-)  In your example, += did not actually work in
> the MAKE_ENV tests, did it?

Correct, += syntax isn't supported by sh(1) and += syntax in gmake(1)
acts differently between on command line and in a file.

> Did you mean that the commit message should have referred to
> MAKE_ARGS instead of MAKE_FLAGS?

MAKE_ARGS usage looks wrong as it discards system CXXFLAGS i.e., -O2
-fno-strict-aliasing -pipe. Replacing = with += won't fix it.

Whether there's a bug or not requires understanding the intent. With
commit messages that state only "what" was done, but not "why", one can
only guess the former. Testing can help deriving the intent but in my
case poudriere fails to create armv6 jail (unlike aarch64).

Or perhaps you can answer "Why do you need the incremental assignment?"