public inbox for systemtap@sourceware.org
 help / color / mirror / Atom feed
* [Bug translator/20510] New: stap -L colorizes non-tty stdout
@ 2016-08-23 19:47 jistone at redhat dot com
  2016-08-24 15:29 ` [Bug translator/20510] " flu at redhat dot com
  0 siblings, 1 reply; 2+ messages in thread
From: jistone at redhat dot com @ 2016-08-23 19:47 UTC (permalink / raw)
  To: systemtap

https://sourceware.org/bugzilla/show_bug.cgi?id=20510

            Bug ID: 20510
           Summary: stap -L colorizes non-tty stdout
           Product: systemtap
           Version: unspecified
            Status: NEW
          Severity: normal
          Priority: P2
         Component: translator
          Assignee: systemtap at sourceware dot org
          Reporter: jistone at redhat dot com
  Target Milestone: ---

Commit 1ad27a4b0b6aa for PR14787 added colorize() calls to stap -L output. 
That coloring is predicated on color_errors = isatty(STDERR_FILENO), which is
fine for error messages that do go on stderr, but -L writes to stdout.

So if you run a command like "stap -L 'syscall.*' >syscall.list", stderr is
still a tty, and that file will include a bunch of colorization escapes.

-- 
You are receiving this mail because:
You are the assignee for the bug.

^ permalink raw reply	[flat|nested] 2+ messages in thread

* [Bug translator/20510] stap -L colorizes non-tty stdout
  2016-08-23 19:47 [Bug translator/20510] New: stap -L colorizes non-tty stdout jistone at redhat dot com
@ 2016-08-24 15:29 ` flu at redhat dot com
  0 siblings, 0 replies; 2+ messages in thread
From: flu at redhat dot com @ 2016-08-24 15:29 UTC (permalink / raw)
  To: systemtap

https://sourceware.org/bugzilla/show_bug.cgi?id=20510

Felix Lu <flu at redhat dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
                 CC|                            |flu at redhat dot com
         Resolution|---                         |FIXED

--- Comment #1 from Felix Lu <flu at redhat dot com> ---
Commit 2d6ef45b30fa.

-- 
You are receiving this mail because:
You are the assignee for the bug.

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2016-08-24 15:29 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-08-23 19:47 [Bug translator/20510] New: stap -L colorizes non-tty stdout jistone at redhat dot com
2016-08-24 15:29 ` [Bug translator/20510] " flu at redhat dot com

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).