package/icu: bump version to 60.1

PATCH New Bernd Kuhls 3 participants 5 messages Dec 3, 2017 – Dec 4, 2017

Share & tools

Bernd Kuhls

author

2017-12-03 19:41 UTC #0 lore raw

PATCH 1/1 package/icu: bump version to 60.1

Removed 0006-i18n-Drop-include-xlocale.h.patch, problem was fixed upstream with a different solution:
http://bugs.icu-project.org/trac/changeset/40603

Added upstream sha512 hash and license hash.

Signed-off-by Bernd Kuhls <[email protected]>

---
 package/icu/0006-i18n-Drop-include-xlocale.h.patch | 39 ----------------------
 package/icu/icu.hash                               |  7 ++--
 package/icu/icu.mk                                 |  2 +-
 3 files changed, 4 insertions(+), 44 deletions(-)
 delete mode 100644 package/icu/0006-i18n-Drop-include-xlocale.h.patch

package/icu/0006-i18n-Drop-include-xlocale.h.patch deleted +0 ~~-39~~

deleted file mode 100644
index a5eb9cb072..0000000000
@@ -1,39 +0,0 @@
-From 5f38e4f8c4be351db96a3be23ce25c97597cc508 Mon Sep 17 00:00:00 2001
-From: Khem Raj <[email protected]>
-Date: Mon, 2 Oct 2017 22:41:02 +0200
-Subject: [PATCH] i18n: Drop include <xlocale.h>
-
-glibc 2.26 drops this header
-
-Signed-off-by: Khem Raj <[email protected]>
-
-Reported Upstream: http://bugs.icu-project.org/trac/ticket/13385
-From: https://git.yoctoproject.org/cgit.cgi/poky/commit/meta/recipes-support/icu/icu/0001-i18n-Drop-include-xlocale.h.patch?id=931430f94c018bc3f58d70e11ca4a4ae7cb2e19c
-
-Signed-off-by: Romain Naour <[email protected]>
----
-Upstream-Status: Pending
----
- source/i18n/digitlst.cpp | 6 +-----
- 1 file changed, 1 insertion(+), 5 deletions(-)
-
-diff --git a/source/i18n/digitlst.cpp b/source/i18n/digitlst.cpp
-index 4d7c3f4..f111cf3 100644
---- a/source/i18n/digitlst.cpp
-+++ b/source/i18n/digitlst.cpp
-@@ -61,11 +61,7 @@
- #endif
- 
- #if U_USE_STRTOD_L
--# if U_PLATFORM_USES_ONLY_WIN32_API || U_PLATFORM == U_PF_CYGWIN
--#   include <locale.h>
--# else
--#   include <xlocale.h>
--# endif
-+# include <locale.h>
- #endif
- 
- // ***************************************************************************
--- 
-2.9.5
-

package/icu/icu.hash +3 -4

index d10a061322..8b54e22443 100644
@@ -1,4 +1,3 @@
-# From http://download.icu-project.org/files/icu4c/59.1/icu4c-src-59_1.md5
-md5 54923fa9fab5b2b83f235fb72523de37  icu4c-59_1-src.tgz
-# Calculated based on the hash above
-sha256 7132fdaf9379429d004005217f10e00b7d2319d0fea22bdfddef8991c45b75fe  icu4c-59_1-src.tgz
+# From http://download.icu-project.org/files/icu4c/60.1/SHASUM512.txt
+sha512 a6798f70add1b3bac2197a49e09a0bb636279af7019ce572f63a30ab713e09657ee9d9b20aac7ea806fbb84667ca2eca981411e5053b47c3c705aa496a669233  icu4c-60_1-src.tgz
+sha256 24c771ce37201a3ab604852e47d10d21595d0493269dd6c8a5e567e998e18c00  LICENSE

package/icu/icu.mk +1 -1

index 166890e080..6ac4a25378 100644
@@ -4,7 +4,7 @@
 #
 ################################################################################
 
-ICU_VERSION = 59.1
+ICU_VERSION = 60.1
 ICU_SOURCE = icu4c-$(subst .,_,$(ICU_VERSION))-src.tgz
 ICU_SITE = http://download.icu-project.org/files/icu4c/$(ICU_VERSION)
 ICU_LICENSE = ICU License

Signature · 1 line

2.11.0

Thomas Petazzoni

2017-12-03 20:00 UTC #1 lore raw

PATCH 1/1 package/icu: bump version to 60.1

Hello,

Quoted 7 lines · Bernd Kuhls

Removed 0006-i18n-Drop-include-xlocale.h.patch, problem was fixed upstream with a different solution:
http://bugs.icu-project.org/trac/changeset/40603
Added upstream sha512 hash and license hash.
Signed-off-by Bernd Kuhls <[email protected]>

