public inbox for dwz@sourceware.org
 help / color / mirror / Atom feed
From: "Frank Ch. Eigler" <fche@elastic.org>
To: Mark Wielaard <mark@klomp.org>
Cc: dwz@sourceware.org
Subject: Re: patch testsuite - verbose logging
Date: Tue, 12 Jul 2022 10:09:00 -0400	[thread overview]
Message-ID: <Ys2AfLG6dSS/YM7H@elastic.org> (raw)
In-Reply-To: <4e51d3d6f9f1671bac7549c88318bc68a7c4034e.camel@klomp.org>

Hi -

> I agree it would be good to have a bit more information in the dwz.log
> file. But -x is very verbose. I might get lost. But maybe others think
> it is fine. I am not against it. But I think just capturing stdout and
> verbose log it is fine. We can add some extra echos in the tests.

There is no 'verbose log' content in the .log file possible at the
present in the sense of RUNTESTFLAGS=-v.  These log files are for
helping someone debug failing tests, so normally one does not need to
look at them, let alone get lost.

> > Accidentally, this makes debuginfod-involving tests pass, since their
> > stderr DEBUGINFOD_PRGRESS traffic show up in stdout rather than
> > stderr (which is recorded as a fail).
> 
> I think that is a reason to not do the 2>@ redirection.
> We did recently find an issue with gdb-add-index because it had error
> output.  And I think we would like to know if the tests accidentially
> triggered debuginfod [...]

A simpler cure to that is to have the testuite makefile driver unset
$DEBUGINFOD_URLS before invoking dejagnu.

- FChE

      reply	other threads:[~2022-07-12 14:09 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-07-06 17:25 Frank Ch. Eigler
2022-07-12 13:49 ` Mark Wielaard
2022-07-12 14:09   ` Frank Ch. Eigler [this message]

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

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

  git send-email \
    --in-reply-to=Ys2AfLG6dSS/YM7H@elastic.org \
    --to=fche@elastic.org \
    --cc=dwz@sourceware.org \
    --cc=mark@klomp.org \
    /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.
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).