“error opening config file” is not an error #2

New Issue

Ghost · 2020-07-29T19:12:22+02:00

Ghost commented

2020-07-29 19:12:22 +02:00

How to reproduce: Create ~/.config/hkd/config and run hkd.

“error opening config file” gets printed although hkd works fine. A better message would be: “info: environment variable $XDG_CONFIG_HOME not set: falling back to ~/.config.” However, it would be even better to follow the rule of silence and print no message.

How to reproduce: Create `~/.config/hkd/config` and run `hkd`. “error opening config file” gets printed although `hkd` works fine. A better message would be: “info: environment variable $XDG_CONFIG_HOME not set: falling back to ~/.config.” However, it would be even better to follow the [rule of silence][1] and print no message. [1]: http://www.linfo.org/rule_of_silence.html

alema referenced this issue from a commit

2020-07-29 20:36:15 +02:00

closing #2: made the program silent turned off the "error" messages by default and using the -v flag to turn them back on as they are not useful for normal use, also now those messages are printed to stdout

alema commented

2020-07-29 20:37:13 +02:00

you're right, I didn't know about the rule of silence before and the "error" messages are not useful apart from debugging.
I made it so that all the messages are disabled by default and -v turns them back on.

you're right, I didn't know about the rule of silence before and the "error" messages are not useful apart from debugging. I made it so that all the messages are disabled by default and -v turns them back on.

alema closed this issue

2020-07-29 21:28:29 +02:00

alema referenced this issue from a commit

2020-08-29 20:45:36 +02:00

fixed #2 skip zero or 1 allocations and uninitialized conditional jump in hotkey_list_add() now the whole output of die() is coloured

Sign in to join this conversation.

No Label

No Milestone

No Assignees

2 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: alema/hkd#2