unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
From: Kei Kebreau <kkebreau@posteo.net>
To: Leo Famulari <leo@famulari.name>
Cc: 29432@debbugs.gnu.org
Subject: [bug#29432] [PATCH] gnu: diffoscope: Fix error messages related to comparison tools.
Date: Sat, 25 Nov 2017 12:51:52 -0500	[thread overview]
Message-ID: <873752jlfr.fsf@posteo.net> (raw)
In-Reply-To: <20171125001716.GC17061@jasmine.lan> (Leo Famulari's message of "Fri, 24 Nov 2017 19:17:16 -0500")

[-- Attachment #1: Type: text/plain, Size: 1284 bytes --]

Leo Famulari <leo@famulari.name> writes:

> On Fri, Nov 24, 2017 at 04:09:58PM -0500, Kei Kebreau wrote:
>> * gnu/packages/package-management.scm (diffoscope)[arguments]: Replace readelf
>> substitution and add stat and getfacl substitutions.
>> [inputs]: Add acl.
>
> Can you add an example of the error messages in the commit message?
> Otherwise LGTM.
>

Like this?

Errors fixed include the following:
  "FileNotFoundError: [Errno 2] No such file or directory: 'readelf'"
  "diffoscope.comparators.directory: Unable to find 'getfacl', some
   directory metadata differences might not be noticed."
  "diffoscope.comparators.directory: Unable to find 'stat'"
>> @@ -520,7 +521,6 @@ transactions from C or Python.")
>>      (build-system python-build-system)
>>      (arguments
>>       `(#:phases (modify-phases %standard-phases
>> -                  (add-before 'unpack 'n (lambda _ #t))
>>                    ;; setup.py mistakenly requires python-magic from PyPi, even
>>                    ;; though the Python bindings of `file` are sufficient.
>>                    ;; https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=815844
>
> Some leftover debugging? :)

You know it! ;)
I figured out that this is unnecessary if one uses --keep-failed when
building with multiple rounds.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 832 bytes --]

  reply	other threads:[~2017-11-25 17:53 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-11-24 21:09 [bug#29432] [PATCH] gnu: diffoscope: Fix error messages related to comparison tools Kei Kebreau
2017-11-25  0:17 ` Leo Famulari
2017-11-25 17:51   ` Kei Kebreau [this message]
2017-11-25 17:57     ` Leo Famulari
2017-11-25 21:20       ` bug#29432: " Kei Kebreau

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=873752jlfr.fsf@posteo.net \
    --to=kkebreau@posteo.net \
    --cc=29432@debbugs.gnu.org \
    --cc=leo@famulari.name \
    /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).