🚀 go-pugleaf

Gorka Olaizola escribi�:

Lo del getopts, ahora mismo ni idea, pero te comento algunas cosillas 
que pones mal

> images=$@
> 
> goaway=0
> for i in $images
           ^^^^^^^
Aqu� hay un fallo. Si usas $@ en el for, cuarquier argumento que tenga
espacios te lo dividir� en palabras. Usa "$@" mejor

> do
>         if [ ! -r "$i" ]
                    ^^^^
Aqu� son correctas las comillas. Tambi�n puedes usar [[ ]], que no
divide palabras y no necesita comillas, aunque es de bash, no de sh

>         then
>                 echo "The picture \"$i\" does not exists or is unreadable"
                                    ^^
No necesitas eso, $i ya va entre comillas

>                 goaway=1
>         fi
>         numimages=$(($numimages+1))
> done
> 
> En otro caso he logrado hacer funcionar los nombres con espacios metiendo
> los ficheros uno por l�nea en un fichero y haciendo esta construcci�n

Los nombres con espacios se puden hacer funcionar all� donde funcionen
los nombres sin espacios, es s�lo cuesti�n de entrecomillar ;-)

> 
> while read f
> do
>     variable="$f"
               ^^^^
               Aqu� no necesitas comillas
> done < "$fichero"
         ^^^^^^^^^^
         pa m� que aqu� tampoco, aunque tenga espacios

Resumiendo,
Entrecomillar:

- comando "argumento de varias palabras"
- "$*" y "$@"  porque los argumentos puden contener espacios.
- echo "$i" siempre que la var i se haya tomado de un read, de una
  concordancia con archivo ( for i in * por ejemplo) o de un argumento (
  for i in "$@" )
- Expansi�n de par�metro dentro de [ ], porque test separa en palabras

No entrecomillar:

- asignaciones: var1=$var2
- concordancias con archivos: for i  in * toma los nombres de archivos
con sus espacios incluidos

No es exhaustivo, pero para que te hagas una idea puede valer

Saludos