diff options
| author |  Fabian Homborg <FHomborg@gmail.com> | 2016-01-18 21:56:59 +0100 |
|---|---|---|
| committer | Fabian Homborg <FHomborg@gmail.com> | 2016-01-18 21:56:59 +0100 |
| commit | 4b2495f28fa420f6271b336b292556bce442b7d3 (patch) | |
| tree | 493c5da7f3b5b2eb70f85f6f9c160dae733df6c7 /share/functions/__fish_config_interactive.fish | |
| parent | 766176443d6edbbd47f1dbe1080d9d1becf2b0c0 (diff) | |
Skip greeting on empty variable
It used to be that way and we recommend `set fish_greeting` (i.e. set to
empty) in the docs - possibly since we check if the variable is defined
on upgrade.
Diffstat (limited to 'share/functions/__fish_config_interactive.fish')
| -rw-r--r-- | share/functions/__fish_config_interactive.fish | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/share/functions/__fish_config_interactive.fish b/share/functions/__fish_config_interactive.fish index 0dbd3ee1..3702c075 100644 --- a/share/functions/__fish_config_interactive.fish +++ b/share/functions/__fish_config_interactive.fish @@ -92,7 +92,9 @@ function __fish_config_interactive -d "Initializations that should be performed if functions -q fish_greeting fish_greeting else - set -q fish_greeting; and echo $fish_greeting + # The greeting used to be skipped when fish_greeting was empty (not just undefined) + # Keep it that way to not print superfluous newlines on old configuration + test -n "$fish_greeting"; and echo $fish_greeting end # |