kopiowanie plików

[stiuart]

witam mam problem z pewnym skryptem
skrypt ma kopiowac plik z jednego katalogu do  innego katalogu a jesli katalog jest pusty to ma wyskoczyc komunikat mam cos takiego


 #!/bin/bash
       cp rR katalog1 katalog2
       
        if[[-e katalog1/*.*]]
        then echo "katalog jest pusty"
        else  echo " skopiowano pliki"
        fi

prosze o komentarze

[de_laurent]

Czesc.

Wybacz, ze tak na wstepie, ale straszny z ciebie niechluj.
Przegladnij swoj skrypt jeszcze raz, potem kontynuuj czytanie.

Nie wiem skad wziales tylde ~ w poleceniu cp, byc moze to literowka.
Zobacz: man cp.
W linii z komenda cp masz catalog1, a w linii z komenda if katalog1!

Nastepnie po nawiasie [ w poleceniu if _musi_ byc spacja, tak samo przed nawiasem ].

Cala linie z if zamienilbym na:
if [[ `ls katalog1` != "" ]];then
  echo ...
else
  echo ...
fi

`ls katalog1` - wykonuje komende ls katalog1 i w swoje miejsce podstawia wynik...
Poza tym najpierw sprawdzalbym czy sa jakies pliki do skopiowania, a potem ew. kopiowal...

Nastepnym razem sprawdz czy nie masz jakichs kulfonow zanim dasz cos innym do sprawdzenia.

Pozdrowienia

Eleganckie sprawdzanie czy katalog istnieje:
if [ -d /home/mariusz ]
then
        echo "jest"
else
        echo "nie ma:("
fi

M.

[stiuart]

jesli chciałbym zrobic ten skrypt do dowolnego katalogu to jak by on wygladał ??

Zmieniasz /home/mariusz na $1.
$1 jest to pierwszy paramter który przekazujesz do skryptu.
Czyli wywołanie będzie ./skrypt katalog

M.

[stiuart]

czyli wnioskując z tego co mi pisaliscie to bedzie to wyglądało nastepnująco

#!/bin/bash
$1=$/$"file"
if [ -d /$1 ]
then
        echo "jest"
else
        echo "nie ma:("
fi
f [[ `ls $1` != "" ]];then
  echo ...
else
  echo ...
fi

Według mnie wystarczy jeden "if".
W miejscu gdzie jest "jest" wpisujesz swoje kopiowanie.
Natomiast wydaje mi się, że powinieneś dodać drugi prametr skryptu($2) określający położenie docelowe.

M.

[stiuart]

#!/bin/bash
$1=$/$"file"
$2=$/$"file"
         if [ -d /$1 ]
           then
              echo "jest"

                      if [[ `ls $1` != "" ]];then
                           echo ...
                       else
                           echo ...
                      fi
                   for file in' ls $1 '; do if[ -x "$1/$file"]; then mv "$2/$file";fi;done

else
        echo "nie ma:("
fi
 znalazłem ten fragment z jednego zadanka dorzuciłem i nie wiem czy pójdzie

Nie wiem czy o tym samym mówimy ale mój skrypt:
mariusz@ciastek:~/skrypt> cat katalog
echo $1 $2
if [ -d $1 ]
then
        cp -rv $1 $2
else
        echo "nie ma:("
fi

M.

[stiuart]

if [ $# !=2 ];then
 echo usage: $0\\ \\
 exit 1
elif [ ! -d $1]; then
 echo katalog zrodlowy nie istnieje!
 exit 1
elif [ ! -d $2 ]; then
 echo Katalog docelowy nie istnieje!
 exit 1
fi
#przenoszenie plików
 mv $1 $2
done



donmarianno co myslisz o tym skrypcie ??

Tak, to zdecydowanie poprawne rozwiązanie i mnie się podoba:)
Fajnie, że zawarłeś w tym sprawdzanie ilości parametrów.

M.

[stiuart]

w zadaniu pisało zebym stworzył skrypt kopiujący zawartosc jednego katalogu do drugiego ma on pytac o nazwe katalogu i jak jest katalog pusty powinien wyskoczyc komunikat
wielkie dzięki za pomoc

Może to szczegół - używasz "mv"(przesunięcie) a nie "cp"(kopiowanie). Mały szczegół.

M.

[stiuart]

cos mie idzie mi ten skrypt
donmarianno ten twój chodzi bez zarzutów

A daj zamiast "mv" "cp -rv $1 $2"

M.