Merge branch 'next' of http://github.com/erkin/ponysay into next

manuals/ponysay.texinfo

@@ -72,7 +72,7 @@ Texts. A copy of the license is included in the section entitled
 * Pony metadata::                       Metadata tags in the pony files.
 * The tool chest::                      Extra Ponysay commands for other stuff than printing ponies.
 * Limitations::                         Known limitations that may not be that easy to overcome.
-* Problems and requests::               Report issues and making requests.
+* Problems and requests::               External bugs, report issues and making requests.
 * Dependencies::                        Ponysay's dependencies.
 * Installing::                          How to install @command{ponysay}.
 * Inner workings::                      Useful information for those whom want to help hack @command{ponysay}.
@@ -900,14 +900,14 @@ is) may be added if required.
 This tag decribes what kind of pony a pony is, it is a comma seperated lower case list,
 and it cannot be empty, by it can be (but shouldn't) skipped for every pony in the image.
 
-Every fitting value should be used, however a alicorn (also known as alacord or winged
+Every fitting value should be used, however a alicorn (also known as alacorn, winged
-unicorn) should have the values @code{pegasus}, @code{unicorn}, @code{pony}, but not
+unicorn or pegacorn) should have the values @code{alicorn} and @code{pony}, but neither
-@code{alicorn}. Earth ponies should have the value @code{pony} but not @code{earth} or
+@code{pegasus} nor @code{unicorn}. Earth ponies should have the value @code{pony} and
-@code{earth pony}.
+@code{earth} (not @code{earth pony}.)
 
 The standard values are (you may use other ones if fitting): @code{unicorn}, @code{pegasus},
-@code{pony}, @code{changeling}, @code{crystal}, @code{seapony}, @code{animal}
+@code{alicorn}, @code{earth}, @code{pony}, @code{changeling}, @code{crystal}, @code{seapony},
-(applies to Spike) and @code{item} (applies to Tom and Pinkamina's imaginare friends).
+@code{animal} (applies to Spike) and @code{item} (applies to Tom and Pinkamina's imaginare friends.)
 
 @item GROUP
 @vindex @var{GROUP}
@@ -1466,16 +1466,67 @@ the top to the left.
 
 @node Problems and requests
 @chapter Problems and requests
+@cindex problems
 
 @menu
-* Problems::           Reporting bugs.
+* External bugs::         Known external bugs.
-* Requests::           Requesting ponies.
+* Reporting bugs::        Reporting bugs and issues in ponysay.
+* Requesting ponies::     Requesting inclusion of your favourite ponies.
 @end menu
 
 
-@node Problems
+@node External bugs
+@section External bugs
+@cindex external bugs
+@cindex bugs, external
+
+@cindex vte-based terminals
+@cindex terminals
+@pindex @command{mate-terminal}
+@pindex @command{gnome-terminal}
+@pindex @command{terminator}
+There is only one known bug that may occour that is external to ponysay,
+meaning that it is a bug that can occour with ponysay, but is not actually
+a bug in ponysay. This bug is common for programs that prints a lot of
+colours, even with @command{cat}; it is only known to happen for VTE-based
+terminals, such as @command{mate-terminal}, @command{gnome-terminal} and
+@command{terminator}.
+
+The bug, is that lines (often no more than one line) can skipped when all
+lines move up one step, the next line is skipped instead, and so one; or
+that an escape sequence is interpreted as pure text (this is common in GNU
+Emacs, however GNU Emacs does not support programs such as @command{ponysay}.)
+
+This bug can often be suppressed by piping the output to @command{cat}
+multiple times when using a VTE-based terminal (or always if you prefer).
+The use of VTE-based terminal can often be determined by checking for the
+environment variable @var{COLORTERM}, to which VTE-based terminal usually
+export their name (some reported another terminals name.)
+
+If you want to do this in GNU Bash you can add this (with possible modifications
+depending on what you also have done with @command{ponysay}) code sample
+to your @file{~/.bashrc} file.
+
+@cartouche
+@example
+[[ ! "$COLORTERM" = "" ]] &&
+    function ponysay
+    {
+        exec ponysay "$@@" | cat | cat | cat | cat
+    }
+@end example
+@end cartouche
+
+It important for this bug workaround that @command{cat} is unbuffered, which
+is default in GNU's version of @command{cat}, but not in Unix's version.
+If this does not work, test adding the option @option{-u} to @command{cat}.
+
+
+
+@node Reporting bugs
 @section Reporting bugs
-@cindex bugs
+@cindex bugs, reporting
+@cindex report bugs
 
 If you find a bug in @command{ponysay}, install the last version
 from @url{https://github.com/erkin/ponysay}, and if it is still
@@ -1484,9 +1535,10 @@ Please be as descriptive as possible, as it will help us verify it
 solve it faster.
 
 
-@node Requests
+@node Requesting ponies
 @section Requesting ponies
 @cindex pony requests
+@cindex request ponies
 
 If you want I specific pony added, ask us at
 @url{https://github.com/erkin/ponysay/issues} and we will add it.

ponysay.py

@@ -65,9 +65,10 @@ fd3 = None
 '''
 def printinfo(text = '', end = '\n'):
     global fd3
-    if os.path.exists('/proc/self/fd/3'):
+    if os.path.exists('/proc/self/fd/3') and os.path.isfile('/proc/self/fd/3'):
         if fd3 is None:
             fd3 = os.fdopen(3, 'w')
+    if fd3 is not None:
         fd3.write(str(text) + end)