Opened 6 years ago

Closed 5 years ago

#56754 closed update (fixed)

icu on MacPorts quite outdated: current version 63.1

Reported by: kencu (Ken) Owned by: ryandesign (Ryan Carsten Schmidt)
Priority: Normal Milestone:
Component: ports Version:
Keywords: Cc: yan12125 (Chih-Hsuan Yen)
Port: icu

Description (last modified by kencu (Ken))

The icu Portfile says to not update it due to c++11 needs.

The current version as of today is 63.1, ours is at 58.2

I came across one of my ports that can’t be updated any further due to the version of icu being too old.

I suppose soon we’re going to need to bite the bullet and push icu forward.

Attachments (1)

patch-icu-update-to-631.diff (35.3 KB) - added by kencu (Ken) 6 years ago.

Download all attachments as: .zip

Change History (10)

comment:1 Changed 6 years ago by ryandesign (Ryan Carsten Schmidt)

comment:2 Changed 6 years ago by kencu (Ken)

Ah, somehow I had forgot about that PR. Thanks.

(Aside: I searched for any tickets against ICU before posting this, but of course the PRs don't show up. I find the PR queue hard to search. One of the downsides of having discussions in two different systems. )

comment:3 Changed 6 years ago by kencu (Ken)

Description: modified (diff)
Summary: icu on MacPorts quite outdated: current version 62.1icu on MacPorts quite outdated: current version 63.1

comment:4 Changed 6 years ago by kencu (Ken)

Here's a patch that accomplished most of the work. Still to do is blacklist gcc versions less than 7 (as there is a math error on older systems with all gcc versions prior to 7 wherein all the std::math functions are excluded). Also, need to sort out what to do about universal builds as gcc can't build cross arch universal with the current MacPorts infrastructure.

Changed 6 years ago by kencu (Ken)

comment:5 Changed 5 years ago by kencu (Ken)

Add to the above that there seems to be no gcc at present that can build icu on 10.4 Intel.

comment:6 Changed 5 years ago by yan12125 (Chih-Hsuan Yen)

Cc: yan12125 added

This issue is blocking upgrading Qt 5 to 5.13 (1).

How about making libxml2-bootstrap independent of ICU as shown in my patch (2)? I didn't test it as building it requires libstdc++, which is missing from new systems.

(1) ticket:58841

(2) https://github.com/yan12125/macports-ports/commit/9a5eb3e6e342aaee278595e6667d24194b77a0f8

comment:7 Changed 5 years ago by jmroot (Joshua Root)

Why couldn't icu-bootstrap be left at the older version?

comment:8 in reply to:  7 Changed 5 years ago by yan12125 (Chih-Hsuan Yen)

Replying to jmroot:

Why couldn't icu-bootstrap be left at the older version?

That sounds a better solution!

The only potential issue out of my head is that a future libxml2 version might not be compatible with old icu. Though I bet it will not occur in a few years.

comment:9 Changed 5 years ago by MarcusCalhoun-Lopez (Marcus Calhoun-Lopez)

Resolution: fixed
Status: assignedclosed

In 8cf9b206c3c2fcca0e0f6ee2a46f47bc828a3cb7/macports-ports (master):

icu: update version 58.2->65.1

Fixes #56754
Fixes #58353

Note: See TracTickets for help on using tickets.