Ubuntu
icu package

icu FTBFS on lunar

Bug #2016021 reported by Simon Chopin
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
icu (Ubuntu)
Fix Released
High
Unassigned

Bug Description

During the last test rebuild, it was noticed that icu failed to build from source, with the following failures in the test suite:

-------------
| *** FAILING TEST SUMMARY FOR: intltest
         TestAliasedNames
         TestCanonicalID
      TimeZoneTest
   format
| *** END FAILING TEST SUMMARY FOR: intltest

The issue is related to the recent tzdata updates: not only the usual batch of changes made to reflect the current state of the world, but also changes made to have a better data set for pre-1970 dates.

Tags: ftbfs

Related branches

~liushuyu-011/ubuntu/+source/icu:ubuntu/devel
Julian Andres Klode (community): Approve
Ubuntu Sponsors: Pending requested
Ubuntu Sponsors: Pending requested
Andreas Hasenack: Pending requested
git-ubuntu import: Pending requested
Diff: 255 lines (+198/-1)
5 files modified
debian/changelog (+47/-0)
debian/control (+2/-1)
debian/patches/disable-precision-fpmath-tests-on-i386.patch (+123/-0)
debian/patches/series (+2/-0)
debian/patches/test-canonical-ids.patch (+24/-0)
~liushuyu-011/ubuntu/+source/icu:ubuntu/devel
Andreas Hasenack: Needs Information
Simon Quigley (community): Needs Fixing
Julian Andres Klode (community): Needs Fixing
git-ubuntu import: Pending requested
Diff: 81 lines (+40/-1) (has conflicts)
2 files modified
debian/changelog (+31/-0)
debian/rules (+9/-1)
Revision history for this message
Steven R. Loomis (srl-icu-project) wrote :

What ICU is this? I can't tell which version from here. Your analysis sounds reasonable, may have to ignore the tests (there's a logKnownIssue mechanism) - put the following before the failing test case:

if (logKnownIssue("https://bugs.launchpad.net/ubuntu/+source/icu/+bug/2016021", "tzdata issue")) {
  return;
}

Revision history for this message
Simon Chopin (schopin) wrote : Re: [Bug 2016021] Re: icu FTBFS on lunar

Quoting Steven R. Loomis (2023-04-12 17:30:25)
> What ICU is this? I can't tell which version from here. Your analysis

That'd be 72.1-3ubuntu1, now ubuntu2 with this patch. This bug was
mostly for reference for the folks in the Ubuntu release team since
we're in feature freeze, almost at final freeze. I uploaded the fix
directly afterward. That's my bad, I should have been more explicit
about that :)

> sounds reasonable, may have to ignore the tests (there's a logKnownIssue
> mechanism) - put the following before the failing test case:
>
> if (logKnownIssue("https://bugs.launchpad.net/ubuntu/+source/icu/+bug/2016021", "tzdata issue")) {
> return;
> }

Given that icu is a regular in our FTBFS reports due to tzdata updates,
we might indeed end up doing so, but in general I'd rather adapt the
tests to the specificities of the distro than disable them altogether.
Thanks for the tip though, much appreciated!

Revision history for this message
Steven R. Loomis (srl-icu-project) wrote :

OK makes sense. Tests that are 'skipped' as above are listed at the end of the test run (with the bug url) and can be optionally enabled.

(BTW ICU 73 is in RC and releases soon.)

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package icu - 72.1-3ubuntu2

---------------
icu (72.1-3ubuntu2) lunar; urgency=medium

  * d/p/test-canonical-ids.patch: update TestCanonicalIDs to match latest
    tzdata, including the de-aliasing due to pre-1970 data (LP: #2016021)

 -- Simon Chopin <email address hidden> Wed, 12 Apr 2023 17:02:53 +0200

Changed in icu (Ubuntu):
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.