Merge pull request #217 from kachick/correct-shells

Restrict shellscripts
kachick · Jul 30, 2023 · 0faf989 · 0faf989
2 parents e515a42 + a390dfb
commit 0faf989
Show file tree

Hide file tree

Showing 5 changed files with 16 additions and 3 deletions.
diff --git a/Makefile.toml b/Makefile.toml
@@ -61,8 +61,11 @@ args = [
 ]
 
 [tasks.apply]
-script = [
-  "home-manager switch -f ./home/.config/home-manager/home.nix",
+command = 'home-manager'
+args = [
+  'switch',
+  '-f',
+  './home/.config/home-manager/home.nix',
 ]
 
 [tasks.enable_login_shells]

diff --git a/home/.local/share/homemade/bin/bench_shells.bash b/home/.local/share/homemade/bin/bench_shells.bash
@@ -1,5 +1,7 @@
 #!/usr/bin/env bash
 
+set -euo pipefail
+
 # ~ my feeling ~
 # 50ms : blazing fast!
 # 120ms : acceptable

diff --git a/home/.local/share/homemade/bin/la.bash b/home/.local/share/homemade/bin/la.bash
@@ -1,3 +1,5 @@
 #!/usr/bin/env bash
 
+set -euo pipefail
+
 exa --long --all --group-directories-first "$@"
diff --git a/home/.local/share/homemade/bin/updeps.bash b/home/.local/share/homemade/bin/updeps.bash
@@ -1,5 +1,7 @@
 #!/usr/bin/env bash
 
+set -euo pipefail
+
 case ${OSTYPE} in
 linux*)
 	sudo apt update --yes && sudo apt upgrade --yes

diff --git a/home/.local/share/homemade/bin/zj.bash b/home/.local/share/homemade/bin/zj.bash
@@ -1,3 +1,7 @@
 #!/usr/bin/env bash
 
-zellij attach "$(basename "$PWD")" || zellij --session "$(basename "$PWD")"
+set -euo pipefail
+
+name="$(basename "$PWD")"
+
+zellij attach "$name" || zellij --session "$name"