See inline
On 2012-07-13 22:20, Kelly O'Hair wrote:
Something seems strange here:
http://cr.openjdk.java.net/~naoto/6336885/webrev.00/make/java/java/localegen.sh.sdiff.html
It's like someone was avoiding overall quotes, but using them to add spaces
somehow...
I sure would like to get rid of this shell logic, seems like there are lots
repeated logic that
this script does over and over or could be done with makefile pattern subst's
instead of exec's.
The new version isn't any worse than the old in my opinion. In
build-infra, this file is indeed replaced with make logic. After having
decoded both versions I'm confident in converting the changes.
Overall, just looking at the makefiles, the build-infra team may need some time
to fully absorb this into the
new makefiles, some of it will be trivial, not sure all of it will be.
I have applied the patch to a clone of build-infra and done the minimal
changes to keep build-infra building, which was rather trivial. The
resulting build has large differences since I haven't converted all of
it yet. There are a couple of things that will require some more work,
but not more than a day or two.
Not sure how to proceed here, the build-infra team does need an action item to
deal with this, maybe
before it gets integrated because I suspect the new makefiles will break with
all the filename or
directory changes. But I hate to hold up your integration plans.
I see these as possible options:
1. Let this go in, build-infra will break in jdk8 until we do our next
push and it trickles through the repos.
2. The build-infra project provides a patch with the full conversion
that can go in together with these changes.
3. The build-infra project provides a simple patch which just keeps the
build-infra-build from failing that can go in with these changes.
The conversion needs to happen regardless of option. The changes
required are pretty isolated from remaining build-infra work. What do
you think?
Other than that, I have no objections to the review.
/Erik
I'd like to get some advice from Erik on this before saying anything further.
-kto
On Jul 10, 2012, at 1:42 PM, Naoto Sato wrote:
Hello,
Please review the JDK8 changes for JEP 127: Improve Locale Data Packaging and
Adopt Unicode CLDR Data (http://openjdk.java.net/jeps/127). The webrev is
located at:
http://cr.openjdk.java.net/~naoto/6336885/webrev.00/
The main bug id for this enhancement is:
6336885: RFE: Locale Data Deployment Enhancements
Along with this, the following bugs/enhancements are also implemented in this
change:
4609153 Provide locale data for Indic locales
5104387 Support for gl_ES locale (galician language)
6337471 desktop/system locale preferences support
7056139 (cal) SPI support for locale-dependent Calendar parameters
7058206 Provide CalendarData SPI for week params and display field value names
7073852 Support multiple scripts for digits and decimal symbols per locale
7079560 [Fmt-Da] Context dependent month names support in SimpleDateFormat
7171324 getAvailableLocales() of locale sensitive services should return the
actual availability of locales
7151414 (cal) Support calendar type identification
7168528 LocaleServiceProvider needs to be aware of Locale extensions
7171372 (cal) locale's default Calendar should be created if unknown calendar
is specified
Please note that packaging changes that relate to Jigsaw module system aren't
included in this changeset.
Naoto Sato and Masayoshi Okutsu
