From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:41538) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1eQUQY-0002Ps-2z for guix-patches@gnu.org; Sun, 17 Dec 2017 03:36:07 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1eQUQU-00084n-Uw for guix-patches@gnu.org; Sun, 17 Dec 2017 03:36:06 -0500 Received: from debbugs.gnu.org ([208.118.235.43]:57594) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1eQUQU-00084Y-LX for guix-patches@gnu.org; Sun, 17 Dec 2017 03:36:02 -0500 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1eQUQU-00025O-DC for guix-patches@gnu.org; Sun, 17 Dec 2017 03:36:02 -0500 Subject: [bug#29738] Add support for HTTP/2 to curl Resent-Message-ID: Message-Id: <1513499718.3586623.1207635016.7822479B@webmail.messagingengine.com> From: Mark Meyer MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Type: multipart/mixed; boundary="_----------=_151349971835866230" In-Reply-To: <878te1oofb.fsf@elephly.net> Date: Sun, 17 Dec 2017 09:35:18 +0100 References: <1513447932.2411366.1207257248.22729BA6@webmail.messagingengine.com> <878te1oofb.fsf@elephly.net> List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+kyle=kyleam.com@gnu.org Sender: "Guix-patches" To: Ricardo Wurmus Cc: 29738@debbugs.gnu.org This is a multi-part message in MIME format. --_----------=_151349971835866230 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Hi Ricardo, thanks for you feedback. The closure changes from 223.7 MiB to 224.5 MiB. The patches attached apply cleanly to core-updates and split the change into two components. Cheers, Mark On Sun, Dec 17, 2017, at 07:31, Ricardo Wurmus wrote: >=20 > Hi Mark, >=20 > > this patch adds HTTP/2 support to curl, this is a prerequisite to > > closing bug#29733. I am aware that this will trigger about 2500 rebuilt > > packages, but having HTTP/2 in curl seems to be worth the hassle. >=20 > We would make this change on the core-updates branch to avoid disrupting > the availability of substitutes, so rebuilding a large number of > packages is not a big problem. Does this patch apply on top of > core-updates? >=20 > About the patch: it would be better to separate the two changes. One > commit would add the nghttp2 package, the other would modify the curl > package. Could you please split the patch? >=20 > Since curl is used by a great number of packages we need to pay > attention to the size of the closure. How does adding nghttp2 to curl > affect the size of the closure of the curl package? (You can check with > =E2=80=9Cguix size=E2=80=9D.) >=20 > -- > Ricardo >=20 > GPG: BCA6 89B6 3655 3801 C3C6 2150 197A 5888 235F ACAC > https://elephly.net >=20 >=20 --=20 Mark Meyer mark@ofosos.org --_----------=_151349971835866230 Content-Disposition: attachment; filename="0001-gnu-Add-nghttp2-support.patch" Content-Id: <1513499673.3586381.76dd7fd30b8bec395ea4ec8815f75894bd9ffdb7.0C0C7F46@content.messagingengine.com> Content-Transfer-Encoding: base64 Content-Type: text/x-patch; name="0001-gnu-Add-nghttp2-support.patch" RnJvbSAwNWM4ZjRhNDQzYWQ2MmMwMTNhYTVlMThiNDY5NzliOTg1NzM3ZGFj IE1vbiBTZXAgMTcgMDA6MDA6MDAgMjAwMQpGcm9tOiBNYXJrIE1leWVyIDxt YXJrQG9mb3Nvcy5vcmc+CkRhdGU6IFNhdCwgMTYgRGVjIDIwMTcgMTk6MDI6 MTAgKzAxMDAKU3ViamVjdDogW1BBVENIXSBnbnU6IEFkZCBuZ2h0dHAyIHN1 cHBvcnQuCgoqIGdudS9wYWNrYWdlcy9jdXJsLnNjbSAobmdodHRwMik6IE5l dyB2YXJpYWJsZS4KLS0tCiBnbnUvcGFja2FnZXMvY3VybC5zY20gfCAyNiAr KysrKysrKysrKysrKysrKysrKysrKysrKwogMSBmaWxlIGNoYW5nZWQsIDI2 IGluc2VydGlvbnMoKykKCmRpZmYgLS1naXQgYS9nbnUvcGFja2FnZXMvY3Vy bC5zY20gYi9nbnUvcGFja2FnZXMvY3VybC5zY20KaW5kZXggM2RmOGFjYzJj Li43NDE0NTQ5NzggMTAwNjQ0Ci0tLSBhL2dudS9wYWNrYWdlcy9jdXJsLnNj bQorKysgYi9nbnUvcGFja2FnZXMvY3VybC5zY20KQEAgLTYsNiArNiw3IEBA CiA7OzsgQ29weXJpZ2h0IMKpIDIwMTYsIDIwMTcgTGVvIEZhbXVsYXJpIDxs ZW9AZmFtdWxhcmkubmFtZT4KIDs7OyBDb3B5cmlnaHQgwqkgMjAxNyBNYXJp dXMgQmFra2UgPG1iYWtrZUBmYXN0bWFpbC5jb20+CiA7OzsgQ29weXJpZ2h0 IMKpIDIwMTcgRWZyYWltIEZsYXNobmVyIDxlZnJhaW1AZmxhc2huZXIuY28u aWw+Cis7OzsgQ29weXJpZ2h0IMKpIDIwMTcgTWFyayBNZXllciA8bWFya0Bv Zm9zb3Mub3JnPgogOzs7CiA7OzsgVGhpcyBmaWxlIGlzIHBhcnQgb2YgR05V IEd1aXguCiA7OzsKQEAgLTI5LDYgKzMwLDcgQEAKICAgIzp1c2UtbW9kdWxl IChndWl4IHV0aWxzKQogICAjOnVzZS1tb2R1bGUgKGd1aXggYnVpbGQtc3lz dGVtIGdudSkKICAgIzp1c2UtbW9kdWxlIChnbnUgcGFja2FnZXMpCisgICM6 dXNlLW1vZHVsZSAoZ251IHBhY2thZ2VzIGNoZWNrKQogICAjOnVzZS1tb2R1 bGUgKGdudSBwYWNrYWdlcyBjb21wcmVzc2lvbikKICAgIzp1c2UtbW9kdWxl IChnbnUgcGFja2FnZXMgZ3JvZmYpCiAgICM6dXNlLW1vZHVsZSAoZ251IHBh Y2thZ2VzIGdzYXNsKQpAQCAtNDAsNiArNDIsMzAgQEAKICAgIzp1c2UtbW9k dWxlIChnbnUgcGFja2FnZXMgc3NoKQogICAjOnVzZS1tb2R1bGUgKGdudSBw YWNrYWdlcyB0bHMpKQogCisoZGVmaW5lLXB1YmxpYyBuZ2h0dHAyCisgIChw YWNrYWdlCisgICAgKG5hbWUgIm5naHR0cDIiKQorICAgICh2ZXJzaW9uICIx LjI4LjAiKQorICAgIChzb3VyY2UgKG9yaWdpbgorICAgICAgICAgICAgICAo bWV0aG9kIHVybC1mZXRjaCkKKyAgICAgICAgICAgICAgKHVyaSAoc3RyaW5n LWFwcGVuZCAiaHR0cHM6Ly9naXRodWIuY29tL25naHR0cDIvbmdodHRwMi9y ZWxlYXNlcy9kb3dubG9hZC92IgorICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgIHZlcnNpb24gIi9uZ2h0dHAyLSIgdmVyc2lvbiAiLnRhci54 eiIpKQorICAgICAgICAgICAgICAoc2hhMjU2CisgICAgICAgICAgICAgICAo YmFzZTMyCisgICAgICAgICAgICAgICAgIjEzZ3hrNzJtYW5ibWFhZjNtYWh2 aWhmdzcxemFzMW03ejhqMmJzOXM3djJkYzQwM3l2MGQiKSkpKQorICAgIChi dWlsZC1zeXN0ZW0gZ251LWJ1aWxkLXN5c3RlbSkKKyAgICAobmF0aXZlLWlu cHV0cworICAgICBgKCgiY3VuaXQiICxjdW5pdCkpKQorICAgIChhcmd1bWVu dHMKKyAgICAgYCgjOmNvbmZpZ3VyZS1mbGFncyAnKCItLWVuYWJsZS1saWIt b25seSIpKSkKKyAgICAoc3lub3BzaXMgIlRoaXMgaXMgYW4gaW1wbGVtZW50 YXRpb24gb2YgSFRUUCB2ZXJzaW9uIDIgaW4gQyIpCisgICAgKGRlc2NyaXB0 aW9uICJuZ2h0dHAyIGlzIGFuIGltcGxlbWVudGF0aW9uIG9mIGZyYW1pbmcg bGF5ZXIgb2YgSFRUUC8yIGluCityZXVzYWJsZSBDIGxpYnJhcnkuICBPbiB0 b3Agb2YgdGhhdCwgdGhpcyBsaWJyYXJ5IGltcGxlbWVudHMgYW4gSFRUUC8y IGNsaWVudCwKK3NlcnZlciBhbmQgcHJveHkuICBIUEFDSyBlbmNvZGVyIGFu ZCBkZWNvZGVyIGFyZSBhdmFpbGFibGUgYXMgYSBwdWJsaWMgQVBJLiIpCisg ICAgKGxpY2Vuc2UgKGxpY2Vuc2U6bm9uLWNvcHlsZWZ0ICJmaWxlOi8vQ09Q WUlORyIKKyAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIlNl ZSBDT1BZSU5HIGluIHRoZSBkaXN0cmlidXRpb24uIikpCisgICAgKGhvbWUt cGFnZSAiaHR0cHM6Ly9uZ2h0dHAyLm9yZyIpKSkKKwogKGRlZmluZS1wdWJs aWMgY3VybAogICAocGFja2FnZQogICAgKG5hbWUgImN1cmwiKQotLSAKMi4x NS4xCgo= --_----------=_151349971835866230 Content-Disposition: attachment; filename="0002-gnu-Add-http2-support-to-curl.patch" Content-Id: <1513499676.3586559.143859d016432316e55731b7a6073d2cc42fb9c5.68B3516C@content.messagingengine.com> Content-Transfer-Encoding: base64 Content-Type: text/x-patch; name="0002-gnu-Add-http2-support-to-curl.patch" RnJvbSBkNmQzMzIwMDQ2ZGU5MDVlYzNjZjljMTdkY2ExMzNkMDFjYzEwNWU0 IE1vbiBTZXAgMTcgMDA6MDA6MDAgMjAwMQpGcm9tOiBNYXJrIE1leWVyIDxt YXJrQG9mb3Nvcy5vcmc+CkRhdGU6IFN1biwgMTcgRGVjIDIwMTcgMDk6MzE6 NTggKzAxMDAKU3ViamVjdDogW1BBVENIXSBnbnU6IEFkZCBodHRwMiBzdXBw b3J0IHRvIGN1cmwuCgoqIGdudS9wYWNrYWdlcy9jdXJsLnNjbSAoY3VybClb aW5wdXRzXTogQWRkIG5naHR0cDIuCi0tLQogZ251L3BhY2thZ2VzL2N1cmwu c2NtIHwgMSArCiAxIGZpbGUgY2hhbmdlZCwgMSBpbnNlcnRpb24oKykKCmRp ZmYgLS1naXQgYS9nbnUvcGFja2FnZXMvY3VybC5zY20gYi9nbnUvcGFja2Fn ZXMvY3VybC5zY20KaW5kZXggNzQxNDU0OTc4Li4wNzdmMzMwZDIgMTAwNjQ0 Ci0tLSBhL2dudS9wYWNrYWdlcy9jdXJsLnNjbQorKysgYi9nbnUvcGFja2Fn ZXMvY3VybC5zY20KQEAgLTg0LDYgKzg0LDcgQEAgc2VydmVyIGFuZCBwcm94 eS4gIEhQQUNLIGVuY29kZXIgYW5kIGRlY29kZXIgYXJlIGF2YWlsYWJsZSBh cyBhIHB1YmxpYyBBUEkuIikKICAgICAgICAgICAgICAoImdzcyIgLGdzcykK ICAgICAgICAgICAgICAoImxpYmlkbiIgLGxpYmlkbikKICAgICAgICAgICAg ICAoImxpYnNzaDIiICxsaWJzc2gyKQorICAgICAgICAgICAgICgibmdodHRw MiIgLG5naHR0cDIpCiAgICAgICAgICAgICAgKCJvcGVubGRhcCIgLG9wZW5s ZGFwKQogICAgICAgICAgICAgICgiemxpYiIgLHpsaWIpKSkKICAgIChuYXRp dmUtaW5wdXRzCi0tIAoyLjE1LjEKCg== --_----------=_151349971835866230--