Hi,
I was focused mostly on *not *hardcoding the copyright text in yet
another place
and if possible avoiding a manual update of the year.
And yet since the source is being generated and is transient (not
retained after the build), the current year
is correct. (Same as cldr)
Roger
On 7/25/2017 1:16 PM, Nishit Jain wrote:
Hi Roger,
CLDR converter generate the copyright year by the current date in
which it is getting executed. I think the copyright year in
LocaleEquivalentMaps.java should be the year in which the lsr data is
last updated in jdk i.e. the year in which LocaleEquivalentMaps.java
is last modified. This makes it difficult to compute the year
automatically unless someone specifies it while updating the data.
> Can the latest copyright date be taken from the File-Date field of
the subtag-registry?
This may not be necessarily same as the year in which lsr data is
updated in jdk.
Regards,
Nishit Jain
On 25-07-2017 21:28, Roger Riggs wrote:
Hi Nishit,
Can the hardcoded copyright be made more automatic?
I see the make/src/classes/build/tools/cldrconverter has functions to
create/convert copyrights.
Can that be leveraged to generate the needed copyright and reduce
some potential maintenance
of the copyright date and or contents?
Can the latest copyright date be taken from the File-Date field of
the subtag-registry?
Thanks, Roger
On 7/20/2017 1:33 AM, Nishit Jain wrote:
Hi,
Please review the fix for JDK-8177472
Bug: https://bugs.openjdk.java.net/browse/JDK-8177472
Webrev: http://cr.openjdk.java.net/~nishjain/8177472/webrev.05/
Issue:The existing process of updating the LSR data requires manual
execution of EquivMapsGenerator.java for generation of the updated
maps.
Fix:The execution of EquivMapsGenerator.javafor generation of maps
is included in the JDK build process. After this change updation of
LSR data will mostly require replacement of old
language-subtag-registry.txt file with the new one and changing the
copyright year in EquivMapsGenerator.java (if required) for the
generated maps file.
Regards,
Nishit Jain
