Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Redirect all (i.e. non-Python) application output to log file #1239

Closed
remram44 opened this issue Apr 28, 2015 · 0 comments
Closed

Redirect all (i.e. non-Python) application output to log file #1239

remram44 opened this issue Apr 28, 2015 · 0 comments
Assignees
Milestone

Comments

@remram44
Copy link
Contributor

@remram44 remram44 commented Apr 28, 2015

Right now, all output from Python code, on sys.stdout/sys.stderr, is redirected to the log file.

Some people seem to be now arguing in favor of redirecting at the OS-level, so that no output would ever reach the terminal.

Pros:

  • No output in terminal
  • If some of that output is important, it would be in the log that would be submitted on error (I don't think any of it is important)

Cons:

  • Ugly non-portable system hack
  • Not Python
  • Unexpected madness; the system prints diagnostics messages related to UI toolkits, we shouldn't be messing with that
  • Clogging the logfile: these messages are often warnings from the UI toolkit or Qt, about mismatching fonts and compatiblity modes, which are completely useless to diagnose application-level errors
  • This is not what anybody does. Run any random Linux application from a terminal, you will see tons of warnings; why is this more of a problem to us than anybody else?
  • Warning/debug output from UV-CDAT components should be logged the right way. Leftover debug code should be dealt with, and will be annoying when running scripts with no recourse
  • Again, ugly non-Python system hack

For the record, I am also highly opposed to redirecting sys.stdout/sys.stderr in the first place (just use Python's convenient logging facilities).

@remram44 remram44 self-assigned this Apr 28, 2015
@doutriaux1 doutriaux1 added this to the 2.3 milestone Apr 28, 2015
@remram44 remram44 changed the title Redirection all (i.e. non-Python) application output to log file Redirect all (i.e. non-Python) application output to log file May 11, 2015
@aashish24 aashish24 assigned chaosphere2112 and unassigned remram44 Oct 15, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

3 participants