[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
141/376: Filter Nix-specific ANSI escape sequences from stderr
From: |
Ludovic Court�s |
Subject: |
141/376: Filter Nix-specific ANSI escape sequences from stderr |
Date: |
Wed, 28 Jan 2015 22:04:35 +0000 |
civodul pushed a commit to tag 1.8
in repository guix.
commit 954188af2747e796eb4428c493db9aa06ed1af38
Author: Eelco Dolstra <address@hidden>
Date: Wed Aug 20 14:17:07 2014 +0200
Filter Nix-specific ANSI escape sequences from stderr
The Nixpkgs stdenv prints some custom escape sequences to denote
nesting and stuff like that. Most terminals (e.g. xterm, konsole)
ignore them, but some do not (e.g. xfce4-terminal). So for the benefit
of the latter, filter them out.
---
src/libstore/build.cc | 38 +++++++++++++++++++++++++++++++++++++-
1 files changed, 37 insertions(+), 1 deletions(-)
diff --git a/src/libstore/build.cc b/src/libstore/build.cc
index aa539a1..ef0a304 100644
--- a/src/libstore/build.cc
+++ b/src/libstore/build.cc
@@ -2441,6 +2441,42 @@ void DerivationGoal::deleteTmpDir(bool force)
}
+/* Filter out the special ANSI escape codes generated by Nixpkgs'
+ stdenv (used to denote nesting etc.). */
+static string filterNixEscapes(const string & s)
+{
+ string t, r;
+ enum { stTop, stEscape, stCSI } state = stTop;
+ for (auto c : s) {
+ if (state == stTop) {
+ if (c == '\e') {
+ state = stEscape;
+ r = c;
+ } else
+ t += c;
+ } else if (state == stEscape) {
+ r += c;
+ if (c == '[')
+ state = stCSI;
+ else {
+ t += r;
+ state = stTop;
+ }
+ } else {
+ r += c;
+ if (c >= 0x40 && c != 0x7e) {
+ if (c != 'p' && c != 'q' && c != 's' && c != 'a' && c != 'b')
+ t += r;
+ state = stTop;
+ r.clear();
+ }
+ }
+ }
+ t += r;
+ return t;
+}
+
+
void DerivationGoal::handleChildOutput(int fd, const string & data)
{
if ((hook && fd == hook->builderOut.readSide) ||
@@ -2455,7 +2491,7 @@ void DerivationGoal::handleChildOutput(int fd, const
string & data)
return;
}
if (verbosity >= settings.buildVerbosity)
- writeToStderr(data);
+ writeToStderr(filterNixEscapes(data));
if (bzLogFile) {
int err;
BZ2_bzWrite(&err, bzLogFile, (unsigned char *) data.data(),
data.size());
- 137/376: Reduce verbosity, (continued)
- 137/376: Reduce verbosity, Ludovic Court�s, 2015/01/28
- 133/376: Fix download-via-ssh, Ludovic Court�s, 2015/01/28
- 130/376: nix-log2xml: Handle newlines, Ludovic Court�s, 2015/01/28
- 143/376: nix-store -l: Automatically pipe output into $PAGER, Ludovic Court�s, 2015/01/28
- 139/376: Fix --attr parsing, Ludovic Court�s, 2015/01/28
- 142/376: Reduce test verbosity, Ludovic Court�s, 2015/01/28
- 138/376: Doh, Ludovic Court�s, 2015/01/28
- 135/376: nix-build: Propagate exit status from nix-store -r, Ludovic Court�s, 2015/01/28
- 125/376: Refactor option handling, Ludovic Court�s, 2015/01/28
- 134/376: build-remote.pl: Provide defaults for $NIX_CURRENT_LOAD and $NIX_REMOTE_SYSTEMS, Ludovic Court�s, 2015/01/28
- 141/376: Filter Nix-specific ANSI escape sequences from stderr,
Ludovic Court�s <=
- 140/376: Make hook shutdown more reliable, Ludovic Court�s, 2015/01/28
- 144/376: Add some color, Ludovic Court�s, 2015/01/28
- 147/376: Handle header file instantiation, Ludovic Court�s, 2015/01/28
- 152/376: Flush std::cout before closing stdout, Ludovic Court�s, 2015/01/28
- 146/376: Install config.h only once, Ludovic Court�s, 2015/01/28
- 148/376: Force template regeneration, Ludovic Court�s, 2015/01/28
- 158/376: fix disappearing bash arguments, Ludovic Court�s, 2015/01/28
- 150/376: Provide reasonable default flags for $LESS, Ludovic Court�s, 2015/01/28
- 149/376: Merge commit '2aa93858afee22e0c32d8f4366970976374091ac', Ludovic Court�s, 2015/01/28
- 156/376: Use PR_SET_PDEATHSIG to ensure child cleanup, Ludovic Court�s, 2015/01/28