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

[GNU bug Tracking System]

Your message dated Tue, 18 May 2021 22:25:46 +0200
with message-id <87im3fbxh1.fsf@gnu.org>
and subject line Re: [bug#47898] [PATCH] [core-updates] Add option --xpath0 to 
xmllint from libxml2
has caused the debbugs.gnu.org bug report #47898,
regarding [PATCH] [core-updates] Add option --xpath0 to xmllint from libxml2
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs@gnu.org.)


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

--- Begin Message --- Subject: [PATCH] [core-updates] Add option --xpath0 to xmllint from libxml2 Date: Mon, 19 Apr 2021 20:48:47 +0200

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
Description: Text Data

--- End Message ---

--- Begin Message --- Subject: Re: [bug#47898] [PATCH] [core-updates] Add option --xpath0 to xmllint from libxml2 Date: Tue, 18 May 2021 22:25:46 +0200

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

> Hi, new patch attached. There's a lint warning though: 
> "libxml2-xpath0@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@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
Description: PGP signature

--- End Message ---