Opened 12 years ago

Closed 12 years ago

Last modified 4 years ago

#18325 closed Cleanup/optimization (fixed)

Wrap management stdout/stderr

Reported by: Claude Paroz Owned by: nobody
Component: Core (Management commands) Version: dev
Severity: Normal Keywords:
Cc: Triage Stage: Unreviewed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

This cleanup is both a preparation to Python 3 (where sys.stdout expects Unicode) and to redirect output to logging (#15107).

Attachments (1)

18325.diff​ (20.1 KB ) - added by Claude Paroz 12 years ago.
Wrap self.stdout/selfstderr in OutputWrapper

Download all attachments as: .zip

Change History (4)

by Claude Paroz, 12 years ago

Attachment: 18325.diff​ added

Wrap self.stdout/selfstderr in OutputWrapper

comment:1 by Claude Paroz <claude@…>, 12 years ago

Resolution: → fixed
Status: new → closed

In [822d6d6dabc959532fb2904376580e8947c519f6]:

Fixed #18325 -- Wrapped self.stdout/stderr in OutputWrapper class

comment:2 by Claude Paroz <claude@…>, 12 years ago

In dcdaf9a07901c80c0d93663b348595dd2ba7b7a6:

Fixed error output from runserver

This has been missed in commit 822d6d6dab (Refs #18325).

comment:3 by Mariusz Felisiak <felisiak.mariusz@…>, 4 years ago

In 71d9876e:

Refs #18325 -- Removed unnecessary line endings in management commands.

The OutputWrapper automatically adds \n when it’s not present.

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