Bug 222433

Summary: devel/icu: drop C++98 support in consumers
Product: Ports & Packages Reporter: Jan Beich <jbeich>
Component: Individual Port(s)Assignee: FreeBSD Office Team <office>
Status: Closed FIXED    
Severity: Affects Only Me CC: w.schwarzenfeld
Priority: --- Keywords: patch
Version: Latest   
Hardware: Any   
OS: Any   
URL: https://ssl.icu-project.org/trac/ticket/12643
Bug Depends on: 218788, 222223, 222225, 222434, 222435    
Bug Blocks:    
Attachments:
Description Flags
ICU 59 version
none
ICU 60 version none

Description Jan Beich freebsd_committer freebsd_triage 2017-09-18 21:33:55 UTC
C++98 support currently relies on downstream patches. GCC already defaults to C++14, so this affects only Clang platforms. If not fixed it'd complicate future ICU updates by requiring to revert upstream commits e.g.,

https://ssl.icu-project.org/trac/changeset/40077
Comment 1 Jan Beich freebsd_committer freebsd_triage 2017-09-18 21:54:14 UTC
Created attachment 186520 [details]
ICU 59 version
Comment 2 Walter Schwarzenfeld freebsd_triage 2018-01-22 15:12:40 UTC
Makefile shows:
Version 60_2
patch is in the port.

I think this could closed.
Comment 3 Jan Beich freebsd_committer freebsd_triage 2018-02-06 16:36:35 UTC
Created attachment 190361 [details]
ICU 60 version
Comment 4 commit-hook freebsd_committer freebsd_triage 2018-02-09 09:13:48 UTC
A commit references this bug:

Author: jbeich
Date: Fri Feb  9 09:12:50 UTC 2018
New revision: 461290
URL: https://svnweb.freebsd.org/changeset/ports/461290

Log:
  devel/icu: drop C++98 compatibility workarounds

  Now that both USE_GCC and -CURRENT default to C++14 there's little
  reason to complicate maintenance. Revert r449685 to usher consumers
  into post-C++11 world.

  PR:		222433 222434 222435

Changes:
  head/databases/evolution-data-server/Makefile
  head/devel/icu/Makefile
  head/devel/icu/files/patch-common_unicode_platform.h
  head/devel/icu/files/patch-common_unicode_umachine.h
  head/devel/icu/files/patch-common_unicode_unistr.h
  head/devel/icu/files/patch-revert-r40077
  head/devel/icu-le-hb/Makefile
  head/devel/pecl-intl/Makefile
  head/devel/php70-intl/Makefile
  head/devel/php71-intl/Makefile
  head/devel/php72-intl/Makefile
  head/games/openttd/Makefile
  head/lang/phantomjs/files/patch-src_qt_qtbase_config.tests_unix_icu_icu.pro
  head/lang/v8-devel/Makefile
  head/misc/sword/Makefile
  head/print/tex-xetex/Makefile
  head/print/texlive-base/Makefile
  head/sysutils/gdisk/Makefile
  head/www/cppcms/Makefile