Dotaz: BASH script je příliš ukecaný

[Käyttäjä 11133]

Ahoj, snažím se napsat jednoduchý script, který mi namountuje LUKS šifrované disky na Turris Omnia... Zatím mám následující:

#!/bin/bash

verbose=0

if [ -b /dev/sda ]; then
  echo "Disk /dev/sda detected!"
  set $disks
else
  echo "Disk /dev/sda not detected!"
  unset $disks
fi

if [ -b /dev/sdb ]; then
  echo "Disk /dev/sdb detected!"
  set $disks
else
  echo "Disk /dev/sdb not detected!"
  unset $disks
fi

if [ -z ${disks+x} ]; then
  if [ -b /dev/md0 ]; then
    echo "RAID md0 already exists"
  else
    if mdadm --assemble /dev/md0 /dev/sda /dev/sdb; then
      echo "RAID /dev/md0 assembled!"
    else
      echo "Raid /dev/md0 failed to assemble!"
    fi
  fi
else
  echo "Disks sda and sdb are not present!"
fi

Avšak, když jej spustím, tak je neskutečně ukecaný, viz níže:

root@turris01:~# mount-internal-crypto-drives bbb
Disk /dev/sda detected!
BASH=/bin/bash
BASHOPTS=cmdhist:complete_fullquote:extquote:force_fignore:hostcomplete:interactive_comments:progcomp:promptvars:sourcepath
BASH_ALIASES=()
BASH_ARGC=([0]="1")
BASH_ARGV=([0]="bbb")
BASH_CMDS=()
BASH_LINENO=([0]="0")
BASH_SOURCE=([0]="/usr/local/bin//mount-internal-crypto-drives")
BASH_VERSINFO=([0]="4" [1]="4" [2]="11" [3]="1" [4]="release" [5]="arm-openwrt-linux-gnu")
BASH_VERSION='4.4.11(1)-release'
DIRSTACK=()
EUID=0
GROUPS=()
HOME=/root
HOSTNAME=turris01.czko.perlur.cloud
HOSTTYPE=arm
IFS=$' \t\n'
LOGNAME=root
MACHTYPE=arm-openwrt-linux-gnu
MAIL=/var/mail/root
OPTERR=1
OPTIND=1
OSTYPE=linux-gnu
PATH=/usr/bin:/usr/sbin:/bin:/sbin:/root/.acme.sh/:/usr/local/bin/
PIPESTATUS=([0]="0")
PPID=14913
PS4='+ '
PWD=/root
SHELL=/bin/ash
SHELLOPTS=braceexpand:hashall:interactive-comments
SHLVL=2
SSH_AUTH_SOCK=/tmp/ssh-NqAmXg83Kd/agent.14911
SSH_CLIENT='192.168.1.12 53651 22'
SSH_CONNECTION='192.168.1.12 53651 82.209.54.74 22'
SSH_TTY=/dev/pts/0
TERM=xterm-256color
UID=0
USER=root
_='Disk /dev/sda detected!'
verbose=0
Disk /dev/sdb detected!
BASH=/bin/bash
BASHOPTS=cmdhist:complete_fullquote:extquote:force_fignore:hostcomplete:interactive_comments:progcomp:promptvars:sourcepath
BASH_ALIASES=()
BASH_ARGC=([0]="1")
BASH_ARGV=([0]="bbb")
BASH_CMDS=()
BASH_LINENO=([0]="0")
BASH_SOURCE=([0]="/usr/local/bin//mount-internal-crypto-drives")
BASH_VERSINFO=([0]="4" [1]="4" [2]="11" [3]="1" [4]="release" [5]="arm-openwrt-linux-gnu")
BASH_VERSION='4.4.11(1)-release'
DIRSTACK=()
EUID=0
GROUPS=()
HOME=/root
HOSTNAME=turris01.czko.perlur.cloud
HOSTTYPE=arm
IFS=$' \t\n'
LOGNAME=root
MACHTYPE=arm-openwrt-linux-gnu
MAIL=/var/mail/root
OPTERR=1
OPTIND=1
OSTYPE=linux-gnu
PATH=/usr/bin:/usr/sbin:/bin:/sbin:/root/.acme.sh/:/usr/local/bin/
PIPESTATUS=([0]="0")
PPID=14913
PS4='+ '
PWD=/root
SHELL=/bin/ash
SHELLOPTS=braceexpand:hashall:interactive-comments
SHLVL=2
SSH_AUTH_SOCK=/tmp/ssh-NqAmXg83Kd/agent.14911
SSH_CLIENT='192.168.1.12 53651 22'
SSH_CONNECTION='192.168.1.12 53651 82.209.54.74 22'
SSH_TTY=/dev/pts/0
TERM=xterm-256color
UID=0
USER=root
_='Disk /dev/sdb detected!'
verbose=0
RAID md0 already exists
root@turris01:~# 

