[MacPorts] #67574: dia @0.97.3_6 is broken by icu update
MacPorts
noreply at macports.org
Fri Jun 30 01:32:53 UTC 2023
#67574: dia @0.97.3_6 is broken by icu update
------------------------+-----------------------
Reporter: fhgwright | Owner: fhgwright
Type: defect | Status: closed
Priority: Normal | Milestone:
Component: ports | Version: 2.8.1
Resolution: fixed | Keywords:
Port: dia |
------------------------+-----------------------
Comment (by fhgwright):
Replying to [comment:5 ryandesign]:
> Replying to [comment:4 fhgwright]:
> > Given that it doesn't directly call `icu`, the following seems
suspect:
> > {{{
> > # required for for ICU
> > compiler.cxx_standard 2011
> > configure.cxxflags-append -std=c++11
> > }}}
>
> As I recall, the ICU headers require C++11 or newer so anything
including those headers needs at least a C++11 compiler and to be
compiling in C++11 mode or newer too. So these lines are correct and
should not be removed.
Except that `dia` doesn't use the `icu` headers. I verified that `dia`
can build with the `icu` headers (`/opt/local/include/unicode/`) renamed
out of sight (but not with `icu` deactivated, since that breaks
`xsltproc`).
The C++11 requirement was added by commit
8cf9b206c3c2fcca0e0f6ee2a46f47bc828a3cb7, which modified 64 files for an
`icu` update. Maybe `dia` was a victim of "human overlinking".
I left it alone based on the risk vs. benefit of changing it, not because
I think that's the most correct result.
--
Ticket URL: <https://trac.macports.org/ticket/67574#comment:7>
MacPorts <https://www.macports.org/>
Ports system for macOS
More information about the macports-tickets
mailing list