Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 405965 - Please stabilize dev-libs/icu-4.8.1.1-r1
Summary: Please stabilize dev-libs/icu-4.8.1.1-r1
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Library (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Jesus Rivero (RETIRED)
URL:
Whiteboard:
Keywords: STABLEREQ
Depends on:
Blocks: 396421
  Show dependency tree
 
Reported: 2012-02-26 23:02 UTC by Andreas K. Hüttel
Modified: 2012-04-01 14:46 UTC (History)
3 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
fixed ebuild without using (not longer needed?) lte patch (icu-4.8.1.1-r1.ebuild,2.36 KB, text/plain)
2012-02-27 20:51 UTC, Toralf Goaltenner
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Andreas K. Hüttel archtester gentoo-dev 2012-02-26 23:02:07 UTC
Hi, 

icu-4.8.1 introduces a regression (crash) in libreoffice calc, see bug 396421. Can we please get icu-4.8.1.1 stable soon at least on the libreoffice-relevant arches (amd64, ppc, x86)?

If you agree please set keyword and cc arches... TIA
Comment 1 Tomáš Chvátal (RETIRED) gentoo-dev 2012-02-27 16:21:55 UTC
As per summary please stabilise icu-4.8.1.1-r1.
Comment 2 Maksym 2012-02-27 17:59:14 UTC
>>> Failed to emerge dev-libs/icu-4.8.1.1-r1, Log file:

>>>  '/var/tmp/portage/dev-libs/icu-4.8.1.1-r1/temp/build.log'

 * Messages for package dev-libs/icu-4.8.1.1-r1:

 * Failed Patch: icu-4.8.1-fix_ltr.patch !
 *  ( /usr/portage/dev-libs/icu/files/icu-4.8.1-fix_ltr.patch )
 * 
 * Include in your bugreport the contents of:
 * 
 *   /var/tmp/portage/dev-libs/icu-4.8.1.1-r1/temp/icu-4.8.1-fix_ltr.patch.out
 * ERROR: dev-libs/icu-4.8.1.1-r1 failed (prepare phase):
 *   Failed Patch: icu-4.8.1-fix_ltr.patch!
 * 
 * Call stack:
 *     ebuild.sh, line   85:  Called src_prepare
 *   environment, line 2223:  Called epatch '/usr/portage/dev-libs/icu/files/icu-4.8.1-fix_binformat_fonts.patch' '/usr/portage/dev-libs/icu/files/icu-4.8.1-fix_nan.patch' '/usr/portage/dev-libs/icu/files/icu-4.8.1-fix_ltr.patch'
 *   environment, line  881:  Called epatch '/usr/portage/dev-libs/icu/files/icu-4.8.1-fix_ltr.patch'
 *   environment, line 1039:  Called die
 * The specific snippet of code:
 *               die "Failed Patch: ${patchname}!";
 * 
 * If you need support, post the output of 'emerge --info =dev-libs/icu-4.8.1.1-r1',
 * the complete build log and the output of 'emerge -pqv =dev-libs/icu-4.8.1.1-r1'.
 * The complete build log is located at '/var/tmp/portage/dev-libs/icu-4.8.1.1-r1/temp/build.log'.
 * The ebuild environment file is located at '/var/tmp/portage/dev-libs/icu-4.8.1.1-r1/temp/environment'.
 * S: '/var/tmp/portage/dev-libs/icu-4.8.1.1-r1/work/icu/source'


Patch command output: http://pastebin.com/hY0vkWXR
Comment 3 Mark Nowiasz 2012-02-27 19:09:49 UTC
(In reply to comment #2)
> >>> Failed to emerge dev-libs/icu-4.8.1.1-r1, Log file:

Same here (amd64).
Comment 4 Toralf Goaltenner 2012-02-27 20:48:35 UTC
I think the patch is not needed any longer as the new LESwaps.h is fixed by checking if BYTE_ORDER eq. BIG_ENDIAN.

Cause of this changes the patch does not fit any more and blocks the emerge process.

I created local portage overlay, removed the patch from the ebuild and it compiles fine.

I will attach the "fixed" ebuild.
Comment 5 Toralf Goaltenner 2012-02-27 20:51:39 UTC
Created attachment 303539 [details]
fixed ebuild without using (not longer needed?) lte patch

I didn't want to use new version number, this shall be task of maintainer. In my eyes new source code seems to handle LTE problem and makes patch obsolete. But I didn't test in detail, just read the code.
Comment 6 Ian Delaney (RETIRED) gentoo-dev 2012-02-28 10:43:19 UTC
amd64:

archtester icu # ebuild icu-4.8.1-r2.ebuild clean prepare

 * Applying icu-4.8.1-fix_binformat_fonts.patch ...                                                            [ ok ]
 * Applying icu-4.8.1-fix_nan.patch ...                                                                        [ ok ]

archtester icu # ebuild icu-4.8.1-r2.ebuild clean install
---------------------------------------------------------
 * QA Notice: Package triggers severe warnings which indicate that it
 *            may exhibit random runtime failures.
 * ExtensionSubtables.cpp:32:31: warning: dereferencing type-punned pointer will break strict-aliasing rules

 * Please do not file a Gentoo bug and instead report the above QA
 * issues directly to the upstream developers of this software.
 * Homepage: http://www.icu-project.org/

archtester icu # ebuild icu-4.8.1-r2.ebuild merge

>>> Original instance of package unmerged safely.
>>> dev-libs/icu-4.8.1-r2 merged.

do you want a bug filed for the QA?
Comment 7 Agostino Sarubbo gentoo-dev 2012-02-29 15:24:26 UTC
amd64 stable
Comment 8 Jeroen Roovers (RETIRED) gentoo-dev 2012-02-29 16:44:04 UTC
Stable for HPPA.
Comment 9 Brent Baude (RETIRED) gentoo-dev 2012-03-01 20:07:44 UTC
ppc done
Comment 10 Brent Baude (RETIRED) gentoo-dev 2012-03-02 21:26:45 UTC
ppc64 done
Comment 11 Dan Dexter 2012-03-07 04:24:30 UTC
Archtested on x86: Everything OK.

- icu and various rdeps compiled successfully.
- Of the rdeps that had associated tests, all tests passed.
- Repoman reports no ebuild QA issues
- Manually verified functionality using the latest stable chromium.
Comment 12 Paweł Hajdan, Jr. (RETIRED) gentoo-dev 2012-03-09 10:37:09 UTC
x86 stable
Comment 13 Markus Meier gentoo-dev 2012-03-15 21:58:33 UTC
arm stable
Comment 14 Raúl Porcel (RETIRED) gentoo-dev 2012-04-01 14:46:59 UTC
alpha/ia64/s390/sh/sparc stable