Anonymous | Login | Signup for a new account | 2024-03-29 08:12 UTC |
My View | View Issues | Change Log | Roadmap | Doomseeker Issue Support Ranking | Rules | My Account |
View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] | ||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||
0003372 | Doomseeker | [All Projects] Bug | public | 2018-02-15 02:05 | 2018-10-27 22:55 | ||||
Reporter | Zalewa | ||||||||
Assigned To | Zalewa | ||||||||
Priority | normal | Severity | minor | Reproducibility | always | ||||
Status | closed | Resolution | fixed | ||||||
Platform | OS | OS Version | |||||||
Product Version | 1.1 | ||||||||
Target Version | 1.2 | Fixed in Version | 1.2 | ||||||
Summary | 0003372: Diagnostics messages are written to stdout (instead of stderr) | ||||||||
Description | Doomseeker's startup logs (etc) are like diagnostics messages, but treated as normal output from the program. This goes against POSIX' description of stderr:Quote stdout makes sense for --version-json as conventional JSON output, but the diagnostics messages should really be changed from stdout to stderr. | ||||||||
Steps To Reproduce | On Unix-like machines:doomseeker 1>/tmp/stdout.log 2>/tmp/stderr.log See /tmp/stdout.log and /tmp/stderr.log for messages of the appropriate standard stream. | ||||||||
Additional Information |
| ||||||||
Attached Files | |||||||||
Relationships | ||||||
|
Notes | |
(0019042) WubTheCaptain (reporter) 2018-02-15 02:08 |
This was fixed:'https://bitbucket.org/Doomseeker/doomseeker/commits/87122ce09493fbad5fe718acf64ef2950167fd13 [^]' I thought of submitting a bug for the bug tracker for future reference, anyway. |
(0019043) WubTheCaptain (reporter) 2018-02-15 02:21 edited on: 2018-02-15 03:20 |
Interestingly enough, I would like to reopen this issue for more discussion. One of the answers in the linked Unix & Linux Stack Exchange thread argues whether POSIX-like programs should be quiet by default (that's an Unix-y thing to do), and only output to stderr if the exit status was non-zero. To quote: Quote from POSIX.1-2008, 1.4 Utility Description Defaults Do we need to introduce a command line flag for --quiet or --verbose? I'd keep them to stderr, though. |
(0019044) WubTheCaptain (reporter) 2018-02-15 02:30 |
Quote from WubTheCaptain Haha disregard that, stdout probably makes more sense with a --verbose (or -v) flag with quiet by default behavior. Anyone agree? stderr for real diagnostics errors, though. |
(0019050) Zalewa (developer) 2018-02-15 07:37 |
--version-json could indeed benefit from not being spammed with stderr logs by default |
(0019051) Zalewa (developer) 2018-02-15 08:41 |
"--quiet" and "--verbose" added here:'https://bitbucket.org/Doomseeker/doomseeker/commits/53e2af801b780f3d233bd8a24ea558535d887efd [^]' |
(0019748) WubTheCaptain (reporter) 2018-09-25 03:11 edited on: 2018-09-25 03:12 |
As said, this has been resolved. Side-note: It seems like --verbose is useless at this point and --quiet works as intended. The output isn't quiet by default (except on stdout), but uh, we don't do much with stdout anyway. It's not like Doomseeker had stdout of its server lists in some specific format for other applications to use. |
This issue is already marked as resolved. If you feel that is not the case, please reopen it and explain why. |
|
Supporters: | No one explicitly supports this issue yet. |
Opponents: | No one explicitly opposes this issue yet. |
Issue History | |||
Date Modified | Username | Field | Change |
2018-02-15 02:05 | WubTheCaptain | New Issue | |
2018-02-15 02:05 | WubTheCaptain | Status | new => assigned |
2018-02-15 02:05 | WubTheCaptain | Assigned To | => Zalewa |
2018-02-15 02:06 | WubTheCaptain | Relationship added | related to 0003282 |
2018-02-15 02:06 | WubTheCaptain | Reporter | WubTheCaptain => Zalewa |
2018-02-15 02:06 | WubTheCaptain | Status | assigned => resolved |
2018-02-15 02:06 | WubTheCaptain | Resolution | open => fixed |
2018-02-15 02:06 | WubTheCaptain | Fixed in Version | => 1.2 |
2018-02-15 02:08 | WubTheCaptain | Note Added: 0019042 | |
2018-02-15 02:21 | WubTheCaptain | Note Added: 0019043 | |
2018-02-15 02:21 | WubTheCaptain | Status | resolved => feedback |
2018-02-15 02:21 | WubTheCaptain | Resolution | fixed => reopened |
2018-02-15 02:23 | WubTheCaptain | Summary | Logs are written to stdout instead of stderr => Diagnostics logs are written to stdout (instead of stderr) |
2018-02-15 02:27 | WubTheCaptain | Summary | Diagnostics logs are written to stdout (instead of stderr) => Diagnostics messages are written to stdout (instead of stderr) |
2018-02-15 02:30 | WubTheCaptain | Note Added: 0019044 | |
2018-02-15 03:20 | WubTheCaptain | Note Edited: 0019043 | View Revisions |
2018-02-15 07:37 | Zalewa | Note Added: 0019050 | |
2018-02-15 07:37 | Zalewa | Status | feedback => assigned |
2018-02-15 08:41 | Zalewa | Note Added: 0019051 | |
2018-02-15 08:41 | Zalewa | Status | assigned => needs testing |
2018-09-25 03:11 | WubTheCaptain | Note Added: 0019748 | |
2018-09-25 03:11 | WubTheCaptain | Status | needs testing => resolved |
2018-09-25 03:11 | WubTheCaptain | Resolution | reopened => fixed |
2018-09-25 03:12 | WubTheCaptain | Note Edited: 0019748 | View Revisions |
2018-10-27 22:55 | WubTheCaptain | Status | resolved => closed |
Copyright © 2000 - 2024 MantisBT Team |