Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 759202 - gnustep-base/gnustep-base[icu] fails to compile with dev-libs/icu-68
Summary: gnustep-base/gnustep-base[icu] fails to compile with dev-libs/icu-68
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo Gnustep project
URL:
Whiteboard:
Keywords: PATCH
Depends on:
Blocks: icu-68.1
  Show dependency tree
 
Reported: 2020-12-09 10:20 UTC by Luis Cáceres
Modified: 2020-12-21 10:56 UTC (History)
1 user (show)

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


Attachments
patch for gnustep-base-1.27.0 (gnustep-base-1.27.0-define-FALSE-and-TRUE-for-icu-68.patch,388 bytes, patch)
2020-12-09 10:20 UTC, Luis Cáceres
Details | Diff
patch for gnustep-gui-0.28 (gnustep-gui-true-false.patch,386 bytes, patch)
2020-12-10 06:44 UTC, krator44
Details | Diff
patch for gnustep-gui-0.28 CORRECTION (gnustep-gui-true-false-FIXED.patch,386 bytes, patch)
2020-12-10 07:22 UTC, krator44
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Luis Cáceres 2020-12-09 10:20:59 UTC
Created attachment 677446 [details, diff]
patch for gnustep-base-1.27.0

ICU 68 public headers no longer defines macros TRUE and FALSE (see in https://unicode-org.atlassian.net/browse/ICU-21267 and http://site.icu-project.org/download/68). This leads to a compilation failure.

I'm attaching a simple patch that fixes this for me on gnustep-base-1.27.0.
Comment 1 krator44 2020-12-10 06:44:19 UTC
Created attachment 677533 [details, diff]
patch for gnustep-gui-0.28

attaching equivalent patch for gnustep-gui-0.28
Comment 2 krator44 2020-12-10 07:22:20 UTC
Created attachment 677536 [details, diff]
patch for gnustep-gui-0.28 CORRECTION

sorry about that the patch contained a typo
attaching fixed patch
Comment 3 PrSo 2020-12-10 16:08:31 UTC
I can confirm, that with the first patch applied gnustep-base builds just fine.
Thanks.
P.
Comment 4 Bernard Cafarelli gentoo-dev 2020-12-10 17:01:34 UTC
Thanks for the report, I delayed icu bump for now locally so did not see this issue yet.

Upstream, I see -base has fix already, but -gui has apparently not, did you send it there?

I will check and apply patches on ebuilds
Comment 5 PrSo 2020-12-10 19:32:13 UTC
(In reply to Bernard Cafarelli from comment #4)
> Thanks for the report, I delayed icu bump for now locally so did not see
> this issue yet.
> 
> Upstream, I see -base has fix already, but -gui has apparently not, did you
> send it there?
> 
> I will check and apply patches on ebuilds

No, it wasn't me who send it upstream. I've checked only on my local system, but I'm glad to hear that fix is already there.
Best,
P.
Comment 6 Bernard Cafarelli gentoo-dev 2020-12-10 20:50:08 UTC
Thanks for that, and gnustep apps seem to be OK with bumped icu (still running on my system for other packages).

I only added fixes to latest versions, which are overdue for stabling anyway
Comment 7 Larry the Git Cow gentoo-dev 2020-12-10 20:50:36 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=89f1225543db769277015aab351e720d4a0b31ca

commit 89f1225543db769277015aab351e720d4a0b31ca
Author:     Bernard Cafarelli <voyageur@gentoo.org>
AuthorDate: 2020-12-10 20:50:26 +0000
Commit:     Bernard Cafarelli <voyageur@gentoo.org>
CommitDate: 2020-12-10 20:50:34 +0000

    gnustep-base/gnustep-gui: fix compilation with dev-libs/icu-68
    
    Thanks Luis Cáceres <lacaceres97@gmail.com> for bug report and patch
    
    Closes: https://bugs.gentoo.org/759202
    Package-Manager: Portage-3.0.12, Repoman-3.0.2
    Signed-off-by: Bernard Cafarelli <voyageur@gentoo.org>

 .../gnustep-gui/files/gnustep-gui-0.28.0-icu68.patch      | 15 +++++++++++++++
 gnustep-base/gnustep-gui/gnustep-gui-0.28.0.ebuild        |  2 ++
 2 files changed, 17 insertions(+)

Additionally, it has been referenced in the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=091ceaaa1be63d498cff9f287f1f60028aeba22e

commit 091ceaaa1be63d498cff9f287f1f60028aeba22e
Author:     Bernard Cafarelli <voyageur@gentoo.org>
AuthorDate: 2020-12-10 20:40:45 +0000
Commit:     Bernard Cafarelli <voyageur@gentoo.org>
CommitDate: 2020-12-10 20:50:34 +0000

    gnustep-base/gnustep-base: fix compilation with dev-libs/icu-68
    
    Bug: https://bugs.gentoo.org/759202
    Package-Manager: Portage-3.0.12, Repoman-3.0.2
    Signed-off-by: Bernard Cafarelli <voyageur@gentoo.org>

 .../files/gnustep-base-1.27.0-icu68.patch          | 31 ++++++++++++++++++++++
 .../gnustep-base/gnustep-base-1.27.0.ebuild        |  5 +++-
 2 files changed, 35 insertions(+), 1 deletion(-)