mbox

[bug#43823,0/3] Knot: Enable DoH; build Info manual; work on aarch64-linux

Message ID cover.1601941844.git.simon@simonsouth.net
Headers show

Message

Simon South Oct. 6, 2020, 12:25 a.m. UTC 
These changes to the Knot package complement work done today by
Tobias[0] by silencing the remaining warnings from configure while
adding a bit of new functionality. The patches

- Enable DNS-over-HTTPS support in the kdig utility, by adding the
  nghttp2 library as a native input;

- Cause the Info manual to be built and included in the output; and

- Fix a test[1] that prevented the package from building on
  aarch64-linux systems.

[0] Now that everyone's finished building commit 2eb0585.
[1] See https://gitlab.nic.cz/knot/knot-dns/-/issues/693

--
Simon South
simon@simonsouth.net


Simon South (3):
  gnu: knot: Enable DNS-over-HTTPS support in kdig.
  gnu: knot: Build the Info manual.
  gnu: knot: Fix test failure on aarch64-linux.

 gnu/packages/dns.scm                          | 16 ++++++++++++--
 ...est_net_shortwrite-ensure-connection.patch | 21 +++++++++++++++++++
 2 files changed, 35 insertions(+), 2 deletions(-)
 create mode 100644 gnu/packages/patches/knot-test_net_shortwrite-ensure-connection.patch

Comments

Tobias Geerinckx-Rice Oct. 6, 2020, 1:08 a.m. UTC | #1 
Simon,

Simon South 写道：
> These changes to the Knot package complement work done today by
> Tobias[0] by silencing the remaining warnings from configure 
> while
> adding a bit of new functionality. The patches

Neatness!  Thank you.

I would like to take another stab at enabling the new XDP support 
and sneak it into your series before pushing the lot.  Everyone's 
built quite enough knots for a while.

Kind regards,

T G-R
Simon South Oct. 6, 2020, 1:56 p.m. UTC | #2 
This updated set of patches incorporates feedback from Tobias
Geerinckx-Rice (thank you!) and improves on the original by

- Using input packages' own name as their label;

- Removing mention in the commit descriptions of silencing warnings
  from configure (he's right, it's a bit silly to worry about this);

- Replacing the description in the patch file with something more
  informative that includes a link to the original issue; and

- Registering the patch file in gnu/local.mk.

For convenience, here's my original cover letter:

These changes to the Knot package complement work done today by
Tobias[0] by silencing the remaining warnings from configure while
adding a bit of new functionality. The patches

- Enable DNS-over-HTTPS support in the kdig utility, by adding the
  nghttp2 library as a native [actually regular] input;

- Cause the Info manual to be built and included in the output; and

- Fix a test[1] that prevented the package from building on
  aarch64-linux systems.

[0] Now that everyone's finished building commit 2eb0585.
[1] See https://gitlab.nic.cz/knot/knot-dns/-/issues/693

--
Simon South
simon@simonsouth.net


Simon South (3):
  gnu: knot: Enable DNS-over-HTTPS support in kdig.
  gnu: knot: Build the Info manual.
  gnu: knot: Fix test failure on aarch64-linux.

 gnu/local.mk                                  |  1 +
 gnu/packages/dns.scm                          | 16 ++++++++++++++--
 ...est_net_shortwrite-ensure-connection.patch | 19 +++++++++++++++++++
 3 files changed, 34 insertions(+), 2 deletions(-)
 create mode 100644 gnu/packages/patches/knot-test_net_shortwrite-ensure-connection.patch