Opened 9 years ago

Closed 9 years ago

#5319 closed (fixed)

Java is another special case for terminal colors

Reported by: Marty Alchin <gulopine@…> Owned by: Adrian Holovaty
Component: Core (Management commands) Version: master
Severity: Keywords: jython
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:

Description

Jython doesn't fit into the existing tests for terminal colors, as its sys.stdout doesn't have an isatty() method.

Attachments (1)

color.diff (601 bytes) - added by Marty Alchin <gulopine@…> 9 years ago.
Added a test for Java so that Jython doesn't use terminal colors

Download all attachments as: .zip

Change History (4)

Changed 9 years ago by Marty Alchin <gulopine@…>

Attachment: color.diff added

Added a test for Java so that Jython doesn't use terminal colors

comment:1 Changed 9 years ago by anonymous

Needs documentation: unset
Needs tests: unset
Patch needs improvement: unset
Triage Stage: UnreviewedReady for checkin

comment:2 Changed 9 years ago by Simon G. <dev@…>

(sorry - the above triage was by me).

comment:3 Changed 9 years ago by Adrian Holovaty

Resolution: fixed
Status: newclosed

(In [6032]) Fixed #5319 -- Changed terminal colors in django.core.management.colors not to be used under Jython. Thanks, Marty Alchin

Note: See TracTickets for help on using tickets.
Back to Top