Re: [Django] #17221: `manage.py` (do not) sends ANSI escape sequences when terminal (can) can't handle them

[Django]

#17221: `manage.py` (do not) sends ANSI escape sequences  when terminal (can) 
can't
handle them
-+-
 Reporter:  rabio|Owner:  nobody
 Type:  Bug  |   Status:  closed
Component:  Core (Management |  Version:  SVN
  commands)  |   Resolution:
 Severity:  Normal   |  worksforme
 Keywords:   | Triage Stage:
Has patch:  0|  Unreviewed
  Needs tests:  0|  Needs documentation:  0
Easy pickings:  0|  Patch needs improvement:  0
 |UI/UX:  0
-+-
Changes (by aaugustin):

 * status:  new => closed
 * needs_better_patch:   => 0
 * resolution:   => worksforme
 * needs_tests:   => 0
 * needs_docs:   => 0


Comment:

 I agree that Django's detection for color support in terminals isn't
 perfect in theory. However, this isn't a core functionality of the
 framework at all, and the current implementation appears to work
 sufficiently well in practice.

 Could you describe a reasonable use case where the current code fails? If
 it works fine in most cases, I won't feel a compelling need to re-
 implement it.

 Given this, and the lack of precise ideas for improvements or a solid
 patch, I'll close this ticket.

 However, if it's really a problem that occurs in common setups, or you
 have a well-tested patch that improves a realistic use case, I'd be glad
 to revisit this decision.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To post to this group, send email to django-updates@googlegroups.com.
To unsubscribe from this group, send email to 
django-updates+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/django-updates?hl=en.

[Django] #17221: `manage.py` (do not) sends ANSI escape sequences when terminal (can) can't handle them

[Django]

#17221: `manage.py` (do not) sends ANSI escape sequences  when terminal (can) 
can't
handle them
+
 Reporter:  rabio   |  Owner:  nobody
 Type:  Bug | Status:  new
Component:  Core (Management commands)  |Version:  SVN
 Severity:  Normal  |   Keywords:
 Triage Stage:  Unreviewed  |  Has patch:  0
Easy pickings:  0   |  UI/UX:  0
+
 `manage.py` may sends ANSI escape sequences for output colourization.
 Unfortunately, guessing if terminal support colours is broken. Guessing if
 output colourisation is desirable do not take all possible use cases into
 account.

 === Rationale: ===

 As it is now, partial checking is made (only platform is checked by
 `django.core.management.color.supports_color()`) and `manage.py` may sends
 colour codes even when terminal can not handle them properly (as it is a
 case for most, if not all, dumb terminals).

 For guessing to work correctly, checking if terminal can handle colours
 need to be reimplemented. For this, potentially `curses.has_colors()`
 could be used but I'm not sure is it supported on windows, though. In
 addition, `django.utils.termcolors` allow to request "blink" option
 (fortunately not used in default palettes), but not all terminals that
 support colours know how to blink.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To post to this group, send email to django-updates@googlegroups.com.
To unsubscribe from this group, send email to 
django-updates+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/django-updates?hl=en.