On Fri, Jun 26, 2020 at 10:26 AM Alexander Kanavin <[email protected]> wrote: > > 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
Agreed, too many changes to take for LTS. I think I'm just going to live with the occasional autobuilder failure. This seems like way more work that it is worth :-) Steve > On Fri, 26 Jun 2020 at 21:35, Steve Sakoman <[email protected]> wrote: >> >> On Fri, Jun 26, 2020 at 8:56 AM Steve Sakoman via >> lists.openembedded.org <[email protected]> >> wrote: >> > >> > On Fri, Jun 26, 2020 at 8:48 AM Alexander Kanavin >> > <[email protected]> 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 <[email protected]> wrote: >> > >> >> > >> On Fri, Jun 26, 2020 at 5:49 AM Ross Burton <[email protected]> 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 <[email protected]> wrote: >> > >> > > >> > >> > > On Thu, Jun 25, 2020 at 6:23 AM Steve Sakoman <[email protected]> >> > >> > > wrote: >> > >> > > > >> > >> > > > On Thu, Jun 25, 2020 at 6:09 AM Alexander Kanavin >> > >> > > > <[email protected]> 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 <[email protected]> >> > >> > > > > 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 >> > >> > > > >> <[email protected]> wrote: >> > >> > > > >> > >> > >> > > > >> > Both parts are rather convoluted, so it is better to have >> > >> > > > >> > them in a single file. >> > >> > > > >> > >> > >> > > > >> > Signed-off-by: Alexander Kanavin <[email protected]> >> > >> > > > >> > --- >> > >> > > > >> > 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 (#140034): https://lists.openembedded.org/g/openembedded-core/message/140034 Mute This Topic: https://lists.openembedded.org/mt/75069295/21656 Group Owner: [email protected] Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
