Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 5 Jul 2023 10:13:45 +0100
From:      Nuno Teixeira <eduardo@freebsd.org>
To:        Raphael Kubo da Costa <rakuco@freebsd.org>
Cc:        ports-committers@freebsd.org, dev-commits-ports-all@freebsd.org,  dev-commits-ports-main@freebsd.org
Subject:   Re: git: b56d0bc41af7 - main - devel/libcutl: Explain why USE_CXXSTD=c++11 is used
Message-ID:  <CAFDf7UJRc4bd3XVq%2Bb7Za2cLN7h_hkN1mpPTGQiSajV1p9A=0g@mail.gmail.com>
In-Reply-To: <202307050850.3658oPOA049562@gitrepo.freebsd.org>
References:  <202307050850.3658oPOA049562@gitrepo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
--000000000000c8d2cd05ffb9d1df
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

Hello Raphael,

Really nice and explaining comment on std c++11 fixes. I will use same
comment on ports needed it to work in clang 16, e.g., net-p2p/amule that
needs it.

Just a question, should a PORTREVISION bump reccomended when we set
USE_CXXSTD=3Dc++11?

Thanks,

Raphael Kubo da Costa <rakuco@freebsd.org> escreveu no dia quarta,
5/07/2023 =C3=A0(s) 09:50:

> The branch main has been updated by rakuco:
>
> URL:
> https://cgit.FreeBSD.org/ports/commit/?id=3Db56d0bc41af79bf3a51f89249e96e=
c02b7fc464e
>
> commit b56d0bc41af79bf3a51f89249e96ec02b7fc464e
> Author:     Raphael Kubo da Costa <rakuco@FreeBSD.org>
> AuthorDate: 2023-07-05 08:49:09 +0000
> Commit:     Raphael Kubo da Costa <rakuco@FreeBSD.org>
> CommitDate: 2023-07-05 08:49:09 +0000
>
>     devel/libcutl: Explain why USE_CXXSTD=3Dc++11 is used
>
>     While here, also add a note to its only consumer, devel/xsd, to menti=
on
>     why it also needs USE_CXXSTD and when it can be removed.
> ---
>  devel/libcutl/Makefile | 2 ++
>  devel/xsd/Makefile     | 3 +++
>  2 files changed, 5 insertions(+)
>
> diff --git a/devel/libcutl/Makefile b/devel/libcutl/Makefile
> index 323b2b44feee..bbaad70028c2 100644
> --- a/devel/libcutl/Makefile
> +++ b/devel/libcutl/Makefile
> @@ -17,6 +17,8 @@ LIB_DEPENDS=3D  libboost_system.so:devel/boost-libs \
>                 libexpat.so:textproc/expat2
>
>  USES=3D          compiler:c++11-lib libtool
> +# libcutl 1.10.0 is not compatible with C++17, which is the default sinc=
e
> clang
> +# 16. This means ports depending on it also need to use at most C++14.
>  USE_CXXSTD=3D    c++11
>  GNU_CONFIGURE=3D yes
>  INSTALL_TARGET=3D        install-strip
> diff --git a/devel/xsd/Makefile b/devel/xsd/Makefile
> index 06e02ce6f72c..010b73e6e9e4 100644
> --- a/devel/xsd/Makefile
> +++ b/devel/xsd/Makefile
> @@ -23,6 +23,9 @@ LIB_DEPENDS=3D  libcutl.so:devel/libcutl \
>                 libxerces-c.so:textproc/xerces-c3
>
>  USES=3D          compiler:c++11-lang ghostscript:build gmake
> localbase:ldflags tar:bzip2
> +# devel/libcutl 1.10.0 does not work with C++17 which is the default sin=
ce
> +# clang 16. This can be removed once that port starts working with C++17
> and
> +# has its own USE_CXXSTD removed.
>  USE_CXXSTD=3D    c++11
>
>  MAKEFILE=3D      makefile
>


--=20
Nuno Teixeira
FreeBSD Committer (ports)

--000000000000c8d2cd05ffb9d1df
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr"><div>Hello Raphael,</div><div><br></div><div>Really nice a=
nd explaining comment on std c++11 fixes. I will use same comment on ports =
needed it to work in clang 16, e.g., net-p2p/amule that needs it.<br><br></=
div><div>Just a question, should a PORTREVISION bump reccomended when we se=
t USE_CXXSTD=3Dc++11?</div><div><br></div><div>Thanks,<br></div></div><br><=
div class=3D"gmail_quote"><div dir=3D"ltr" class=3D"gmail_attr">Raphael Kub=
o da Costa &lt;<a href=3D"mailto:rakuco@freebsd.org">rakuco@freebsd.org</a>=
&gt; escreveu no dia quarta, 5/07/2023 =C3=A0(s) 09:50:<br></div><blockquot=
e class=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;border-left:1px s=
olid rgb(204,204,204);padding-left:1ex">The branch main has been updated by=
 rakuco:<br>
