bash scripting issue ao estender a função sample

Question

Estou usando abcde (um melhor codificador de CD) com meu pi de framboesa e quero usar a nova funcionalidade de pesquisar e incorporar a arte do álbum. Para isso, o abcde.conf tem uma parte onde você pode definir sua própria função post_encode () .

Um exemplo de trabalho do link é assim:

#--------------------------------------------------------------------------#
#  A post_encode function to embed album art downloaded with abcde 2.7     #
#  and greater using the new getalbumart function. OUTPUTTYPE must be      # 
#  mp3 and and tagging is with eyeD3. To use this function copy the        # 
#  entire code block and paste it into your ~/.abcde.conf file.            #
#                                                                          # 
#                abcde: Downloading Album Art...                           #
#         http://www.andrews-corner.org/getalbumart.html                   #
#--------------------------------------------------------------------------#
post_encode ()
{
ARTISTFILE="$(mungefilename "$TRACKARTIST")"
ALBUMFILE="$(mungefilename "$DALBUM")"

if [ "$VARIOUSARTISTS" = "y" ] ; then
FINDPATH="$(eval echo "$VAOUTPUTFORMAT")"
else
FINDPATH="$(eval echo "$OUTPUTFORMAT")"
fi

FINALDIR="$(dirname "$OUTPUTDIR/$FINDPATH")"
cd "$FINALDIR"

if [ "$OUTPUTTYPE" = "mp3" ] && [ "$TAGGER" = "$EYED3" ] ; then
vecho "Preparing to embed the album art..." >&2
else
vecho "Not embedding album art, you need mp3 output and eyeD3 tagging..."  >&2
return 1
fi

if [ -e "cover.jpg" ] ; then
for i in *.mp3
do
eyeD3 --add-image cover.jpg:FRONT_COVER "$i"
done

mkdir backup
mv cover.jpg backup
vecho "Your files have had the album art embedded..." >&2
else
vecho "No album art found so no image embedded..." >&2
fi
}

Eu quero ser capaz de fazer isso por vários formatos de uma só vez (um dos pontos strongs de abcde) e não conseguir tudo em uma função.

Esta é a minha tentativa falhada de criar uma função post_encode () :

post_encode ()
{
ARTISTFILE="$(mungefilename "$TRACKARTIST")"
ALBUMFILE="$(mungefilename "$DALBUM")"

if [ "$VARIOUSARTISTS" = "y" ] ; then
FINDPATH="$(eval echo "$VAOUTPUTFORMAT")"
else
FINDPATH="$(eval echo "$OUTPUTFORMAT")"
fi

FINALDIR="$(dirname "$OUTPUTDIR/$FINDPATH")"
cd "$FINALDIR"

if [[ "$OUTPUTTYPE" == *"mp3"* ]] && [ "$TAGGER" = "$EYED3" ] ; then
vecho "Preparing to embed the album art..." >&2
else
vecho "Not embedding album art, you need mp3 output and eyeD3 tagging..." >&2
return 1
fi

if [ -e "cover.jpg" ] ; then
for i in *.mp3
do
eyeD3 --add-image cover.jpg:FRONT_COVER "$i"
done

if [[ "$OUTPUTTYPE" == *"flac"* ]] ; then
vecho "Preparing to embed the album art..." >&2
else
vecho "Not embedding album art, you need flac output.." >&2
return 1
fi

if [ -e "cover.jpg" ] ; then
for i in *.flac
do
metaflac --import-picture-from=cover.jpg "$i"
done

mkdir backup
mv cover.jpg backup
vecho "Your files have had the album art embedded..." >&2
else
vecho "No album art found so no image embedded..." >&2
fi
}

E aqui estão as reclamações de execução de abcde :

pi@EMK-RPi2B ~ $ abcde
/etc/abcde.conf: line 512: syntax error at unexpected word '}'
/etc/abcde.conf: line 512: '}'
^[Grabbing entire CD - tracks01 02 03 04 05 06 07 08 09 10 11 12 13 14 15

O que eu fiz de errado aqui? Existe apenas um par {} correspondente, a reclamação para a linha 512 é sobre o último } acima ...

    

Answer 1

Corrija isso:

if [ -e "cover.jpg" ] ; then
for i in *.mp3
do
eyeD3 --add-image cover.jpg:FRONT_COVER "$i"
done

Em:

if [ -e "cover.jpg" ] ; then
for i in *.mp3
do
eyeD3 --add-image cover.jpg:FRONT_COVER "$i"
done
fi