Technically, there are. I am not qualified to comment on how destabilizing they could be: http://site.icu-project.org/download/67
Perhaps it is possible to bisect the code to identify the fix that can be cherry picked: https://github.com/unicode-org/icu/releases/tag/release-67-1 Alex Alex On Fri, 26 Jun 2020 at 21:35, Steve Sakoman <st...@sakoman.com> wrote: > On Fri, Jun 26, 2020 at 8:56 AM Steve Sakoman via > lists.openembedded.org <steve=sakoman....@lists.openembedded.org> > wrote: > > > > On Fri, Jun 26, 2020 at 8:48 AM Alexander Kanavin > > <alex.kana...@gmail.com> wrote: > > > > > > Can you, for the sake of testing, upgrade icu to the same version as > master? Those failures do not seem to be happening there. > > > > Yes, I'll give that a try with a local build first since the > > autobuilder is undergoing normal maintenance. > > The local build succeeded, so the failure does seem to be related to > the earlier version. > > Is an upgrade to the new version reasonable for dunfell, or are there > feature additions that would make it unsuitable? > > Steve > > > > On Fri, 26 Jun 2020 at 17:59, Steve Sakoman <st...@sakoman.com> wrote: > > >> > > >> On Fri, Jun 26, 2020 at 5:49 AM Ross Burton <r...@burtonini.com> > wrote: > > >> > > > >> > Failures in edgerouter and MIPS screams the big-endian thing again. > > >> > ICU is a disaster here, be warned. > > >> > > >> Good theory -- ppc failed too. > > >> > > >> Steve > > >> > > >> > On Fri, 26 Jun 2020 at 16:27, Steve Sakoman <st...@sakoman.com> > wrote: > > >> > > > > >> > > On Thu, Jun 25, 2020 at 6:23 AM Steve Sakoman <st...@sakoman.com> > wrote: > > >> > > > > > >> > > > On Thu, Jun 25, 2020 at 6:09 AM Alexander Kanavin > > >> > > > <alex.kana...@gmail.com> wrote: > > >> > > > > > > >> > > > > I think you may have to recreate them by hand. > > >> > > > > > >> > > > OK, I'll put together a patch for the 66.1 recipe in dunfell > that > > >> > > > implements the changes you have in the second patch of the > series. In > > >> > > > the interest of minimizing changes I'll keep the .inc file > separate as > > >> > > > is. > > >> > > > > >> > > I recreated the patch for 66.1 and did a couple of quick local > builds > > >> > > and things looked fine. > > >> > > > > >> > > On the autobuilder I get failures for edgerouter and mips. I > seem to > > >> > > recall some discussion about this (perhaps on irc?) but can only > find > > >> > > one thread here on the list back when Khem's original patch went > in. > > >> > > > > >> > > Did you recall any issues like this? > > >> > > > > >> > > Steve > > >> > > > > >> > > > > >> > > > > On Thu, 25 Jun 2020 at 18:04, Steve Sakoman < > st...@sakoman.com> wrote: > > >> > > > >> > > >> > > > >> Is this patch series suitable for dunfell as-is (the concern > being the > > >> > > > >> contents of the version bump) or would a backport to 66.1 be > > >> > > > >> necessary? > > >> > > > >> > > >> > > > >> Steve > > >> > > > >> > > >> > > > >> On Tue, Jun 23, 2020 at 11:09 AM Alexander Kanavin > > >> > > > >> <alex.kana...@gmail.com> wrote: > > >> > > > >> > > > >> > > > >> > Both parts are rather convoluted, so it is better to have > them in a single file. > > >> > > > >> > > > >> > > > >> > Signed-off-by: Alexander Kanavin <alex.kana...@gmail.com> > > >> > > > >> > --- > > >> > > > >> > meta/recipes-support/icu/icu.inc | 84 > --------------------------- > > >> > > > >> > meta/recipes-support/icu/icu_67.1.bb | 85 > +++++++++++++++++++++++++++- > > >> > > > >> > 2 files changed, 84 insertions(+), 85 deletions(-) > > >> > > > >> > delete mode 100644 meta/recipes-support/icu/icu.inc > > >> > > > >> > > > >> > > > >> > diff --git a/meta/recipes-support/icu/icu.inc > b/meta/recipes-support/icu/icu.inc > > >> > > > >> > deleted file mode 100644 > > >> > > > >> > index 6d9728903d..0000000000 > > >> > > > >> > --- a/meta/recipes-support/icu/icu.inc > > >> > > > >> > +++ /dev/null > > >> > > > >> > @@ -1,84 +0,0 @@ > > >> > > > >> > -SUMMARY = "International Component for Unicode libraries" > > >> > > > >> > -DESCRIPTION = "The International Component for Unicode > (ICU) is a mature, \ > > >> > > > >> > -portable set of C/C++ and Java libraries for Unicode > support, software \ > > >> > > > >> > -internationalization (I18N) and globalization (G11N), > giving applications the \ > > >> > > > >> > -same results on all platforms." > > >> > > > >> > -HOMEPAGE = "http://site.icu-project.org/"; > > >> > > > >> > - > > >> > > > >> > -LICENSE = "ICU" > > >> > > > >> > -DEPENDS = "icu-native" > > >> > > > >> > -DEPENDS_class-native = "" > > >> > > > >> > - > > >> > > > >> > -CVE_PRODUCT = "international_components_for_unicode" > > >> > > > >> > - > > >> > > > >> > -S = "${WORKDIR}/icu/source" > > >> > > > >> > -SPDX_S = "${WORKDIR}/icu" > > >> > > > >> > -STAGING_ICU_DIR_NATIVE = > "${STAGING_DATADIR_NATIVE}/${BPN}/${PV}" > > >> > > > >> > - > > >> > > > >> > -BINCONFIG = "${bindir}/icu-config" > > >> > > > >> > - > > >> > > > >> > -ICU_MAJOR_VER = "${@d.getVar('PV').split('.')[0]}" > > >> > > > >> > - > > >> > > > >> > -inherit autotools pkgconfig binconfig multilib_script > > >> > > > >> > - > > >> > > > >> > -MULTILIB_SCRIPTS = "${PN}-dev:${bindir}/icu-config" > > >> > > > >> > - > > >> > > > >> > -# ICU needs the native build directory as an argument to > its --with-cross-build option when > > >> > > > >> > -# cross-compiling. Taken the situation that different > builds may share a common sstate-cache > > >> > > > >> > -# into consideration, the native build directory needs to > be staged. > > >> > > > >> > -EXTRA_OECONF = > "--with-cross-build=${STAGING_ICU_DIR_NATIVE}" > > >> > > > >> > -EXTRA_OECONF_class-native = "" > > >> > > > >> > -EXTRA_OECONF_class-nativesdk = > "--with-cross-build=${STAGING_ICU_DIR_NATIVE}" > > >> > > > >> > - > > >> > > > >> > -EXTRA_OECONF_append_class-target = > "${@oe.utils.conditional('SITEINFO_ENDIANNESS', 'be', ' > --with-data-packaging=archive', '', d)}" > > >> > > > >> > -TARGET_CXXFLAGS_append = > "${@oe.utils.conditional('SITEINFO_ENDIANNESS', 'be', ' > -DICU_DATA_DIR=\\""${datadir}/${BPN}/${PV}\\""', '', d)}" > > >> > > > >> > - > > >> > > > >> > -ASNEEDED = "" > > >> > > > >> > - > > >> > > > >> > -do_compile_prepend_class-target () { > > >> > > > >> > - # Make sure certain build host references do not > end up being compiled > > >> > > > >> > - # in the image. This only affects libicutu and > icu-dbg > > >> > > > >> > - sed \ > > >> > > > >> > - -e 's,DU_BUILD=,DU_BUILD_unused=,g' \ > > >> > > > >> > - -e '/^CPPFLAGS.*/ > s,--sysroot=${STAGING_DIR_TARGET},,g' \ > > >> > > > >> > - -i ${B}/tools/toolutil/Makefile > > >> > > > >> > -} > > >> > > > >> > - > > >> > > > >> > -PREPROCESS_RELOCATE_DIRS = "${datadir}/${BPN}/${PV}" > > >> > > > >> > -do_install_append_class-native() { > > >> > > > >> > - mkdir -p ${D}/${STAGING_ICU_DIR_NATIVE}/config > > >> > > > >> > - cp -r ${B}/config/icucross.mk > ${D}/${STAGING_ICU_DIR_NATIVE}/config > > >> > > > >> > - cp -r ${B}/config/icucross.inc > ${D}/${STAGING_ICU_DIR_NATIVE}/config > > >> > > > >> > - cp -r ${B}/lib ${D}/${STAGING_ICU_DIR_NATIVE} > > >> > > > >> > - cp -r ${B}/bin ${D}/${STAGING_ICU_DIR_NATIVE} > > >> > > > >> > - cp -r ${B}/tools ${D}/${STAGING_ICU_DIR_NATIVE} > > >> > > > >> > -} > > >> > > > >> > - > > >> > > > >> > -do_install_append_class-target() { > > >> > > > >> > - # The native pkgdata can not generate the correct > data file. > > >> > > > >> > - # Use icupkg to re-generate it. > > >> > > > >> > - if [ "${SITEINFO_ENDIANNESS}" = "be" ] ; then > > >> > > > >> > - rm -f > ${D}/${datadir}/${BPN}/${PV}/icudt${ICU_MAJOR_VER}b.dat > > >> > > > >> > - icupkg -tb > ${S}/data/in/icudt${ICU_MAJOR_VER}l.dat > ${D}/${datadir}/${BPN}/${PV}/icudt${ICU_MAJOR_VER}b.dat > > >> > > > >> > - fi > > >> > > > >> > - > > >> > > > >> > - # Remove build host references... > > >> > > > >> > - sed -i \ > > >> > > > >> > - -e 's,--sysroot=${STAGING_DIR_TARGET},,g' \ > > >> > > > >> > - -e 's|${DEBUG_PREFIX_MAP}||g' \ > > >> > > > >> > - -e 's:${HOSTTOOLS_DIR}/::g' \ > > >> > > > >> > - ${D}/${bindir}/icu-config > ${D}/${libdir}/${BPN}/${PV}/Makefile.inc \ > > >> > > > >> > - ${D}/${libdir}/${BPN}/${PV}/pkgdata.inc > > >> > > > >> > -} > > >> > > > >> > - > > >> > > > >> > -PACKAGES =+ "libicudata libicuuc libicui18n libicutu > libicuio" > > >> > > > >> > - > > >> > > > >> > -FILES_${PN}-dev += "${libdir}/${BPN}/" > > >> > > > >> > - > > >> > > > >> > -FILES_libicudata = "${libdir}/libicudata.so.*" > > >> > > > >> > -FILES_libicuuc = "${libdir}/libicuuc.so.*" > > >> > > > >> > -FILES_libicui18n = "${libdir}/libicui18n.so.*" > > >> > > > >> > -FILES_libicutu = "${libdir}/libicutu.so.*" > > >> > > > >> > -FILES_libicuio = "${libdir}/libicuio.so.*" > > >> > > > >> > - > > >> > > > >> > -BBCLASSEXTEND = "native nativesdk" > > >> > > > >> > diff --git a/meta/recipes-support/icu/icu_67.1.bb > b/meta/recipes-support/icu/icu_67.1.bb > > >> > > > >> > index 58638507c9..5814fa41bd 100644 > > >> > > > >> > --- a/meta/recipes-support/icu/icu_67.1.bb > > >> > > > >> > +++ b/meta/recipes-support/icu/icu_67.1.bb > > >> > > > >> > @@ -1,4 +1,87 @@ > > >> > > > >> > -require icu.inc > > >> > > > >> > +SUMMARY = "International Component for Unicode libraries" > > >> > > > >> > +DESCRIPTION = "The International Component for Unicode > (ICU) is a mature, \ > > >> > > > >> > +portable set of C/C++ and Java libraries for Unicode > support, software \ > > >> > > > >> > +internationalization (I18N) and globalization (G11N), > giving applications the \ > > >> > > > >> > +same results on all platforms." > > >> > > > >> > +HOMEPAGE = "http://site.icu-project.org/"; > > >> > > > >> > + > > >> > > > >> > +LICENSE = "ICU" > > >> > > > >> > +DEPENDS = "icu-native" > > >> > > > >> > +DEPENDS_class-native = "" > > >> > > > >> > + > > >> > > > >> > +CVE_PRODUCT = "international_components_for_unicode" > > >> > > > >> > + > > >> > > > >> > +S = "${WORKDIR}/icu/source" > > >> > > > >> > +SPDX_S = "${WORKDIR}/icu" > > >> > > > >> > +STAGING_ICU_DIR_NATIVE = > "${STAGING_DATADIR_NATIVE}/${BPN}/${PV}" > > >> > > > >> > + > > >> > > > >> > +BINCONFIG = "${bindir}/icu-config" > > >> > > > >> > + > > >> > > > >> > +ICU_MAJOR_VER = "${@d.getVar('PV').split('.')[0]}" > > >> > > > >> > + > > >> > > > >> > +inherit autotools pkgconfig binconfig multilib_script > > >> > > > >> > + > > >> > > > >> > +MULTILIB_SCRIPTS = "${PN}-dev:${bindir}/icu-config" > > >> > > > >> > + > > >> > > > >> > +# ICU needs the native build directory as an argument to > its --with-cross-build option when > > >> > > > >> > +# cross-compiling. Taken the situation that different > builds may share a common sstate-cache > > >> > > > >> > +# into consideration, the native build directory needs to > be staged. > > >> > > > >> > +EXTRA_OECONF = > "--with-cross-build=${STAGING_ICU_DIR_NATIVE}" > > >> > > > >> > +EXTRA_OECONF_class-native = "" > > >> > > > >> > +EXTRA_OECONF_class-nativesdk = > "--with-cross-build=${STAGING_ICU_DIR_NATIVE}" > > >> > > > >> > + > > >> > > > >> > +EXTRA_OECONF_append_class-target = > "${@oe.utils.conditional('SITEINFO_ENDIANNESS', 'be', ' > --with-data-packaging=archive', '', d)}" > > >> > > > >> > +TARGET_CXXFLAGS_append = > "${@oe.utils.conditional('SITEINFO_ENDIANNESS', 'be', ' > -DICU_DATA_DIR=\\""${datadir}/${BPN}/${PV}\\""', '', d)}" > > >> > > > >> > + > > >> > > > >> > +ASNEEDED = "" > > >> > > > >> > + > > >> > > > >> > +do_compile_prepend_class-target () { > > >> > > > >> > + # Make sure certain build host references do not > end up being compiled > > >> > > > >> > + # in the image. This only affects libicutu and > icu-dbg > > >> > > > >> > + sed \ > > >> > > > >> > + -e 's,DU_BUILD=,DU_BUILD_unused=,g' \ > > >> > > > >> > + -e '/^CPPFLAGS.*/ > s,--sysroot=${STAGING_DIR_TARGET},,g' \ > > >> > > > >> > + -i ${B}/tools/toolutil/Makefile > > >> > > > >> > +} > > >> > > > >> > + > > >> > > > >> > +PREPROCESS_RELOCATE_DIRS = "${datadir}/${BPN}/${PV}" > > >> > > > >> > +do_install_append_class-native() { > > >> > > > >> > + mkdir -p ${D}/${STAGING_ICU_DIR_NATIVE}/config > > >> > > > >> > + cp -r ${B}/config/icucross.mk > ${D}/${STAGING_ICU_DIR_NATIVE}/config > > >> > > > >> > + cp -r ${B}/config/icucross.inc > ${D}/${STAGING_ICU_DIR_NATIVE}/config > > >> > > > >> > + cp -r ${B}/lib ${D}/${STAGING_ICU_DIR_NATIVE} > > >> > > > >> > + cp -r ${B}/bin ${D}/${STAGING_ICU_DIR_NATIVE} > > >> > > > >> > + cp -r ${B}/tools ${D}/${STAGING_ICU_DIR_NATIVE} > > >> > > > >> > +} > > >> > > > >> > + > > >> > > > >> > +do_install_append_class-target() { > > >> > > > >> > + # The native pkgdata can not generate the correct > data file. > > >> > > > >> > + # Use icupkg to re-generate it. > > >> > > > >> > + if [ "${SITEINFO_ENDIANNESS}" = "be" ] ; then > > >> > > > >> > + rm -f > ${D}/${datadir}/${BPN}/${PV}/icudt${ICU_MAJOR_VER}b.dat > > >> > > > >> > + icupkg -tb > ${S}/data/in/icudt${ICU_MAJOR_VER}l.dat > ${D}/${datadir}/${BPN}/${PV}/icudt${ICU_MAJOR_VER}b.dat > > >> > > > >> > + fi > > >> > > > >> > + > > >> > > > >> > + # Remove build host references... > > >> > > > >> > + sed -i \ > > >> > > > >> > + -e 's,--sysroot=${STAGING_DIR_TARGET},,g' \ > > >> > > > >> > + -e 's|${DEBUG_PREFIX_MAP}||g' \ > > >> > > > >> > + -e 's:${HOSTTOOLS_DIR}/::g' \ > > >> > > > >> > + ${D}/${bindir}/icu-config > ${D}/${libdir}/${BPN}/${PV}/Makefile.inc \ > > >> > > > >> > + ${D}/${libdir}/${BPN}/${PV}/pkgdata.inc > > >> > > > >> > +} > > >> > > > >> > + > > >> > > > >> > +PACKAGES =+ "libicudata libicuuc libicui18n libicutu > libicuio" > > >> > > > >> > + > > >> > > > >> > +FILES_${PN}-dev += "${libdir}/${BPN}/" > > >> > > > >> > + > > >> > > > >> > +FILES_libicudata = "${libdir}/libicudata.so.*" > > >> > > > >> > +FILES_libicuuc = "${libdir}/libicuuc.so.*" > > >> > > > >> > +FILES_libicui18n = "${libdir}/libicui18n.so.*" > > >> > > > >> > +FILES_libicutu = "${libdir}/libicutu.so.*" > > >> > > > >> > +FILES_libicuio = "${libdir}/libicuio.so.*" > > >> > > > >> > + > > >> > > > >> > +BBCLASSEXTEND = "native nativesdk" > > >> > > > >> > > > >> > > > >> > LIC_FILES_CHKSUM = > "file://../LICENSE;md5=a3808a5b70071b07f87ff2205e4d75a0" > > >> > > > >> > > > >> > > > >> > -- > > >> > > > >> > 2.27.0 > > >> > > > >> > > > >> > > > >> > > > >> > > > > >
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#140033): https://lists.openembedded.org/g/openembedded-core/message/140033 Mute This Topic: https://lists.openembedded.org/mt/75069295/21656 Group Owner: openembedded-core+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
