GNU bug report logs - #47898
[PATCH] [core-updates] Add option --xpath0 to xmllint from libxml2

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: david larsson <david.larsson <at> selfhosted.xyz>
Subject: bug#47898: closed (Re: [bug#47898] [PATCH] [core-updates] Add
 option --xpath0 to xmllint from libxml2)
Date: Tue, 18 May 2021 20:26:02 +0000

[Message part 1 (text/plain, inline)]

Your bug report

#47898: [PATCH] [core-updates] Add option --xpath0 to xmllint from libxml2

which was filed against the guix-patches package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to 47898 <at> debbugs.gnu.org.

-- 
47898: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=47898
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems

[Message part 2 (message/rfc822, inline)]

From: Marius Bakke <marius <at> gnu.org>
To: david larsson <david.larsson <at> selfhosted.xyz>
Cc: 47898-done <at> debbugs.gnu.org
Subject: Re: [bug#47898] [PATCH] [core-updates] Add option --xpath0 to
 xmllint from libxml2
Date: Tue, 18 May 2021 22:25:46 +0200

[Message part 3 (text/plain, inline)]

david larsson <david.larsson <at> selfhosted.xyz> skriver:

> Hi, new patch attached. There's a lint warning though: 
> "libxml2-xpath0 <at> 2.9.10: no updater for libxml2-xpath0". I don't know 
> what it means, maybe it should be fixed before committing?

That warning is harmless and affects a lot of packages.  I've applied
the patch with a couple minor tweaks:

> +From e1df743329bdfd94fbfdea18303c5c6c6fe13403 Mon Sep 17 00:00:00 2001
> +From: methuselah-0 <david.larsson <at> selfhosted.xyz>
> +Date: Thu, 1 Apr 2021 08:33:56 +0200
> +Subject: [PATCH] Add option --xpath0
> +
> +---
> + doc/xmllint.xml | 16 ++++++++++++++++
> + xmllint.c       | 32 +++++++++++++++++++++++---------
> + 2 files changed, 39 insertions(+), 9 deletions(-)

I replaced this git-style patch header with a short description and an
URL to the upstream issue, like we usually do.

> +(define-public libxml2-xpath0
> +  (package/inherit libxml2
> +    (name "libxml2-xpath0")
> +    (source (origin
> +              (inherit (package-source libxml2))
> +              (patches (append (search-patches
> +                                "libxml2-xpath0-Add-option-xpath0.patch")
> +                               (origin-patches (package-source libxml2))))))
> +    (description
> +     "Libxml2-xpath0 is like libxml2 but with a patch applied that
> +provides an --xpath0 option to xmllint that enables it to output xpath
> +results with a null delimiter")))

...and sprinkled some markup and punctuation into the description.

Pushed as b58efbc6611550ad9234163e198ff71ace5306ea, thank you!

[signature.asc (application/pgp-signature, inline)]

[Message part 5 (message/rfc822, inline)]

From: david larsson <david.larsson <at> selfhosted.xyz>
To: guix-patches <at> gnu.org
Subject: [PATCH] [core-updates] Add option --xpath0 to xmllint from libxml2
Date: Mon, 19 Apr 2021 20:48:47 +0200

[Message part 6 (text/plain, inline)]

Hi!
This patch adds the option to separate xpath results from xmllint by a 
null delimiter when using the --xpath0 option-flag. It is something 
that's been asked for for a long time by users on stackoverflow and 
merge-attempts have been made to upstream without success.

Examples:
  - https://gitlab.gnome.org/GNOME/libxml2/-/issues/227
  - https://gitlab.gnome.org/GNOME/libxml2/-/merge_requests/8

Best regards,
David

[0001-gnu-libxml2-Add-patch-for-xpath0-option-to-xmllint.patch (text/x-diff, attachment)]

Previous Next

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