Your message dated Sun, 15 Jan 2017 19:48:30 +0000
with message-id <[email protected]>
and subject line Bug#843948: fixed in duck 0.12
has caused the Debian Bug report #843948,
regarding duck: [PATCH] Add exception for Emacs M-x shell (no colors)
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact [email protected]
immediately.)


-- 
843948: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=843948
Debian Bug Tracking System
Contact [email protected] with problems
--- Begin Message ---
Package: duck
Version: 0.10
Severity: wishlist
Tags: patch

By default program enables color. However if run inside standard Emacs
M-x shell, the color cored are not interpreted and the output looks
garbled.

Please consider applying the following patch which adds exception for
Emacs.

Consideration not to enable colors by default
-----------------------------------------------

(1) In general, it might be considered that programs wouldn't use
colors by default. It's quite impossible to select correct colors for
various terminals considering that user may have chosen their own
default backgrounds and foregrounds -- thus the program's choices may
not play well with the users' selected colors.

(2) Typically features are enabled by request.

E.g in git(1) colors are enabled explicitly.
>From e35a09133f782a205396288bbc5a52b7fba188ed Mon Sep 17 00:00:00 2001
From: Jari Aalto <[email protected]>
Date: Fri, 11 Nov 2016 06:32:26 +0200
Subject: [PATCH] Disable colors in Emacs M-x shell
Organization: Private
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit

Signed-off-by: Jari Aalto <[email protected]>
---
 duck | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/duck b/duck
index e89d982..8d09c9c 100755
--- a/duck
+++ b/duck
@@ -67,7 +67,17 @@ my $checksdir='/usr/share/duck/lib/checks';
 
 my $try_https=0;
 my $nocolor=0;
+
 our $enforceColor="auto";
+
+if (exists $ENV{'INSIDE_EMACS'})
+{
+    # Program is being run inside
+    # Emacs M-x shell buffer
+
+    $enforceColor = "never";
+}
+
 my @allowedColorVals=qw/always auto never/;
 my $showMissingHelpers;
 my $urlFixEnableOptions;
-- 
2.9.3


--- End Message ---
--- Begin Message ---
Source: duck
Source-Version: 0.12

We believe that the bug you reported is fixed in the latest version of
duck, which is due to be installed in the Debian FTP archive.

A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to [email protected],
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Simon Kainz <[email protected]> (supplier of updated duck package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing [email protected])


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512

Format: 1.8
Date: Mon, 12 Dec 2016 21:35:55 +0100
Source: duck
Binary: duck
Architecture: source all
Version: 0.12
Distribution: unstable
Urgency: low
Maintainer: Simon Kainz <[email protected]>
Changed-By: Simon Kainz <[email protected]>
Description:
 duck       - checks URLs in debian/control and debian/upstream files
Closes: 826694 834520 838167 838169 843948
Changes:
 duck (0.12) unstable; urgency=low
 .
   * Run checks in parallel.
 .
   * Add new parameter --tasks to specify how many checks may be
     started in parallel - Default value is 24.
 .
   * Do not show colors on dumb terminals (e.g. M-x shell in Emacs)
     (Closes: #843948)
 .
   * Do not use hardcoded /tmp/... path for temporary directories
     (Closes: #838169)
 .
   * Fix parsing of appstream-metadata files, handle screenshot elements
     with additional data. (Closes: #838167)
 .
   * Fix possible include path issues during build time.
     Thanks to Dominic Hargreaves <[email protected]> for the patch.
     (Closes: #834520)
 .
   * Verify SSL certificates against ca-certificates. (Closes: #826694)
 .
   * Add new command line option --no-check-certificate to enable ignoring
     SSL failures on browser-style checks. (Closes: #826694)
Checksums-Sha1:
 e23054fa12b699ac8e1bb8469dd98ec545b63fbc 1882 duck_0.12.dsc
 4184ea76c95b95902fab8c7f09797a0e27d3903a 388188 duck_0.12.tar.xz
 6485ac07fc0d645948f955d67119cb9659dac974 22354 duck_0.12_all.deb
Checksums-Sha256:
 9fa3d71c9d59a8f0e8326d441599fa1cf7a5d1e9377d00dc92bfc495cfbef195 1882 
duck_0.12.dsc
 fdd628ad6cc42b08daad6884347d0a90fe9a62083f10b62e05497efcb7b6f8c3 388188 
duck_0.12.tar.xz
 cafe280f970e6891d5ee64d8ade71318aa00ae487c7850814a664de9df275df4 22354 
duck_0.12_all.deb
Files:
 bf56c28f921c35894b929acdfb00ad07 1882 devel optional duck_0.12.dsc
 4d593262f0f8bc38eaa12d776a4cc04b 388188 devel optional duck_0.12.tar.xz
 809d4f4d584c28dd89ef9f75b6f3adbc 22354 devel optional duck_0.12_all.deb

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1

iQIcBAEBCgAGBQJYe85MAAoJEBy08PeN7K/pWo0P/jJ9wD6L/VDge0aeXStIIpRl
nKJVhLWPq5Gx0+C/3RMCF7vD/XdYcayuhK1hfnuz0iyvJaiNMtdLuPw0SyUCYlh8
0dcSPksKPTH10jVv4Yk7v2Cbx8MX8xSw23Bs07KFKWTh9DmPxand63rNDf3Eos9G
uXhijV9v4KO1wg+FCjFb7NMNBIY0pKSdK6rN/ZTi4qYwotDMnqNefxnxZSxmuND2
ULBkDMmNrbLvKx8zQJ6kg/WnOo4NADk/DU3CDy6I8dpIg5snyHm6BIPV5DzJe/EQ
gw1POEkByILmRhSh60cvS9SDYKWHMPRJhROpxsS23VRSJLgnwZc3JNwohFRYp+/Q
qg1h2VDwnuia6fhW+F+ewCGL+lo48tVNI/JRhVMe03e713pi91RAII8uw5xR2Aqr
bq0MM4y5uLlYCEA/iuldDGrO7sjK3vxQ2rUWY4Db0ZZq1LwIrMDZB5bd7qLbJqYG
MUL9Ue5l7Ys2+zD8m2ikpaNtx/PIy/2L9cfzwXwR7oE6g6V3evmZIvZUWKqnacck
TFFa6nsKjO4n5p/bizgBzcd9bqivOFCmJM2rHN9AGtU93vjWxPLklwnk1oXWiK0P
/YvPCqb6G7KkAGQrfeKcMz753jQMn/VPNvvs2h/WRNIkpZNxJb6kM8HCtBHL3YRQ
FwRrYvzNoKg5LdP8Ebe0
=MSag
-----END PGP SIGNATURE-----

--- End Message ---

Reply via email to