GNU bug report logs - #55168
[PATCH 0/15] Fix aws-lc checksum and use with aws-sdk-cpp.

Previous Next

Package: guix-patches;

Reported by: Greg Hogan <code <at> greghogan.com>

Date: Thu, 28 Apr 2022 16:00:02 UTC

Severity: normal

Tags: patch

Done: Ludovic Courtès <ludo <at> gnu.org>

Bug is archived. No further changes may be made.

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 55168 in the body.
You can then email your comments to 55168 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to guix-patches <at> gnu.org:
bug#55168; Package guix-patches. (Thu, 28 Apr 2022 16:00:03 GMT) Full text and rfc822 format available.

Acknowledgement sent to Greg Hogan <code <at> greghogan.com>:
New bug report received and forwarded. Copy sent to guix-patches <at> gnu.org. (Thu, 28 Apr 2022 16:00:03 GMT) Full text and rfc822 format available.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Greg Hogan <code <at> greghogan.com>
To: guix-patches <at> gnu.org
Subject: [PATCH 0/15] Fix aws-lc checksum and use with aws-sdk-cpp.
Date: Thu, 28 Apr 2022 11:59:00 -0400
[Message part 1 (text/plain, inline)]
I happened to notice the broken build for aws-lc when exploring the
dashboard at ci.guix.gnu.org. In my last patchset I had updated aws-lc to
the proper version but failed to update the checksum since it was not built
as a dependency of aws-sdk-cpp.

These patches replace openssl with aws-lc as the default dependency of s2n,
making aws-lc a transitive dependency of aws-sdk-cpp. I have tested with
the following builds:
  ./pre-inst-env guix build aws-sdk-cpp
  ./pre-inst-env guix build --with-input=aws-lc=libressl aws-sdk-cpp
  ./pre-inst-env guix build --with-input=aws-lc=openssl <at> 1.1.1l aws-sdk-cpp

aws-c-common is now marked as supported only for i686-linux and
x86_64-linux, and s2n as supported only for x86_64-linux, as various tests
are failing or timing out on other systems.

Greg
[Message part 2 (text/html, inline)]
[0003-gnu-aws-c-event-stream-Fix-comment-formatting.patch (text/x-patch, attachment)]
[0005-gnu-aws-c-cal-Fix-comment-formatting.patch (text/x-patch, attachment)]
[0002-gnu-aws-checksums-Fix-comment-formatting.patch (text/x-patch, attachment)]
[0001-gnu-aws-c-common-Only-support-i686-and-x86_64.patch (text/x-patch, attachment)]
[0004-gnu-aws-c-io-Fix-comment-formatting.patch (text/x-patch, attachment)]
[0006-gnu-aws-c-sdkutils-Fix-comment-formatting.patch (text/x-patch, attachment)]
[0008-gnu-aws-c-compression-Fix-comment-formatting.patch (text/x-patch, attachment)]
[0009-gnu-aws-c-auth-Fix-comment-formatting.patch (text/x-patch, attachment)]
[0007-gnu-aws-c-http-Fix-comment-formatting.patch (text/x-patch, attachment)]
[0010-gnu-aws-c-s3-Fix-comment-formatting.patch (text/x-patch, attachment)]
[0012-gnu-aws-crt-cpp-Fix-comment-formatting.patch (text/x-patch, attachment)]
[0011-gnu-aws-c-mqtt-Fix-comment-formatting.patch (text/x-patch, attachment)]
[0013-gnu-aws-lc-Fix-checksum-and-enable-tests.patch (text/x-patch, attachment)]
[0015-gnu-aws-sdk-cpp-Remove-openssl-input.patch (text/x-patch, attachment)]
[0014-gnu-s2n-Link-to-aws-lc-as-libcrypto-dependency.patch (text/x-patch, attachment)]

Reply sent to Ludovic Courtès <ludo <at> gnu.org>:
You have taken responsibility. (Mon, 02 May 2022 12:47:02 GMT) Full text and rfc822 format available.

Notification sent to Greg Hogan <code <at> greghogan.com>:
bug acknowledged by developer. (Mon, 02 May 2022 12:47:02 GMT) Full text and rfc822 format available.

Message #10 received at 55168-done <at> debbugs.gnu.org (full text, mbox):

From: Ludovic Courtès <ludo <at> gnu.org>
To: Greg Hogan <code <at> greghogan.com>
Cc: 55168-done <at> debbugs.gnu.org
Subject: Re: bug#55168: [PATCH 0/15] Fix aws-lc checksum and use with
 aws-sdk-cpp.
Date: Mon, 02 May 2022 14:46:10 +0200
Hi,

Greg Hogan <code <at> greghogan.com> skribis:

> I happened to notice the broken build for aws-lc when exploring the
> dashboard at ci.guix.gnu.org. In my last patchset I had updated aws-lc to
> the proper version but failed to update the checksum since it was not built
> as a dependency of aws-sdk-cpp.

Oops, I plaid guilty as well, then.

> These patches replace openssl with aws-lc as the default dependency of s2n,
> making aws-lc a transitive dependency of aws-sdk-cpp. I have tested with
> the following builds:
>   ./pre-inst-env guix build aws-sdk-cpp
>   ./pre-inst-env guix build --with-input=aws-lc=libressl aws-sdk-cpp
>   ./pre-inst-env guix build --with-input=aws-lc=openssl <at> 1.1.1l aws-sdk-cpp
>
> aws-c-common is now marked as supported only for i686-linux and
> x86_64-linux, and s2n as supported only for x86_64-linux, as various tests
> are failing or timing out on other systems.

Alright.  Applied the whole series, thanks!

Ludo’.




bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> debbugs.gnu.org. (Tue, 31 May 2022 11:24:09 GMT) Full text and rfc822 format available.

This bug report was last modified 3 years and 16 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.