Nevíte čím to může být způsobeno?

---

---

Odpovědi

[Filip Jirsák]

Je to způsobeno voláním set bez parametrů (konkrétně set $disks, kde $disks je nedefinovaná proměnná). set bez parametrů vypíše všechny proměnné prostředí.

[Käyttäjä 11133]

Ok, já myslel že právě set ji zadefinuje... :-/

[Michal Kubeček]

Nikde jste nenastavil proměnnou disks a když spustíte příkaz set bez parametrů, vypíše hodnoty všech proměnných. Ale abych pravdu řekl, nením mi moc jasné, o co se to tam s tou proměnnou vlastně snažíte (a co očekáváte, že by v ní mělo být).

[Käyttäjä 11133]

Cokoli, pokud by v ní bylo cokoli tak se nespustí mdadm; samozřejmě script ještě nebyl kompletní, protože při kontrole sdb jsem ještě nekontroloval stav té proměnné, pokud by byla nenastavená tak by se test na sdb vůbec nespustil.

[k3dAR]

pak nemuzes "set $promena" ale "set promena"

[Michal Kubeček]

V tom případě je problém asi v nepochopení, co dělají shellové příkazy set a unset. První umožňuje nastavit poziční parametry shellu (se kterými ale skript nepracuje) a druhý sice odstraní proměnnou, ale jako argument musí dostat její jméno (ne hodnotu).

[Filip Jirsák]

Takže jestli to chápu dobře, ta pasáž na začátku měla vypadat spíš nějak takhle

ALL_DISKS="/dev/sda /dev/sdb"

disks=""
for d in $ALL_DISKS; do
    if [ -b "$d" ]; then
        disks="$disks $d"
    fi
done

Výsledkem by bylo, že v proměnné disks budou ta ze zařízení v ALL_DISKS, která skript v systému najde.

[mankind_boost]

To mas z toho ze jsi veganska palice. Pichni si B12 a dalsich 100 vitaminu, ktere lze ziskat jen z masa, a nastav $disks

[Käyttäjä 11133]

Tady má někdo komplexy :) Má úroveň B12 je naprosto v pořádku, děkuji. Stejně tak železa, zinku, kalcia a vitamínu D.

[mankind_boost]

Vis jak poznas vegana? Rekne ti to

[Käyttäjä 11133]

A nechceš to své tvrzení alespoň ozdrojovat, když už plácáš hlouposti?

[k3dAR]

vis jak poznas hovado? zvoli si nick jako ty

[mankind_boost]

Tak dík no.

[Käyttäjä 11133]

Komplexy? I to se dá léčit...

[mankind_boost]

Víš co už se nedá léčit? Osteoporóza. Dobře sis podělal život

[Käyttäjä 11133]

Škoda že tady nejdou postovat emoji

[Michal Kubeček]

Když tak čtu tohle vlákno, napadá mne ještě jedna věc, která se moc nedá léčit…

[manind_boost]

ditto [pdf]

[Käyttäjä 11133]

There you go... and now go back in your corner to cry.

[mankind_boost]

Všiml sis, že jsem jiná osoba než on? Nebo je to pro některé příliš mentálně náročný úkol...

[Filip Jirsák]

Zkus tohle - je to ještě méně ukecané, než jsi původně zamýšlel

#!/bin/bash

declare -a disks=("/dev/sda" "/dev/sdb")
raid="/dev/md0"
for disk in ${disks[@]}; do
    if [ ! -b "$disk" ]; then
        echo "Disk $disk not detected!" >&2
        exit 1
    fi
done
if [ -b "$raid" ]; then
    echo "RAID $raid already exists" >&2
    exit 2
fi
mdadm --assemble "$raid" ${disks[@]}
if [ ! -b "$raid" ]; then
    echo "Raid $raid failed to assemble!" >&2
    exit 3
fi

[Käyttäjä 11133]

Díky, s poli v bashi jsem už tak dlouho nedělal, že se mi s tí nechtělo patlat. Takže díky za pomoc. Tohle funguje, jen trošku poupravím!

[Kit]

Dá se to udělat i se stringem - viz výše v jiném příspěvku.