Curious: how many packages that depend on ICU did you build? From past experience, each major ICU bump very often causes a number of build failures in other packages.

What are the changes between 59.1 and 60.1 ? Anything significant that has a chance of breaking packages that depend on ICU ?

Best regards,

Thomas

Signature · 3 lines

Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

Mario Lang

2017-12-03 22:01 UTC #2 lore raw

PATCH 1/1 package/icu: bump version to 60.1

Quoted 7 lines · Thomas Petazzoni

Hello,
Quoted 4 lines · Bernd Kuhls
Removed 0006-i18n-Drop-include-xlocale.h.patch, problem was fixed upstream with a different solution:
http://bugs.icu-project.org/trac/changeset/40603

Quoted 5 lines

Quoted 3 lines
Added upstream sha512 hash and license hash.
Signed-off-by Bernd Kuhls <[email protected]>
Curious: how many packages that depend on ICU did you build?

Just gave this a whirl. The following packages
at least build successfully with icu 60.1:

boost (locale), brltty (from patch-queue), dovecot, harfbuzz, xerces

Quoted 2 lines

From past experience, each major ICU bump very often causes a number of build failures in other packages.

None spotted so far.

Signature · 2 lines

CYa,
  ?????

Bernd Kuhls

author

2017-12-04 07:20 UTC #3 lore raw

PATCH 1/1 package/icu: bump version to 60.1

In reply to Thomas Petazzoni

Hi Thomas,

Am Sun, 03 Dec 2017 21:00:12 +0100 schrieb Thomas Petazzoni:

Quoted 3 lines

Curious: how many packages that depend on ICU did you build? From past experience, each major ICU bump very often causes a number of build failures in other packages.

this defconfig builds fine:

BR2_x86_64=y
BR2_x86_core2=y
BR2_TOOLCHAIN_EXTERNAL=y
BR2_TOOLCHAIN_EXTERNAL_DOWNLOAD=y BR2_TOOLCHAIN_EXTERNAL_URL="http://autobuild.buildroot.org/toolchains/ tarballs/br-x86-64-core2-full-2017.05-1078-g95b1dae.tar.bz2" BR2_TOOLCHAIN_EXTERNAL_GCC_6=y BR2_TOOLCHAIN_EXTERNAL_HEADERS_4_4=y BR2_TOOLCHAIN_EXTERNAL_LOCALE=y
# BR2_TOOLCHAIN_EXTERNAL_HAS_THREADS_DEBUG is not set BR2_TOOLCHAIN_EXTERNAL_CXX=y
BR2_PACKAGE_MPD=y
BR2_PACKAGE_MESA3D=y
BR2_PACKAGE_MESA3D_DRI_DRIVER_I965=y BR2_PACKAGE_MESA3D_OPENGL_EGL=y
BR2_PACKAGE_QT5=y
BR2_PACKAGE_XORG7=y
BR2_PACKAGE_NODEJS=y
BR2_PACKAGE_PHP=y
BR2_PACKAGE_PYTHON=y
BR2_PACKAGE_PYTHON_PYICU=y
BR2_PACKAGE_HARFBUZZ=y
BR2_PACKAGE_RAPTOR=y
BR2_PACKAGE_XERCES=y
BR2_PACKAGE_CPPCMS=y
BR2_PACKAGE_CPPCMS_ICU=y
BR2_PACKAGE_DOVECOT=y
BR2_PACKAGE_ZNC=y

Quoted 2 lines

What are the changes between 59.1 and 60.1 ? Anything significant that has a chance of breaking packages that depend on ICU ?

The release notes can be found here:
http://site.icu-project.org/download/60
and, afaics, they do not include a hint for breaking changes.

Regards, Bernd

Thomas Petazzoni

Applied

2017-12-04 10:33 UTC #4 lore raw

PATCH 1/1 package/icu: bump version to 60.1

In reply to Bernd Kuhls

Hello,

Quoted 6 lines · Bernd Kuhls

Removed 0006-i18n-Drop-include-xlocale.h.patch, problem was fixed upstream with a different solution:
http://bugs.icu-project.org/trac/changeset/40603
Added upstream sha512 hash and license hash.

Quoted 7 lines

Signed-off-by Bernd Kuhls <[email protected]>

---
 package/icu/0006-i18n-Drop-include-xlocale.h.patch | 39 ----------------------
 package/icu/icu.hash                               |  7 ++--
 package/icu/icu.mk                                 |  2 +-
 3 files changed, 4 insertions(+), 44 deletions(-)
 delete mode 100644 package/icu/0006-i18n-Drop-include-xlocale.h.patch

Thanks a lot Mario and Bernd for your feedback confirming that ICU 60.1 still works fine with its reverse dependencies.

I've applied the version bump to master, thanks!

Best regards,

Thomas

Signature · 3 lines

Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com