Mirrors/fish-shell
2
0
Fork 0
mirror of https://github.com/fish-shell/fish-shell synced 2024-12-28 05:43:11 +00:00
Code Issues Projects Releases Packages Wiki Activity

Make term warning less shouty

Browse source 
We don't need to know that it tried these five before finally getting
one, the list is *right there*.

It is also very unlikely that someone has "xterm" or "ansi" but not "xterm-256color"

For xterm-256color, we don't warn *at all* because we have that one hardcoded.
This commit is contained in:
Fabian Boehm 2024-02-05 20:15:41 +01:00
parent 75f7cda6ab
commit 785d784482
1 changed files with 15 additions and 20 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

35
src/env_dispatch.rs
View file

@ -518,17 +518,9 @@ fn initialize_curses_using_fallbacks(vars: &EnvStack) {
// `term` here is one of our hard-coded strings above; we can unwrap because we can
// guarantee it doesn't contain any interior NULs.
success = curses::setup(Some(&term), |term| apply_term_hacks(vars, term)).is_some();
if is_interactive_session() {
if success {
FLOG!(warning, wgettext!("Using fallback terminal type"), term);
} else {
FLOG!(
warning,
wgettext!("Could not set up terminal using the fallback terminal type"),
term,
);
}
success = curses::setup(Some(term), |term| apply_term_hacks(vars, term)).is_some();
if is_interactive_session() && success {
FLOG!(warning, wgettext!("Using fallback terminal type"), term);
}
if success {
@ -661,15 +653,18 @@ fn init_curses(vars: &EnvStack) {
if curses::setup(None, |term| apply_term_hacks(vars, term)).is_none() {
if is_interactive_session() {
let term = vars.get_unless_empty(L!("TERM")).map(|v| v.as_string());
FLOG!(warning, wgettext!("Could not set up terminal."));
if let Some(term) = term {
FLOG!(warning, wgettext!("TERM environment variable set to"), term);
FLOG!(
warning,
wgettext!("Check that this terminal type is supported on this system.")
);
} else {
FLOG!(warning, wgettext!("TERM environment variable not set."));
// We do not warn for xterm-256color at all, we know that one.
if term != Some("xterm-256color".into()) {
FLOG!(warning, wgettext!("Could not set up terminal."));
if let Some(term) = term {
FLOG!(warning, wgettext!("TERM environment variable set to"), term);
FLOG!(
warning,
wgettext!("Check that this terminal type is supported on this system.")
);
} else {
FLOG!(warning, wgettext!("TERM environment variable not set."));
}
}
}