unofficial mirror of bug-guile@gnu.org 
 help / color / mirror / Atom feed
From: David Pirotte <david@altosw.be>
To: Mark H Weaver <mhw@netris.org>
Cc: 36677@debbugs.gnu.org, Robert Vollmert <rob@vllmrt.net>
Subject: bug#36677: [PATCH] Don't truncate backtraces
Date: Sun, 21 Jul 2019 19:59:43 -0300	[thread overview]
Message-ID: <20190721195943.096ebac6@capac> (raw)
In-Reply-To: <87ef2ownkf.fsf@netris.org>

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

Hello Mark,

> > This change was prompted by recent discussion on the Guix lists:
> >   https://lists.gnu.org/archive/html/guix-devel/2019-07/msg00207.html
> > In Guix, the truncation of stack traces frequently obscures
> > important information due to the long filenames.  

> I'm sympathetic to this problem, but simply disabling the truncated
> printing during backtraces is not workable.  It is quite often the case
> that some of the structures printed in backtraces are *huge*, or even
> cyclic.

I am very pleased to read that you think it is important to enable truncated
printing as a default for backtrace, I think so to. But maybe Guile could provide an
easy mechanism to overwrite these defaults, using procedures, or parameters? (not
depending on an 'external' variable I mean (*)

I wrote "these defaults", "procedures or parameters", using plural, because I think
that the default should also enable truncated printing for the repl and the
raised exception system, what do you think?

I wrote about this a couple of times, and as a gentle ping, here is my last email
about this request, which is a good summary which also points to other discussion on
this topic:

	https://lists.gnu.org/archive/html/guile-devel/2019-05/msg00034.html

David.

(*)	if an easy mechanism would depends on variables, let's make these Guile
variable then. like GUILE_BACKTRACE_PRINTER_TO_USE_N_COLUMN_AT_MOST
 (or what ever, I am not the best to name things ...),
GUILE_REPL_PRINTER_TO_USE_N_COLUMN_AT_MOST and
GUILE_RAISED_EXCEPTION_SYSTEM_TO_USE_N_COLUMN_AT_MOST, with -1 meaning no
truncated printing ...

But I would prefer procedures to set these, 'keeping' the default to be what
truncated-print uses as defined 'now', in  (ice-9 pretty-print), so we could use
them in our repl, our .guile, or the global init.scm setting, and change that on the
fly as we wish ... as for the procedure names, or one procedure and two args, one
for the parmeter to set, one for the value ... let's think about it ...



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

  parent reply	other threads:[~2019-07-21 22:59 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-07-15 20:29 bug#36677: [PATCH] Don't truncate backtraces Robert Vollmert
2019-07-17 17:57 ` Mark H Weaver
2019-07-17 18:11   ` Robert Vollmert
2019-07-21 15:35     ` Robert Vollmert
2019-07-21 22:59   ` David Pirotte [this message]
2019-07-22  0:33     ` Mark H Weaver
2019-07-25 16:27       ` David Pirotte

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://www.gnu.org/software/guile/

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

  git send-email \
    --in-reply-to=20190721195943.096ebac6@capac \
    --to=david@altosw.be \
    --cc=36677@debbugs.gnu.org \
    --cc=mhw@netris.org \
    --cc=rob@vllmrt.net \
    /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).