On Wed, Dec 31, 2014 at 02:23:03AM +0000, Stuart Henderson wrote:
> On 2014/12/31 00:13, Marc Espie wrote:
> > well, jdk 1.7 just stopped building. because one file is >10 years old and
> > there is an explicit check for that in the build process.
> > 
> > Just think about it.
> 
> Can't go perpetuating 10-year-old currencies, that would be terrible!
> 
> This should be reasonable until openjdk catches up with the newer files
> from java 8.
> 
> Index: 
> patches/patch-jdk_make_tools_src_build_tools_generatecurrencydata_GenerateCurrencyData_java
> ===================================================================
> RCS file: 
> patches/patch-jdk_make_tools_src_build_tools_generatecurrencydata_GenerateCurrencyData_java
> diff -N 
> patches/patch-jdk_make_tools_src_build_tools_generatecurrencydata_GenerateCurrencyData_java
> --- /dev/null 1 Jan 1970 00:00:00 -0000
> +++ 
> patches/patch-jdk_make_tools_src_build_tools_generatecurrencydata_GenerateCurrencyData_java
>        31 Dec 2014 02:21:00 -0000
> @@ -0,0 +1,13 @@
> +$OpenBSD$
> +--- 
> jdk/make/tools/src/build/tools/generatecurrencydata/GenerateCurrencyData.java.orig
>        Tue Dec 30 16:33:52 2014
> ++++ 
> jdk/make/tools/src/build/tools/generatecurrencydata/GenerateCurrencyData.java 
>    Tue Dec 30 19:20:43 2014
> +@@ -281,9 +281,6 @@ public class GenerateCurrencyData {
> +             checkCurrencyCode(newCurrency);
> +             String timeString = currencyInfo.substring(4, length - 4);
> +             long time = format.parse(timeString).getTime();
> +-            if (Math.abs(time - System.currentTimeMillis()) > ((long) 10) * 
> 365 * 24 * 60 * 60 * 1000) {
> +-                throw new RuntimeException("time is more than 10 years from 
> present: " + time);
> +-            }
> +             specialCaseCutOverTimes[specialCaseCount] = time;
> +             specialCaseOldCurrencies[specialCaseCount] = oldCurrency;
> +             specialCaseOldCurrenciesDefaultFractionDigits[specialCaseCount] 
> = getDefaultFractionDigits(oldCurrency);
okay! that check is idiotic

Reply via email to