Date: Thu, 2 Mar 2023 13:34:22 +0000 From: Nuno Teixeira <eduardo@freebsd.org> To: Danilo Egea Gondolfo <danilo@freebsd.org> Cc: ports-committers@freebsd.org, dev-commits-ports-all@freebsd.org, dev-commits-ports-main@freebsd.org Subject: Re: git: e7b134bf62ce - main - devel/aws-sdk-cpp: Update to 1.11.25 Message-ID: <CAFDf7ULhOc1uAFw=KfpZB3iv4NXoh_1vGO3d6bO1=tg4qzBWoA@mail.gmail.com> In-Reply-To: <04e9e204-a2f1-cbac-c7ff-d07f70ba1738@FreeBSD.org> References: <202302271328.31RDScwO056423@gitrepo.freebsd.org> <e9ed6f0d-d593-3423-da74-7ee12c3a8b27@gmail.com> <04e9e204-a2f1-cbac-c7ff-d07f70ba1738@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--000000000000ada60d05f5eae347 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Hello Danilo, I will test it on i386 too to figure it out. If it is like you said, I'll remove broken_tests very soon. Thanks, Danilo Egea Gondolfo <danilo@freebsd.org> escreveu no dia segunda, 27/02/2023 =C3=A0(s) 16:32: > On 27/02/2023 14:30, Danilo Egea Gondolfo wrote: > > > On 27/02/2023 13:28, Nuno Teixeira wrote: > > > >> The branch main has been updated by eduardo: > >> > >> URL: > >> > https://cgit.FreeBSD.org/ports/commit/?id=3De7b134bf62ce15bd1daf1f7fbed51= 2262baa8eea > >> > >> commit e7b134bf62ce15bd1daf1f7fbed512262baa8eea > >> Author: Nuno Teixeira <eduardo@FreeBSD.org> > >> AuthorDate: 2023-02-27 09:00:18 +0000 > >> Commit: Nuno Teixeira <eduardo@FreeBSD.org> > >> CommitDate: 2023-02-27 13:28:07 +0000 > >> > >> devel/aws-sdk-cpp: Update to 1.11.25 > >> - fix TEST option that was wrongly set to ON and causing > >> build failures > >> on some 32bit archs. > >> - unbreak on mentioned archs > > > > Interesting. Does that mean the build will not work on 32bit archs if > > TEST is enabled? > > > > As the former maintainer of devel/aws-sdk-cpp I'm just curious. > > > > I didn't really try but if it's building on 32bit archs now the real > > reason might be this refactoring mentioned in the changelog for > > version 1.11.0 > > > > > https://github.com/aws/aws-sdk-cpp/blob/main/CHANGELOG.md#1110-2022-01-24 > > > > The build would fail before because they generate huge unified cpp > > files (option ENABLE_UNITY_BUILD) that were making a single instance > > of the compiler to use more than 4GB of memory (more than 6GB actually > > on 64bit). > > > > Anyway, I'm glad you fixed it for 32 bit archs. Thanks :) > > > Just test on i386. The build failed with tests because there are tests > failing, not because the compiler is consuming too much memory. > > I'd say those TEST_BROKEN could all be removed. > > > >> ChangeLog: > >> https://github.com/aws/aws-sdk-cpp/compare/1.11.10...1.11.25 > >> PR: 269837 > >> Reported by: John Hein <jcfyecrayz@liamekaens.com> > >> --- > >> devel/aws-sdk-cpp/Makefile | 13 ++++++------- > >> devel/aws-sdk-cpp/distinfo | 6 +++--- > >> 2 files changed, 9 insertions(+), 10 deletions(-) > >> > >> diff --git a/devel/aws-sdk-cpp/Makefile b/devel/aws-sdk-cpp/Makefile > >> index 94de9263a751..608d138e75bb 100644 > >> --- a/devel/aws-sdk-cpp/Makefile > >> +++ b/devel/aws-sdk-cpp/Makefile > >> @@ -1,5 +1,5 @@ > >> PORTNAME=3D aws-sdk-cpp > >> -DISTVERSION=3D 1.11.10 > >> +DISTVERSION=3D 1.11.25 > >> CATEGORIES=3D devel > >> MAINTAINER=3D eduardo@FreeBSD.org > >> @@ -9,11 +9,6 @@ WWW=3D https://github.com/aws/aws-sdk-cpp > >> LICENSE=3D APACHE20 > >> LICENSE_FILE=3D ${WRKSRC}/LICENSE > >> -BROKEN_armv6=3D fails to build: runaway process > >> -BROKEN_armv7=3D fails to build: runaway process > >> -BROKEN_i386=3D fails to build: compiler runs out of memory > >> -BROKEN_powerpc=3D fails to build: compiler runs out of memory > >> - > >> LIB_DEPENDS=3D libaws-c-auth.so:security/aws-c-auth \ > >> libaws-c-cal.so:security/aws-c-cal \ > >> libaws-c-common.so:devel/aws-c-common \ > >> @@ -58,7 +53,11 @@ BUILD_ONLY?=3D > >> OPTIONS_DEFINE=3D TEST > >> OPTIONS_SUB=3D yes > >> -TEST_CMAKE_BOOL_OFF=3D ENABLE_TESTING > >> +TEST_CMAKE_BOOL=3D ENABLE_TESTING > >> +TEST_BROKEN_armv6=3D fails to build: runaway process > >> +TEST_BROKEN_armv7=3D fails to build: runaway process > >> +TEST_BROKEN_i386=3D fails to build: compiler runs out of memory > >> +TEST_BROKEN_powerpc=3D fails to build: compiler runs out of memory > >> pre-everything:: > >> @${ECHO_MSG} > >> diff --git a/devel/aws-sdk-cpp/distinfo b/devel/aws-sdk-cpp/distinfo > >> index df6d21d967f7..2a639227971b 100644 > >> --- a/devel/aws-sdk-cpp/distinfo > >> +++ b/devel/aws-sdk-cpp/distinfo > >> @@ -1,3 +1,3 @@ > >> -TIMESTAMP =3D 1675456421 > >> -SHA256 (aws-aws-sdk-cpp-1.11.10_GH0.tar.gz) =3D > >> e1370a57252147df35bd8860f35ad69548bb46077adf7524c2d1dd02ee8aa643 > >> -SIZE (aws-aws-sdk-cpp-1.11.10_GH0.tar.gz) =3D 49752262 > >> +TIMESTAMP =3D 1677440274 > >> +SHA256 (aws-aws-sdk-cpp-1.11.25_GH0.tar.gz) =3D > >> 1103cf18516ca70d76c062af5a2b5c2418c5162235895baf3550a9f356d69f10 > >> +SIZE (aws-aws-sdk-cpp-1.11.25_GH0.tar.gz) =3D 49938123 > --=20 Nuno Teixeira FreeBSD Committer (ports) --000000000000ada60d05f5eae347 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable <div dir=3D"ltr"><div>Hello Danilo,</div><div><br></div><div>I will test it= on i386 too to figure it out.</div><div>If it is like you said, I'll r= emove broken_tests very soon.</div><div><br></div><div>Thanks,<br></div></d= iv><br><div class=3D"gmail_quote"><div dir=3D"ltr" class=3D"gmail_attr">Dan= ilo Egea Gondolfo <<a href=3D"mailto:danilo@freebsd.org">danilo@freebsd.= org</a>> escreveu no dia segunda, 27/02/2023 =C3=A0(s) 16:32:<br></div><= blockquote class=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;border-l= eft:1px solid rgb(204,204,204);padding-left:1ex">On 27/02/2023 14:30, Danil= o Egea Gondolfo wrote:<br> <br> > On 27/02/2023 13:28, Nuno Teixeira wrote:<br> ><br> >> The branch main has been updated by eduardo:<br> >><br> >> URL: <br> >> <a href=3D"https://cgit.FreeBSD.org/ports/commit/?id=3De7b134bf62c= e15bd1daf1f7fbed512262baa8eea" rel=3D"noreferrer" target=3D"_blank">https:/= /cgit.FreeBSD.org/ports/commit/?id=3De7b134bf62ce15bd1daf1f7fbed512262baa8e= ea</a><br> >><br> >> commit e7b134bf62ce15bd1daf1f7fbed512262baa8eea<br> >> Author:=C2=A0=C2=A0=C2=A0=C2=A0 Nuno Teixeira <eduardo@FreeBSD.= org><br> >> AuthorDate: 2023-02-27 09:00:18 +0000<br> >> Commit:=C2=A0=C2=A0=C2=A0=C2=A0 Nuno Teixeira <eduardo@FreeBSD.= org><br> >> CommitDate: 2023-02-27 13:28:07 +0000<br> >><br> >> =C2=A0=C2=A0=C2=A0=C2=A0 devel/aws-sdk-cpp: Update to 1.11.25<br> >> =C2=A0=C2=A0=C2=A0=C2=A0 =C2=A0=C2=A0=C2=A0=C2=A0 - fix TEST optio= n that was wrongly set to ON and causing <br> >> build failures<br> >> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 on some 32bit archs.<br> >> =C2=A0=C2=A0=C2=A0=C2=A0 - unbreak on mentioned archs<br> ><br> > Interesting. Does that mean the build will not work on 32bit archs if = <br> > TEST is enabled?<br> ><br> > As the former maintainer of devel/aws-sdk-cpp I'm just curious.<br= > ><br> > I didn't really try but if it's building on 32bit archs now th= e real <br> > reason might be this refactoring mentioned in the changelog for <br> > version 1.11.0<br> ><br> > <a href=3D"https://github.com/aws/aws-sdk-cpp/blob/main/CHANGELOG.md#1= 110-2022-01-24" rel=3D"noreferrer" target=3D"_blank">https://github.com/aws= /aws-sdk-cpp/blob/main/CHANGELOG.md#1110-2022-01-24</a><br> ><br> > The build would fail before because they generate huge unified cpp <br= > > files (option ENABLE_UNITY_BUILD) that were making a single instance <= br> > of the compiler to use more than 4GB of memory (more than 6GB actually= <br> > on 64bit).<br> ><br> > Anyway, I'm glad you fixed it for 32 bit archs. Thanks :)<br> ><br> Just test on i386. The build failed with tests because there are tests <br> failing, not because the compiler is consuming too much memory.<br> <br> I'd say those TEST_BROKEN could all be removed.<br> <br> <br> >> =C2=A0=C2=A0=C2=A0=C2=A0 =C2=A0=C2=A0=C2=A0=C2=A0 ChangeLog: <br> >> <a href=3D"https://github.com/aws/aws-sdk-cpp/compare/1.11.10...1.= 11.25" rel=3D"noreferrer" target=3D"_blank">https://github.com/aws/aws-sdk-= cpp/compare/1.11.10...1.11.25</a><br> >> =C2=A0=C2=A0=C2=A0=C2=A0 PR:=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 269837<br> >> =C2=A0=C2=A0=C2=A0=C2=A0 Reported by:=C2=A0=C2=A0=C2=A0 John Hein = <<a href=3D"mailto:jcfyecrayz@liamekaens.com" target=3D"_blank">jcfyecra= yz@liamekaens.com</a>><br> >> ---<br> >> =C2=A0 devel/aws-sdk-cpp/Makefile | 13 ++++++-------<br> >> =C2=A0 devel/aws-sdk-cpp/distinfo |=C2=A0 6 +++---<br> >> =C2=A0 2 files changed, 9 insertions(+), 10 deletions(-)<br> >><br> >> diff --git a/devel/aws-sdk-cpp/Makefile b/devel/aws-sdk-cpp/Makefi= le<br> >> index 94de9263a751..608d138e75bb 100644<br> >> --- a/devel/aws-sdk-cpp/Makefile<br> >> +++ b/devel/aws-sdk-cpp/Makefile<br> >> @@ -1,5 +1,5 @@<br> >> =C2=A0 PORTNAME=3D=C2=A0=C2=A0=C2=A0 aws-sdk-cpp<br> >> -DISTVERSION=3D=C2=A0=C2=A0=C2=A0 1.11.10<br> >> +DISTVERSION=3D=C2=A0=C2=A0=C2=A0 1.11.25<br> >> =C2=A0 CATEGORIES=3D=C2=A0=C2=A0=C2=A0 devel<br> >> =C2=A0 =C2=A0 MAINTAINER=3D=C2=A0=C2=A0=C2=A0 eduardo@FreeBSD.org<= br> >> @@ -9,11 +9,6 @@ WWW=3D=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 = <a href=3D"https://github.com/aws/aws-sdk-cpp" rel=3D"noreferrer" target=3D= "_blank">https://github.com/aws/aws-sdk-cpp</a><br>; >> =C2=A0 LICENSE=3D=C2=A0=C2=A0=C2=A0 APACHE20<br> >> =C2=A0 LICENSE_FILE=3D=C2=A0=C2=A0=C2=A0 ${WRKSRC}/LICENSE<br> >> =C2=A0 -BROKEN_armv6=3D=C2=A0=C2=A0=C2=A0 fails to build: runaway = process<br> >> -BROKEN_armv7=3D=C2=A0=C2=A0=C2=A0 fails to build: runaway process= <br> >> -BROKEN_i386=3D=C2=A0=C2=A0=C2=A0 fails to build: compiler runs ou= t of memory<br> >> -BROKEN_powerpc=3D=C2=A0=C2=A0=C2=A0 fails to build: compiler runs= out of memory<br> >> -<br> >> =C2=A0 LIB_DEPENDS=3D=C2=A0=C2=A0=C2=A0 libaws-c-auth.so:security/= aws-c-auth \<br> >> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 libaws-c-ca= l.so:security/aws-c-cal \<br> >> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 libaws-c-co= mmon.so:devel/aws-c-common \<br> >> @@ -58,7 +53,11 @@ BUILD_ONLY?=3D<br> >> =C2=A0 OPTIONS_DEFINE=3D=C2=A0=C2=A0=C2=A0 TEST<br> >> =C2=A0 OPTIONS_SUB=3D=C2=A0=C2=A0=C2=A0 yes<br> >> =C2=A0 -TEST_CMAKE_BOOL_OFF=3D=C2=A0=C2=A0=C2=A0 ENABLE_TESTING<br= > >> +TEST_CMAKE_BOOL=3D=C2=A0=C2=A0=C2=A0 ENABLE_TESTING<br> >> +TEST_BROKEN_armv6=3D=C2=A0=C2=A0=C2=A0 fails to build: runaway pr= ocess<br> >> +TEST_BROKEN_armv7=3D=C2=A0=C2=A0=C2=A0 fails to build: runaway pr= ocess<br> >> +TEST_BROKEN_i386=3D=C2=A0=C2=A0=C2=A0 fails to build: compiler ru= ns out of memory<br> >> +TEST_BROKEN_powerpc=3D=C2=A0=C2=A0=C2=A0 fails to build: compiler= runs out of memory<br> >> =C2=A0 =C2=A0 pre-everything::<br> >> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 @${ECHO_MSG}<br> >> diff --git a/devel/aws-sdk-cpp/distinfo b/devel/aws-sdk-cpp/distin= fo<br> >> index df6d21d967f7..2a639227971b 100644<br> >> --- a/devel/aws-sdk-cpp/distinfo<br> >> +++ b/devel/aws-sdk-cpp/distinfo<br> >> @@ -1,3 +1,3 @@<br> >> -TIMESTAMP =3D 1675456421<br> >> -SHA256 (aws-aws-sdk-cpp-1.11.10_GH0.tar.gz) =3D <br> >> e1370a57252147df35bd8860f35ad69548bb46077adf7524c2d1dd02ee8aa643<b= r> >> -SIZE (aws-aws-sdk-cpp-1.11.10_GH0.tar.gz) =3D 49752262<br> >> +TIMESTAMP =3D 1677440274<br> >> +SHA256 (aws-aws-sdk-cpp-1.11.25_GH0.tar.gz) =3D <br> >> 1103cf18516ca70d76c062af5a2b5c2418c5162235895baf3550a9f356d69f10<b= r> >> +SIZE (aws-aws-sdk-cpp-1.11.25_GH0.tar.gz) =3D 49938123<br> </blockquote></div><br clear=3D"all"><br>-- <br><div dir=3D"ltr" class=3D"g= mail_signature"><div dir=3D"ltr"><span style=3D"color:rgb(102,102,102)">Nun= o Teixeira<br>FreeBSD Committer (ports)</span></div></div> --000000000000ada60d05f5eae347--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAFDf7ULhOc1uAFw=KfpZB3iv4NXoh_1vGO3d6bO1=tg4qzBWoA>