Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 742650 - dev-libs/aws-sdk-cpp-1.8.46 fails to compile
Summary: dev-libs/aws-sdk-cpp-1.8.46 fails to compile
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Sven Eden
URL:
Whiteboard:
Keywords: PullRequest
Depends on:
Blocks:
 
Reported: 2020-09-15 12:08 UTC by Agostino Sarubbo
Modified: 2020-09-28 11:34 UTC (History)
2 users (show)

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


Attachments
build.log (build.log,351.62 KB, text/plain)
2020-09-15 12:08 UTC, Agostino Sarubbo
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Agostino Sarubbo gentoo-dev 2020-09-15 12:08:18 UTC
https://blogs.gentoo.org/ago/2020/07/04/gentoo-tinderbox/

Issue: dev-libs/aws-sdk-cpp-1.8.46 fails to compile.
Discovered on: amd64 (internal ref: ci)

NOTE:
This machine uses GLIBC-2.32. If you think that this issue is strictly related to GLIBC please block bug 736174.
Comment 1 Agostino Sarubbo gentoo-dev 2020-09-15 12:08:21 UTC
Created attachment 660384 [details]
build.log

build log and emerge --info
Comment 2 Sven Eden 2020-09-17 12:56:40 UTC
I have glibc-2.32-r1 installed, and have merged the SDK with _ALL_ modules enabled without a single issue.

Currently I am rebuilding again plus USE="test" to see what is going on.

*but* : As the three failed tests are all EventStream* tests, it might be that the reason for the failure is that https://github.com/gentoo/gentoo/pull/17355 did not get merged beforehand, although I hinted it should.
https://github.com/gentoo/gentoo/pull/17303 did also not get merged, yet.

All three upgrades are trivial, and I have these locally. But before anybody rushes for a quick merge, please let me finish building and testing.

I'll report back here when I have seen results.
Comment 3 Sven Eden 2020-09-17 15:33:14 UTC
With aws-c-event-stream-0.1.3 I get:

--------
[----------] Global test environment tear-down
[==========] 376 tests from 67 test cases ran. (4120 ms total)
[  PASSED  ] 373 tests.
[  FAILED  ] 3 tests, listed below:
[  FAILED  ] EventStreamDecoderTest.BadInternalPayloadTest
[  FAILED  ] EventStreamTest.TestXmlErrorPayloadWithAwsErrorMarshaller
[  FAILED  ] EventStreamTest.TestXmlErrorPayloadWithAwsString
--------

With aws-c-event-stream-0.1.6 I get:

--------
[----------] Global test environment tear-down
[==========] 376 tests from 67 test cases ran. (4152 ms total)
[  PASSED  ] 376 tests.
--------

I am currently working on a fixing PR that will bump aws-c-common, needed for bumping aws-c-event-stream, needed for aws-sdk-cpp tests succeeding.
Comment 4 Larry the Git Cow gentoo-dev 2020-09-28 11:34:28 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=6fe9cad6f5e2959783df9a7870013a13c8ef9d30

commit 6fe9cad6f5e2959783df9a7870013a13c8ef9d30
Author:     Sven Eden <sven.eden@prydeworx.com>
AuthorDate: 2020-09-17 14:31:37 +0000
Commit:     Joonas Niilola <juippis@gentoo.org>
CommitDate: 2020-09-28 11:25:05 +0000

    dev-libs/aws-sdk-cpp: Fixed dependencies and a typo
    
    * Depend on
      * >=dev-libs/aws-c-common-0.4.26 and
      * >=dev-libs/aws-c-event-stream-0.1.6
      The latter is needed for the test suite to succeed.
    * A dash was missing, and thus the elastic-inference module could not
      be activated. Instead, setting USE="elasticinference" lead to the
      configure stage to fail.
    
    Bug: https://bugs.gentoo.org/742650
    Closes: https://bugs.gentoo.org/742650
    
    Package-Manager: Portage-3.0.7, Repoman-3.0.1
    Signed-off-by: Sven Eden <sven.eden@prydeworx.com>
    Closes: https://github.com/gentoo/gentoo/pull/17576
    Signed-off-by: Joonas Niilola <juippis@gentoo.org>

 .../{aws-sdk-cpp-1.8.46.ebuild => aws-sdk-cpp-1.8.46-r1.ebuild}     | 6 +++---
 dev-libs/aws-sdk-cpp/metadata.xml                                   | 2 +-
 2 files changed, 4 insertions(+), 4 deletions(-)

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

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=6fe9cad6f5e2959783df9a7870013a13c8ef9d30

commit 6fe9cad6f5e2959783df9a7870013a13c8ef9d30
Author:     Sven Eden <sven.eden@prydeworx.com>
AuthorDate: 2020-09-17 14:31:37 +0000
Commit:     Joonas Niilola <juippis@gentoo.org>
CommitDate: 2020-09-28 11:25:05 +0000

    dev-libs/aws-sdk-cpp: Fixed dependencies and a typo
    
    * Depend on
      * >=dev-libs/aws-c-common-0.4.26 and
      * >=dev-libs/aws-c-event-stream-0.1.6
      The latter is needed for the test suite to succeed.
    * A dash was missing, and thus the elastic-inference module could not
      be activated. Instead, setting USE="elasticinference" lead to the
      configure stage to fail.
    
    Bug: https://bugs.gentoo.org/742650
    Closes: https://bugs.gentoo.org/742650
    
    Package-Manager: Portage-3.0.7, Repoman-3.0.1
    Signed-off-by: Sven Eden <sven.eden@prydeworx.com>
    Closes: https://github.com/gentoo/gentoo/pull/17576
    Signed-off-by: Joonas Niilola <juippis@gentoo.org>

 .../{aws-sdk-cpp-1.8.46.ebuild => aws-sdk-cpp-1.8.46-r1.ebuild}     | 6 +++---
 dev-libs/aws-sdk-cpp/metadata.xml                                   | 2 +-
 2 files changed, 4 insertions(+), 4 deletions(-)