> 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"`.

Magnus Ihse Bursie has updated the pull request incrementally with one 
additional commit since the last revision:

  Fix copyright year and remove space

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

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/5276/files
  - new: https://git.openjdk.java.net/jdk/pull/5276/files/3083307d..5dbc5ed1

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=5276&range=02
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=5276&range=01-02

  Stats: 2 lines in 1 file changed: 0 ins; 0 del; 2 mod
  Patch: https://git.openjdk.java.net/jdk/pull/5276.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/5276/head:pull/5276

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

Reply via email to