<br>
URL: <a href=3D"https://cgit.FreeBSD.org/ports/commit/?id=3Db56d0bc41af79bf=
3a51f89249e96ec02b7fc464e" rel=3D"noreferrer" target=3D"_blank">https://cgi=
t.FreeBSD.org/ports/commit/?id=3Db56d0bc41af79bf3a51f89249e96ec02b7fc464e</=
a><br>
<br>
commit b56d0bc41af79bf3a51f89249e96ec02b7fc464e<br>
Author:=C2=A0 =C2=A0 =C2=A0Raphael Kubo da Costa &lt;rakuco@FreeBSD.org&gt;=
<br>
AuthorDate: 2023-07-05 08:49:09 +0000<br>
Commit:=C2=A0 =C2=A0 =C2=A0Raphael Kubo da Costa &lt;rakuco@FreeBSD.org&gt;=
<br>
CommitDate: 2023-07-05 08:49:09 +0000<br>
<br>
=C2=A0 =C2=A0 devel/libcutl: Explain why USE_CXXSTD=3Dc++11 is used<br>
<br>
=C2=A0 =C2=A0 While here, also add a note to its only consumer, devel/xsd, =
to mention<br>
=C2=A0 =C2=A0 why it also needs USE_CXXSTD and when it can be removed.<br>
---<br>
=C2=A0devel/libcutl/Makefile | 2 ++<br>
=C2=A0devel/xsd/Makefile=C2=A0 =C2=A0 =C2=A0| 3 +++<br>
=C2=A02 files changed, 5 insertions(+)<br>
<br>
diff --git a/devel/libcutl/Makefile b/devel/libcutl/Makefile<br>
index 323b2b44feee..bbaad70028c2 100644<br>
--- a/devel/libcutl/Makefile<br>
+++ b/devel/libcutl/Makefile<br>
@@ -17,6 +17,8 @@ LIB_DEPENDS=3D=C2=A0 libboost_system.so:devel/boost-libs =
\<br>
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 libexpat.so:textpro=
c/expat2<br>
<br>
=C2=A0USES=3D=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 compiler:c++11-lib libtool<=
br>
+# libcutl 1.10.0 is not compatible with C++17, which is the default since =
clang<br>
+# 16. This means ports depending on it also need to use at most C++14.<br>
=C2=A0USE_CXXSTD=3D=C2=A0 =C2=A0 c++11<br>
=C2=A0GNU_CONFIGURE=3D yes<br>
=C2=A0INSTALL_TARGET=3D=C2=A0 =C2=A0 =C2=A0 =C2=A0 install-strip<br>
diff --git a/devel/xsd/Makefile b/devel/xsd/Makefile<br>
index 06e02ce6f72c..010b73e6e9e4 100644<br>
--- a/devel/xsd/Makefile<br>
+++ b/devel/xsd/Makefile<br>
@@ -23,6 +23,9 @@ LIB_DEPENDS=3D=C2=A0 libcutl.so:devel/libcutl \<br>
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 libxerces-c.so:text=
proc/xerces-c3<br>
<br>
=C2=A0USES=3D=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 compiler:c++11-lang ghostsc=
ript:build gmake localbase:ldflags tar:bzip2<br>
+# devel/libcutl 1.10.0 does not work with C++17 which is the default since=
<br>
+# clang 16. This can be removed once that port starts working with C++17 a=
nd<br>
+# has its own USE_CXXSTD removed.<br>
=C2=A0USE_CXXSTD=3D=C2=A0 =C2=A0 c++11<br>
<br>
=C2=A0MAKEFILE=3D=C2=A0 =C2=A0 =C2=A0 makefile<br>
</blockquote></div><br clear=3D"all"><br><span class=3D"gmail_signature_pre=
fix">-- </span><br><div dir=3D"ltr" class=3D"gmail_signature"><div dir=3D"l=
tr"><span style=3D"color:rgb(102,102,102)">Nuno Teixeira<br>FreeBSD Committ=
er (ports)</span></div></div>

--000000000000c8d2cd05ffb9d1df--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAFDf7UJRc4bd3XVq%2Bb7Za2cLN7h_hkN1mpPTGQiSajV1p9A=0g>