xenia/home
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

rc windowing wayland/x11: do not pollute terminal environment

Browse Source 
The x11-terminal command spawns a potentially long-lived terminal
process. The terminal can is completely independent of the Kakoune
session that created it.
Due to how it's implemented, the spawned terminals will have
environment variables "kak_opt_termcmd" and  "kak_quoted_reg_a"
set.  This can be surprising, especially since, by convention, the
environment contains no lowercase variables. Let's stop exporting them.
This commit is contained in:
Johannes Altmanninger 2022-12-11 19:30:02 +01:00
parent 2adc81c4c9
commit 94e3ea9687
2 changed files with 8 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
rc/windowing/wayland.kak
View File

@ -39,7 +39,10 @@ The program passed as argument will be executed in the new terminal' \
echo "fail 'termcmd option is not set'" echo "fail 'termcmd option is not set'"
exit exit
fi fi
setsid ${kak_opt_termcmd} "$kak_quoted_reg_a" < /dev/null > /dev/null 2>&1 & termcmd=$kak_opt_termcmd
args=$kak_quoted_reg_a
unset kak_opt_termcmd kak_quoted_reg_a
setsid ${termcmd} "$args" < /dev/null > /dev/null 2>&1 &
} }
} }
} }

5
rc/windowing/x11.kak
View File

@ -44,7 +44,10 @@ The program passed as argument will be executed in the new terminal' \
echo "fail 'termcmd option is not set'" echo "fail 'termcmd option is not set'"
exit exit
fi fi
setsid ${kak_opt_termcmd} "$kak_quoted_reg_a" < /dev/null > /dev/null 2>&1 & termcmd=$kak_opt_termcmd
args=$kak_quoted_reg_a
unset kak_opt_termcmd kak_quoted_reg_a
setsid ${termcmd} "$args" < /dev/null > /dev/null 2>&1 &
} }
} }
} }