☰

Error de cd a un directorio al mover archivos a otro directorio

tag-icon • tag-icon bash
Error de cd a un directorio al mover archivos a otro directorio

He escrito el siguiente código para MOVER TODOS LOS ARCHIVOS DE SEGUIMIENTO CREADOS ANTES DE 2 MINUTOS A UN DIRECTORIO SEPARADO. Ahora, cuando estoy codificando la ruta de origen y de destino en el script, funciona bien; pero cuando los paso como variables, aparece un mensaje de error. El siguiente es mi código:

##THIS SCRIPT WILL MOVE ALL THE TRACE FILES CREATED BEFORE 2 MINUTES TO A SEPARATE DIRECTORY
cd /home/adhikarisubir/test/basic_unix  //MY TRACE_BKP.SH FILE EXISTS HERE
#TRACE_PATH=/home/adhikarisubir/test/basic_unix
#BKUP_PATH=/home/adhikarisubir/test_bkp
MINUTES=$1  #IT WILL GET THE NUMBER OF MINUTES FROM THE COMMAND LINE
TRACE_PATH=$2   #INSTEAD OF HARD-CODING TAKING PATHS AS VARIABLE
BKUP_PATH=$3    #^^
echo $MINUTES   
echo $BKUP_PATH
echo $TRACE_PATH
cd $TRACE_PATH
echo $pwd

for i in `find . -cmin +$MINUTES -name "*.trc"`  ##FINDING FILES WITH EXTENSTION .TRC WHICH ARE OLDER THAN GIVEN MINUTES
do
echo "Moving files $i, size `du -h $i`, which are older than $MINUTES minutes"
echo $i
mv $i $BKUP_PATH
done

Estoy ejecutando el código como:

adhikarisubir@ubuntu:~/test/basic_unix$ sh trace_bkp.sh 2 home/adhikarisubir/test/basic_unix home/adhikarisubir/test_bkp

La salida del error es:

2
home/adhikarisubir/test_bkp
home/adhikarisubir/test/basic_unix
trace_bkp.sh: 11: cd: can't cd to home/adhikarisubir/test/basic_unix 

Moving files ./elnc.trc, size 0 ./elnc.trc, which are older than 2 minutes
./elnc.trc
mv: cannot move ‘./elnc.trc’ to ‘home/adhikarisubir/test_bkp’: No such file or directory
Moving files ./ph.trc, size 0   ./ph.trc, which are older than 2 minutes
./ph.trc
mv: cannot move ‘./ph.trc’ to ‘home/adhikarisubir/test_bkp’: No such file or directory

Respuesta1

Una reescritura:

#!/bin/bash

minutes=$1
trace_path=$2
bkup_path=$3

if [[ ! -d "$trace_path" ]]; then
    echo "Error: trace_path '$trace_path' is not a directory" >&2
    exit 1
fi
if [[ ! -d "$bkup_path" ]]; then
    echo "Error: bkup_path '$bkup_path' is not a directory" >&2
    exit 1
fi

echo "Moving files which are older than $minutes minutes"
find "$trace_path" -cmin +"$minutes" -name "*.trc" -exec mv -v -t "$bkup_path" '{}' \+

Notas:

  • no necesitas ir a cdningún lado
    • Si desea cambiar el directorio donde está el script, haga esto:cd "$(dirname "$0")"
  • cite siempre su "$variables"a menos que sepa exactamente cuándo dejar las comillas fuera
  • no uses UPPER_CASE_VARS: un día lo harásescribir accidentalmentePATH=fooy luego me pregunto por qué tu guión está roto
  • validar la entrada del usuario
  • findpuede hacer cosas con los archivos que encuentra.
  • mvtiene opciones, consulte su página de manual

información relacionada