[getopts] Keine Leerabfrage von $OPTARG

[Wraith]

Hallo allerseits,

ich brauche mal Hilfe bei Folgendem:

Code: Alles auswählen

#!/bin/bash
[...]
while getopts ":u:" var_cmdopt
do
   case $var_cmdopt in
      u)
         if [ -z $OPTARG ]
            then
               fct_wrgopt "Username missing." 5
            else
               if [ -z `cut -d ':' -f 1 /etc/passwd | grep $OPTARG` ]
                  then
                     fct_wrgopt "User <"$OPTARG"> does NOT exist." 6
                  else
                     fct_setperm $OPTARG
               fi
         fi
         ;;

      *)
         fct_wrgopt "Wrong option." 1
         ;;
   esac
done

Code: Alles auswählen

# ./script.sh -u root

funktioniert ordnungsgemaess.

Code: Alles auswählen

# ./script.sh -u dummy-user

funktioniert auch. Da der User "dummy-user" nicht existiert, wird ordnungsgemaess die Meldung "User <dummy-user> does NOT exist." ausgegeben, aber bei

Code: Alles auswählen

# ./script.sh -u

kommt leider nicht die Meldung "Username missing", sondern die "Wrong option"-Meldung.

Wo habe ich da einen Denkfehler bzw. wieso schlaegt die erste IF-Abfrage (-z $OPTARG) fehl?

PS: Ob ich bei "getopts" den ersten Doppelpunkt setze oder nicht, aendert nichts. Er ist aber notwendig, da "getopts" grundsaetzlich schweigen soll.

Besten Dank im voraus.
- Wraith

[Cae]

Shell-seitig solltest du wohl die Variablen quoten, so wie du es teilweise (unnoetigerweise) mit einigen Argumenten tust. Zu getopt(1) kann ich nix sagen, ich versuche diesen Mist zu meiden, wo es nur geht.

Gruss Cae

[Wraith]

Shell-seitig solltest du wohl die Variablen quoten, so wie du es teilweise (unnoetigerweise) mit einigen Argumenten tust. Zu getopt(1) kann ich nix sagen, ich versuche diesen Mist zu meiden, wo es nur geht.

Gruss Cae

Achtung: Ich nutze NICHT getopt(1), sondern getopts (Shell Build In Command).
Davon abgesehen, ob mit oder ohne Quoting funktioniert es nicht. Die Frage ist aber, wenn DU so einen "Mist" meidest, wie dann?

[linuxCowboy]

zum Beispiel:

Code: Alles auswählen

while getopts ":u:" var_cmdopt; do
	case $var_cmdopt in
		u)
			if [ -z "`cut -d ':' -f 1 /etc/passwd | grep $OPTARG`" ]; then
				fct_wrgopt "User <"$OPTARG"> does NOT exist." 6
			else
				fct_setperm $OPTARG
			fi
			;;
		:)
			fct_wrgopt "Username missing." 5
			;;
		?)
			fct_wrgopt "Wrong option." 1
			;;
	esac
done