Integrated: 8273072: Avoid using += in configure

Magnus Ihse Bursie ihse at openjdk.java.net
Fri Aug 27 13:58:10 UTC 2021


On Fri, 27 Aug 2021 10:21:26 GMT, Magnus Ihse Bursie <ihse at openjdk.org> wrote:

> JDK-8272700 was created to fix a bug in a variable assignment in configure. While it fixed the bug, it kept the problematic syntax that caused the bug in the first place. 
> 
> We do not use the `FOO+="appended"` syntax for appending to variables in shell scripts, since this differs from what you'd expect (and what make produces) in that no space is added before the appended text. 
> 
> Instead, we use the longer, but clearer `FOO="$FOO appended"`.

This pull request has now been integrated.

Changeset: a033aa5a
Author:    Magnus Ihse Bursie <ihse at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/a033aa5a3d9c63d72d11af218b9896b037fbd8de
Stats:     8 lines in 4 files changed: 1 ins; 0 del; 7 mod

8273072: Avoid using += in configure

Reviewed-by: dholmes, jiefu

-------------

PR: https://git.openjdk.java.net/jdk/pull/5276



More information about the build-dev mailing list