unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
From: ludo@gnu.org (Ludovic Courtès)
To: Hartmut Goebel <h.goebel@crazy-compilers.com>
Cc: 26352@debbugs.gnu.org
Subject: bug#26352: [PATCH 1/3] gnu: Update networkmanager to version 1.6.2.
Date: Tue, 11 Apr 2017 17:45:21 +0200	[thread overview]
Message-ID: <87wparq772.fsf@gnu.org> (raw)
In-Reply-To: <58ECDEAD.1040306@crazy-compilers.com> (Hartmut Goebel's message of "Tue, 11 Apr 2017 15:48:29 +0200")

Hello!

Hartmut Goebel <h.goebel@crazy-compilers.com> skribis:

>> Note that ‘pre-configure’, ‘fix-docbook’, and ‘install’ are not fields
>> of <package>, so we normally don’t represent them this way in commit logs.
>
> Fine for me. How do you want it?

Something like
<https://git.savannah.gnu.org/cgit/guix.git/commit/?id=81eaa3f8579df42dbf0410db78b45a718fc544e7>
would do.  :-)

>> A couple of new tests are being skipped, this is for the same reason as
>> the others, right?
>
> Yes. Each of these tests is accompanied by a "fake" one which passes.

OK.

>> Normally adding both libxml2 and libxslt as inputs would save you from
>> writing this phase.  Could you check that this is the case?
> Indeed, adding libxml2 sets XML_CATALOG_FILES. But then it becomes curious:
>
> - With both docbook-xsl and docbook-xml as inputs, the build fails.
> (This did not happen with my patch, since I missed adding docbook-xml to
> XML_CATALOG_FILES.)
>
>       GEN      man/nmcli.1
>     I/O error : Attempt to load network entity
>     http://docbook.sourceforge.net/release/xsl/current/manpages/docbook.xsl
>     warning: failed to load external entity
>     "http://docbook.sourceforge.net/release/xsl/current/manpages/docbook.xsl"
>     cannot parse
>     http://docbook.sourceforge.net/release/xsl/current/manpages/docbook.xsl
>
>
> - With only docbook-xsl, the build passes while showing some errors,
> which are only treated as warnings:
>
>       GEN      man/nmcli.1
>     I/O error : Attempt to load network entity
>     http://www.oasis-open.org/docbook/xml/4.2/docbookx.dtd
>     man/nmcli.xml:7: warning: failed to load external entity
>     "http://www.oasis-open.org/docbook/xml/4.2/docbookx.dtd"
>     ]>
>       ^
>     man/nmcli.xml:2060: parser error : Entity 'ndash' not defined
>               <para>Success &ndash; indicates the operation
>     succeeded.</para>
>                                    ^
>     man/nmcli.xml:2422: parser error : Entity 'mdash' not defined
>         https://bugzilla.gnome.org/ &mdash; product
>     <literal>NetworkManager</literal
>
>
> I'm in favour of using the only docbook-xsl as input, as this makes the
> build pass.

Let’s do that then, sounds good!

Thanks,
Ludo’.

  reply	other threads:[~2017-04-11 15:46 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-04-03 18:54 bug#26352: [PATCH 0/3] Update KDE Frameworks to 5.32 Hartmut Goebel
2017-04-03 18:56 ` bug#26352: [PATCH 1/3] gnu: Update networkmanager to version 1.6.2 Hartmut Goebel
2017-04-03 18:56   ` bug#26352: [PATCH 2/3] gnu: Update phonon to 4.9.1 Hartmut Goebel
2017-04-03 18:56   ` bug#26352: [PATCH 3/3] gnu: Update kde-frameworks to 5.32.0 Hartmut Goebel
2017-04-10 10:04   ` bug#26352: [PATCH 1/3] gnu: Update networkmanager to version 1.6.2 Ludovic Courtès
2017-04-11 13:48     ` Hartmut Goebel
2017-04-11 15:45       ` Ludovic Courtès [this message]
2017-04-12  8:11         ` Hartmut Goebel
2017-04-12  8:11           ` bug#26352: close Hartmut Goebel
2017-04-12 13:34             ` Clément Lassieur
2017-04-03 19:12 ` bug#26352: [PATCH 4/4] gnu: kde-frameworks: No longer setenv CTEST_OUTPUT_ON_FAILURE Hartmut Goebel

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://guix.gnu.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87wparq772.fsf@gnu.org \
    --to=ludo@gnu.org \
    --cc=26352@debbugs.gnu.org \
    --cc=h.goebel@crazy-compilers.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).