When doing `# Setup compilation for preview classes in the module` a comma is 
missing after `EXCLUDE_FILES`. So instead of setting 
`<module>-preview_KEEP_ALL_TRANSLATIONS := value` it is added as to the 
`<module>-preview_EXCLUDE_FILES` variable.


<module>-preview_EXCLUDE_FILES := $(EXCLUDE_FILES) KEEP_ALL_TRANSLATIONS := 
$(KEEP_ALL_TRANSLATIONS)


Instead of 

<module>-preview_EXCLUDE_FILES := $(EXCLUDE_FILES) 
<module>-preview_KEEP_ALL_TRANSLATIONS := $(KEEP_ALL_TRANSLATIONS)


The only usage of this variable set to true is in `jdk.localedata`, which is 
probably why it went unnoticed. 

---------
- [x] I confirm that I make this contribution in accordance with the [OpenJDK 
Interim AI Policy](https://openjdk.org/legal/ai).

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

Commit messages:
 - make/CompileJavaModules.gmk: Missing comma

Changes: https://git.openjdk.org/valhalla/pull/2398/files
  Webrev: https://webrevs.openjdk.org/?repo=valhalla&pr=2398&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8384053
  Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
  Patch: https://git.openjdk.org/valhalla/pull/2398.diff
  Fetch: git fetch https://git.openjdk.org/valhalla.git pull/2398/head:pull/2398

PR: https://git.openjdk.org/valhalla/pull/2398

Reply via email to