Bagaimana saya bisa mengeraskan skrip bash agar tidak menyebabkan kerusakan saat diubah di masa mendatang?

Jadi, saya menghapus folder rumah saya (atau, lebih tepatnya, semua file yang saya tulis aksesnya). Apa yang terjadi adalah yang saya miliki

build="build"
...
rm -rf "${build}/"*
...
<do other things with $build>

dalam skrip bash dan, setelah tidak perlu lagi $build, menghapus deklarasi dan semua penggunaannya - tetapi rm. Bash dengan senang hati mengembang rm -rf /*. Ya.

Saya merasa bodoh, menginstal cadangan, redid pekerjaan yang hilang. Mencoba melewati rasa malu.

Sekarang, saya bertanya-tanya: apa teknik untuk menulis skrip bash sehingga kesalahan seperti itu tidak bisa terjadi, atau setidaknya lebih kecil kemungkinannya? Misalnya, sudahkah saya menulis

FileUtils.rm_rf("#{build}/*")

dalam naskah Ruby, penerjemah akan mengeluh karena buildtidak dideklarasikan, jadi bahasa itu melindungi saya.

Apa yang telah saya pertimbangkan dalam bash, selain mengoreksi rm(yang, sebagaimana banyak jawaban dalam pertanyaan terkait menyebutkan, tidak bermasalah):

1. rm -rf "./${build}/"*
   Itu akan membunuh pekerjaan saya saat ini (repo Git) tetapi tidak ada yang lain.
2. Varian / parameterisasi rmyang memerlukan interaksi ketika bertindak di luar direktori saat ini. (Tidak dapat menemukan.) Efek serupa.

Apakah itu, atau adakah cara lain untuk menulis skrip bash yang "kuat" dalam pengertian ini?

set -u

atau

set -o nounset

Ini akan membuat shell memperlakukan ekspansi variabel yang tidak disetel sebagai kesalahan:

$ unset build
$ set -u
$ rm -rf "$build"/*
bash: build: unbound variable

set -udan set -o nounsetmerupakan opsi shell POSIX .

Sebuah kosong nilai akan tidak memicu kesalahan sekalipun.

Untuk itu, gunakan

$ rm -rf "${build:?Error, variable is empty or unset}"/*
bash: build: Error, variable is empty or unset

Perluasan ${variable:?word}akan diperluas ke nilai variablekecuali jika itu kosong atau tidak disetel. Jika kosong atau tidak disetel, worditu akan ditampilkan pada kesalahan standar dan shell akan memperlakukan ekspansi sebagai kesalahan (perintah tidak akan dieksekusi, dan jika berjalan di shell non-interaktif, ini akan berakhir). Membiarkan :keluar akan memicu kesalahan hanya untuk nilai yang tidak disetel, seperti di bawah set -u.

${variable:?word}adalah ekspansi parameter POSIX .

Tidak satu pun dari ini akan menyebabkan shell interaktif berakhir kecuali set -e(atau set -o errexit) juga berlaku. ${variable:?word}menyebabkan skrip keluar jika variabel kosong atau tidak disetel. set -uakan menyebabkan skrip keluar jika digunakan bersama dengan set -e.

Adapun pertanyaan kedua Anda. Tidak ada cara untuk membatasi rmuntuk tidak bekerja di luar direktori saat ini.

Implementasi GNU rmmemiliki --one-file-systemopsi yang menghentikannya dari menghapus filesystem yang dipasang secara rekursif, tapi itu sedekat yang saya percaya bisa kita dapatkan tanpa membungkus rmpanggilan dalam fungsi yang benar-benar memeriksa argumen.

Sebagai catatan samping: ${build}persis sama dengan $buildkecuali jika ekspansi terjadi sebagai bagian dari string di mana karakter yang mengikuti berikut adalah karakter yang valid dalam nama variabel, seperti pada "${build}x".

Saya akan menyarankan pemeriksaan validasi normal menggunakan test/[ ]

Anda akan aman jika Anda menulis skrip Anda seperti itu:

build="build"
...
[ -n "${build}" ] || exit 1
rm -rf "${build}/"*
...

The [ -n "${build}" ]cek yang "${build}"adalah panjang string non-nol .

The ||adalah logis operator OR di bash. Ini menyebabkan perintah lain dijalankan jika yang pertama gagal.

Dengan cara ini, telah ${build}kosong / tidak terdefinisi / dll. skrip akan keluar (dengan kode pengembalian 1, yang merupakan kesalahan umum).

Ini juga akan melindungi Anda jika Anda menghapus semua penggunaan ${build}karena [ -n "" ]akan selalu salah.

Keuntungan menggunakan test/ [ ]adalah ada banyak pemeriksaan lain yang lebih bermakna yang juga dapat digunakan.

Sebagai contoh:

[ -f FILE ] True if FILE exists and is a regular file.
[ -d FILE ] True if FILE exists and is a directory.
[ -O FILE ] True if FILE exists and is owned by the effective user ID.

Dalam kasus spesifik Anda, saya telah mengerjakan 'penghapusan' di masa lalu untuk memindahkan file / direktori sebagai gantinya (dengan asumsi / tmp berada pada partisi yang sama dengan direktori Anda):

# mktemp -d is also a good, reliable choice
trashdir=/tmp/.trash-$USER/trash-`date`
mkdir -p "$trashdir"
...
mv "${build}"/* "$trashdir"
...

Di belakang layar, ini memindahkan referensi file / dir tingkat atas dari sumber ke $trashdirstruktur direktori tujuan semua pada partisi yang sama, dan tidak menghabiskan waktu berjalan struktur direktori dan membebaskan blok-blok disk per-file saat itu juga. Ini menghasilkan pembersihan yang jauh lebih cepat ketika sistem sedang digunakan aktif, sebagai ganti untuk reboot yang sedikit lebih lambat (/ tmp dibersihkan pada reboot).

Atau, entri cron untuk membersihkan /tmp/.trash-$USER secara berkala akan menjaga / tmp dari pengisian, untuk proses (mis. Build) yang menghabiskan banyak ruang disk. Jika direktori Anda berada di partisi berbeda sebagai / tmp, Anda bisa membuat direktori serupa / tmp di partisi Anda dan memiliki cron clean that gantinya.

Namun yang paling penting, jika Anda mengacaukan variabel dengan cara apa pun, Anda dapat memulihkan konten sebelum pembersihan terjadi.

Gunakan substitusi bash paramater untuk menetapkan default ketika variabel tidak diinisialisasi, misalnya:

rm -rf $ {variabel: - "/ nonexistent"}

Saya selalu mencoba memulai skrip Bash saya dengan sebuah baris #!/bin/bash -ue.

-eberarti "gagal pada e rror pertama yang tidak disentuh";

-uberarti "gagal pada penggunaan pertama u ndeclared variabel".

Temukan lebih banyak detail di artikel hebat Gunakan Mode Ketetapan Bash Tidak Resmi (Kecuali Anda Mengecewakan) . Penulis juga merekomendasikan menggunakan set -o pipefail; IFS=$'\n\t'tetapi untuk tujuan saya ini berlebihan.

Saran umum untuk memeriksa apakah variabel Anda ditetapkan, adalah alat yang berguna untuk mencegah masalah semacam ini. Tetapi dalam hal ini ada solusi yang lebih sederhana.

Kemungkinan besar tidak perlu menggumpal isi $builddirektori untuk menghapusnya tetapi tidak pada $builddirektori itu sendiri. Jadi jika Anda melewatkan extraneous *maka nilai yang tidak disetel akan berubah rm -rf /yang secara default sebagian besar implementasi rm dalam dekade terakhir akan menolak untuk melakukan (kecuali jika Anda menonaktifkan perlindungan ini dengan --no-preserve-rootopsi GNU rm ).

Melompati trailing /juga akan menghasilkan rm ''pesan kesalahan:

rm: can't remove '': No such file or directory

Ini berfungsi bahkan jika perintah rm Anda tidak menerapkan perlindungan untuk /.

Anda berpikir dalam istilah bahasa pemrograman, tetapi bash adalah bahasa scripting :-) Jadi, gunakan paradigma instruksi yang sama sekali berbeda untuk paradigma bahasa yang sama sekali berbeda .

Pada kasus ini:

rmdir ${build}

Karena rmdirakan menolak untuk menghapus direktori yang tidak kosong, Anda harus menghapus anggota terlebih dahulu. Anda tahu apa itu anggota, kan? Itu mungkin parameter untuk skrip Anda, atau berasal dari parameter, jadi:

rm -rf ${build}/${parameter}
rmdir ${build}

Sekarang, jika Anda meletakkan beberapa file atau direktori lain di sana seperti file temp atau sesuatu yang seharusnya tidak Anda miliki, rmdirakan menimbulkan kesalahan. Tangani dengan benar, lalu:

rmdir ${build} || build_dir_not_empty "${build}"

Cara berpikir ini telah membantu saya dengan baik karena ... ya, sudah ada, melakukan itu.