Eine bash script Variable veraendert sich - nur wieso.... :)

[Uzer1510]

Oh danke das mit nostdin werde ich mal probieren

Allerdings die Sache ist ja der ffmpeg part der funktioniert ja und extrahiert das enthaltende Bild erfolgreich.

Also der bekommt alle Paramter richtig und macht auch genau was er soll das Bild wird rausgeschrieben.

Nur verändert das irgendwie einen Eintrag in der "file Liste" der while Schleife indem er ein Zeichen bei einem Eintrag entfernt - einfach strange.

Mein Verdacht ist irgendwas mit IFS oder so aber eigentlich sollte ja nichts was in einer sub shell passiert die Laufzeitdaten des Scripts verändern....

Ergänzung (4. März 2025)

Bin nicht sicher, ob das alles geht. D.h. extra 2-3 Dateien zum Testen nehmen. Habe hier keine Möglichkeit es zu testen.

Führe das mit bash -vx <...> aus. Sollte es Probleme geben, schicke mal die Ausgabe.

Naja ich habe jetzt 1400 Dateien umgerechnet aus 16000 Einzeldateien - ich denke schon dass das geht xD

Wie gesagt mit dem "fix" tut es ja mich interessiert nur warum ich den brauche und warum nur das bei einem Eintrag ein Zeichen fehlt wenn ein Bild vorher extrahiert wurde.

Wäre danach alles defekt - ok das wäre evtl einfacher aber so.....

Weil ich halt einfach nicht sehe wie das eine das andere beeinflussen kann.

 

[R00kie]

@R00kie Ein weiterer Nervfaktor bei diesen while/read Schleifen:

$ while read f; do echo $f; done < list
a
b
c d
$ while read f; do cat > /dev/null; echo $f; done < list
a
$

Wie sieht denn deine Datei "list" inhaltlich aus? Und was willst du mit dem Code erreichen?
Wie würdest du den Code oben mit einer for-Schleife schreiben?

Eine Sache, die ich meinen jungen Kollegen immer wieder sage und versuche in deren Köpfe zu hämmern: Variablen immer quoten! Es gibt nur gaanz wenige Ausnahmen, wo man es nicht macht, aber die Chance ist hoch, dass der Code dann nicht ideal und ein kleiner dirty Hack ist.

Ergänzung (4. März 2025)

Allerdings die Sache ist ja der ffmpeg part der funktioniert ja und extrahiert das enthaltende Bild erfolgreich.

Also der bekommt alle Paramter richtig und macht auch genau was er soll das Bild wird rausgeschrieben.

Nur verändert das irgendwie einen Eintrag in der "file Liste" der while Schleife indem er ein Zeichen bei einem Eintrag entfernt - einfach strange.

Ich kenne das Verhalten aus ssh-Kommandos in einer Schleife, bei dem ich mich auch mal dumm und dusselig gesucht habe. Einzelne Kommandos fressen sich gegenseitig Daten von stdin weg. Und hier ist es read und ffmpeg. Also sehr wahrscheinlich. Ich glaube fest daran, dass das dein Problem ist und man ffmpeg mitgeben muss, nichts von stdin zu lesen (wie man es bei ssh zum Teil auch machen muss.

 

[oicfar]

selbst wenn ich Zeile 135 auskommentiere tut alles mit allen Dateinamen
ist Zeile 135 drin dann fehlt nach der Ausführung beim nächsten $file das erste /

Hast du es mit bash -xv laufen lassen? Dann könnte man es besser analysieren, was da pssiert.

 

[foofoobar]

Wie sieht denn deine Datei "list" inhaltlich aus?

Sieht man doch.

Und was willst du mit dem Code erreichen?

Das problematische Verhalten demonstrieren.

Wie würdest du den Code oben mit einer for-Schleife schreiben?

Wahrscheinlich wäre da was mit "xargs -0" fällig, wenn es um spaces geht.

 

[Donnerkind]

Unabhängig von den Skriptproblemen; bist du hart auf MP3 angewiesen? Opus wäre deutlich moderner, effizienter, frei und bei 128 kbps praktisch transparent zum Original (bei Stereo).

 

[Uzer1510]

Wie sieht denn deine Datei "list" inhaltlich aus? Und was willst du mit dem Code erreichen?
Wie würdest du den Code oben mit einer for-Schleife schreiben?

Eine Sache, die ich meinen jungen Kollegen immer wieder sage und versuche in deren Köpfe zu hämmern: Variablen immer quoten! Es gibt nur gaanz wenige Ausnahmen, wo man es nicht macht, aber die Chance ist hoch, dass der Code dann nicht ideal und ein kleiner dirty Hack ist.

Ergänzung (4. März 2025)

Ich kenne das Verhalten aus ssh-Kommandos in einer Schleife, bei dem ich mich auch mal dumm und dusselig gesucht habe. Einzelne Kommandos fressen sich gegenseitig Daten von stdin weg. Und hier ist es read und ffmpeg. Also sehr wahrscheinlich. Ich glaube fest daran, dass das dein Problem ist und man ffmpeg mitgeben muss, nichts von stdin zu lesen (wie man es bei ssh zum Teil auch machen muss.

Nein das ist es denke ich zu 100% nicht

Die ffmpeg Aufrufe selber funktionieren IMMER ffmpeg kann die Parameter immer korrekt lesen und verarbeitet die auch immer korrekt da gab es niemals einen Fehler.

Der einzige Fehler der entsteht ist dass bei der Ausführung von ffmepg Bild Extract in der internene bash Schleife bei einem internen (Array) Datensatz ein Speicher des bash scripts an einem Zeichen verändert wird obwohl nichts darauf zugreift werder lesend noch schreibend.

Es gibt doch keine Möglichkeit wie der ffmpeg Prozess egal was der macht die internen Schleifenparamter des aufrufenden bash scriptes verändern kann ohne dass das explizit gemacht wird. Wie soll das denn gehen da ist ja die Frage die ich mir stelle.

Wieso ist NACH diesem Aufruf der sauber durchläuft - also die Dateien verarbeiten kann - in einer internen bash Arrayliste auf die ich nur lesend zugreife ein Zeichen an einem Eintrag verändert.

Ich benutze wie man im Script sieht schon vorher ffmpeg Befehle (zum Ermitteln der Bitrate) in der gleichen Schleife mit den gleichen Paramtern nur anderen Funktionen und die funktionieren alle immer.

Es ist also weder der ffmpeg Aufruf generell noch das Quoten-denn das funktioniert immer fehlerfrei

Das Problem das ich habe ist das in der INTERNEN bash (file)liste der Schleife bei einem Eintrag das erste Zeichen im nächsten Durchlauf fehlt - und zwar nur bei dem einen Eintrag.

Das einzige das ich als fix mache ist ich schaue nach ist das "/" noch immer vorhanden ganz vorne wenn nicht dann ergänze es. Und das geht beim Bild extrahieren genau bei einem einzigen Eintrag (dem näcvhsten Element in der bash internen schleifen filesliste) verloren

 

[foofoobar]

Der einzige Fehler der entsteht ist dass bei der Ausführung von ffmepg Bild Extract in der internene bash Schleife bei einem internen (Array) Datensatz ein Speicher des bash scripts an einem Zeichen verändert wird obwohl nichts darauf zugreift werder lesend noch schreibend.

Kannst du den fehlerhaften Teil auf möglichst minimalen und einfach nachstellbaren Code reduzieren?

 

[oicfar]

Wieso ist NACH diesem Aufruf der sauber durchläuft - also die Dateien verarbeiten kann - in einer internen bash Arrayliste auf die ich nur lesend zugreife ein Zeichen an einem Eintrag verändert.

Bei so was ist es hilfreich, wenn man die Debug Ausgaben aus bash -xv ... hat. Mit ich ändere was und wundere mich, wieso es nicht klappt, kommt man nicht weiter.

 

[R00kie]

Verstehe ich einfach nicht. Es wird um Hilfe gebeten und wenn Hilfe angeboten wird, dann heißt es, das kann nicht die Lösung sein.
Einfach mal schnell testen!

Jetzt habe ich extra für den Schmarn ffmpeg installieren müssen.

$ 
i=0 \
while read -r file; do \
    echo "File: $file" \
    if [[ "$file" =~ \.mp4 ]]; then \
        ffmpeg -y -i "$file" -map 0:v:0 -c copy "./image_$((++i)).jpg" &>/dev/null \
    fi \
done < <(find . -maxdepth 1)
File: .
File: ./beach.jpg
File: ./e - file.mp4
File: /f - file.mp3
File: ./f - file2.mp3
File: ./g - file.mp4
File: /u - file.mp3
File: ./v - file.mp3
$
$ i=0 \
while read -r file; do \
    echo "File: $file" \
    if [[ "$file" =~ \.mp4 ]]; then \
        ffmpeg -nostdin -y -i "$file" -map 0:v:0 -c copy "./image_$((++i)).jpg" &>/dev/null \
    fi \
done < <(find . -maxdepth 1)
File: .
File: ./beach.jpg
File: ./e - file.mp4
File: ./f - file.mp3
File: ./f - file2.mp3
File: ./g - file.mp4
File: ./u - file.mp3
File: ./v - file.mp3
$

 

[Uzer1510]

Habe ich doch in #1

ist ffmpeg Bild extraction drin ändert sich files der while schleife und zwar nur beim nöächsten EIntrag fehlt das erste / das Bild wird sauber extrahiert.
ist ffmepg Bildextraction draussen bleibt files der while Schleife immer korrekt.

Die Bildextraction selber funktionert völlig problemlos nur dnach ist der $files der while Schleife in einem Eintrag verändert und zwar nicht im aktuellen sondern beim nächsten

In Post #1

Wenn Zeile 50 ausgeführt wird ändert das die $files Variable des nächsten Durchlaufs in dem vorne das / fehlt.

Zeile 50 selber funktionert aber fehlerlos wie gesgat wnen ich dem nächsten $files nacxh 50 ein "/" voranstelle läuft alles problemlos durch - also im nöchsten Schleifendurchlauf.

Ich habe bash -vx aber ich habe nichts gesehen wann die bash die internen Variabeln für den Schleifenduchlauf ändert und das müsste ich ja sehen wann wiird $files[n+1] geändert - und warum?

bash -vx habe ich durchlaufen lassen musste das nur über den PC posten

#!/bin/bash

IFS=
+ IFS=

set -vx
+ set -vx

if [ "$#" -ne 2 ]; then
    echo "Usage: $0 <input_directory> <target_directory>"
    exit 1
fi
+ '[' 2 -ne 2 ']'

INPUT_DIR="$1"
+ INPUT_DIR=/tank/mp3/#TOCONVERT_2_128CBR/TEST
TARGET_DIR="$2"
+ TARGET_DIR=/tank/mp3/#CONVERTED_2_128CBR

INPUT_DIR="${INPUT_DIR%/}"
+ INPUT_DIR=/tank/mp3/#TOCONVERT_2_128CBR/TEST
TARGET_DIR="${TARGET_DIR%/}"
+ TARGET_DIR=/tank/mp3/#CONVERTED_2_128CBR

TEMPIMAGEFILE="myEmbeddedImage.jpg"
+ TEMPIMAGEFILE=myEmbeddedImage.jpg



check_if_running() {

  local script_fullpath
  script_fullpath="$(basename "$0")"  # Canonical path to this script

  local matching_pids

  echo "* checking if another instance is active...."

  matching_pids="$(pgrep -f "$script_fullpath")"

  local count
  count="$(echo "$matching_pids" | wc -l)"

  if [[ "$count" -gt 1 ]] || ([[ "$count" -eq 1 ]] && [[ "$matching_pids" != "$$" ]]); then
    echo "Another instance of $(basename "$script_fullpath") is already running."
    exit 1
  fi
}


echo ""
+ echo ''
echo ""
+ echo ''
echo ""
+ echo ''
echo "-----------------------------------------------------------------------------"
+ echo -----------------------------------------------------------------------------
echo $0
+ echo /custom/scripte/mp3to128convert.sh
echo ""
+ echo ''
echo "Source dir: $1"
+ echo 'Source dir: /tank/mp3/#TOCONVERT_2_128CBR/TEST'
echo "Target dir: $2"
+ echo 'Target dir: /tank/mp3/#CONVERTED_2_128CBR'
echo "-----------------------------------------------------------------------------"
+ echo -----------------------------------------------------------------------------


check_if_running
+ check_if_running
+ local script_fullpath
++ basename /custom/scripte/mp3to128convert.sh
+ script_fullpath=mp3to128convert.sh
+ local matching_pids
+ echo '* checking if another instance is active....'
++ pgrep -f mp3to128convert.sh
+ matching_pids=1800266
+ local count
++ echo 1800266
++ wc -l
+ count=1
+ [[ 1 -gt 1 ]]
+ [[ 1 -eq 1 ]]
+ [[ 1800266 != \1\8\0\0\2\6\6 ]]


if [ ! -d "$INPUT_DIR" ]; then
    echo "Error: Input directory '$INPUT_DIR' does not exist."
    echo ""
    echo ""
    exit 1
fi
+ '[' '!' -d /tank/mp3/#TOCONVERT_2_128CBR/TEST ']'

mkdir -p "$TARGET_DIR"
+ mkdir -p /tank/mp3/#CONVERTED_2_128CBR

SANITIZED_DIR=$(echo "$INPUT_DIR" | sed "s/['\"]/_/g")
++ echo /tank/mp3/#TOCONVERT_2_128CBR/TEST
++ sed 's/['\''"]/_/g'
+ SANITIZED_DIR=/tank/mp3/#TOCONVERT_2_128CBR/TEST

if [ "$INPUT_DIR" != "$SANITIZED_DIR" ]; then
    mv "$INPUT_DIR" "$SANITIZED_DIR"
    INPUT_DIR="$SANITIZED_DIR"
    echo "* renamed directory $INPUT_DIR"
fi
+ '[' /tank/mp3/#TOCONVERT_2_128CBR/TEST '!=' /tank/mp3/#TOCONVERT_2_128CBR/TEST ']'

DIR_NAME=$(basename "$INPUT_DIR")
++ basename /tank/mp3/#TOCONVERT_2_128CBR/TEST
+ DIR_NAME=TEST
OUTPUT_FILE="${TARGET_DIR}/${DIR_NAME}.mp3"
+ OUTPUT_FILE=/tank/mp3/#CONVERTED_2_128CBR/TEST.mp3
echo "* Output filename set to $OUTPUT_FILE"
+ echo '* Output filename set to /tank/mp3/#CONVERTED_2_128CBR/TEST.mp3'
rm -f "$OUTPUT_FILE"
+ rm -f /tank/mp3/#CONVERTED_2_128CBR/TEST.mp3
rm -f "${INPUT_DIR}/${TEMPIMAGEFILE}"
+ rm -f /tank/mp3/#TOCONVERT_2_128CBR/TEST/myEmbeddedImage.jpg

COVER_IMAGE=
+ COVER_IMAGE=

TEMP_LIST=$(mktemp)
++ mktemp
+ TEMP_LIST=/tmp/tmp.iDboqi6sHy

rm -f "$OUTPUT_FILE"
+ rm -f /tank/mp3/#CONVERTED_2_128CBR/TEST.mp3

ALL_128_CBR=true
+ ALL_128_CBR=true


while IFS= read -r -d '' myFile; do


    safe_file=$(echo "$myFile" | sed "s/['\"]/_/g")

    if [ "$myFile" != "$safe_file" ]; then
        mv "$myFile" "$safe_file"
    fi

    if $ALL_128_CBR; then
       BITRATE=$(ffprobe -v error -select_streams a:0 -show_entries format=bit_rate -of default=noprint_wrappers=1:nokey=1 "$safe_file")
       BITRATE=$((BITRATE / 1000)) # Convert to kbit/s

       VBR_DETECTED=$(ffprobe -v error -select_streams a:0 -show_entries format_tags=encoder -of default=noprint_wrappers=1:nokey=1 "$safe_file" | grep -i "VBR")

       if [ "$BITRATE" -ne 128 ] || [ -n "$VBR_DETECTED" ]; then
          ALL_128_CBR=false
          MYTEMPNAME="$(basename "${safe_file}")"
          echo "* file has a different bitrate ${BITRATE} kb/s (filename: ${MYTEMPNAME})"
       fi
    fi


    if [[ $safe_file != /* ]]; then
       safe_file="/$safe_file"
       echo "* had to correct the path of $safe_file   ???????"
    fi


    if [ ! -f "$safe_file" ]; then
       echo "* file error: $safe_file"
       echo ""
       echo ""
       echo "ABORT"
       echo ""
       echo ""
       exit 3
    fi

    echo "file '$safe_file'" >> "$TEMP_LIST"

    if [ -z "$COVER_IMAGE" ]; then

       VIDEO_PRESENT=$(ffprobe -v error -select_streams v:0 -show_entries stream=codec_type -of csv=p=0 "$safe_file")

       if [ "$VIDEO_PRESENT" = "video" ]; then
          if ! [ -f "${INPUT_DIR}/${TEMPIMAGEFILE}" ]; then
             ffmpeg -y -i "$safe_file" -map 0:v:0 -c copy "${INPUT_DIR}/${TEMPIMAGEFILE}"
             if [ -f "${INPUT_DIR}/${TEMPIMAGEFILE}" ]; then
                COVER_IMAGE="${INPUT_DIR}/${TEMPIMAGEFILE}"
                echo "* Cover picture found - will use this"
             fi
          fi
       fi
    fi

done < <(find "$INPUT_DIR" -maxdepth 1 -type f -name "*.mp3" -print0 | sort -z)
+ IFS=
+ read -r -d '' myFile
++ find /tank/mp3/#TOCONVERT_2_128CBR/TEST -maxdepth 1 -type f -name '*.mp3' -print0
++ sort -z
++ echo /tank/mp3/#TOCONVERT_2_128CBR/TEST/1.mp3
++ sed 's/['\''"]/_/g'
+ safe_file=/tank/mp3/#TOCONVERT_2_128CBR/TEST/1.mp3
+ '[' /tank/mp3/#TOCONVERT_2_128CBR/TEST/1.mp3 '!=' /tank/mp3/#TOCONVERT_2_128CBR/TEST/1.mp3 ']'
+ true
++ ffprobe -v error -select_streams a:0 -show_entries format=bit_rate -of default=noprint_wrappers=1:nokey=1 /tank/mp3/#TOCONVERT_2_128CBR/TEST/1.mp3
+ BITRATE=340469
+ BITRATE=340
++ ffprobe -v error -select_streams a:0 -show_entries format_tags=encoder -of default=noprint_wrappers=1:nokey=1 /tank/mp3/#TOCONVERT_2_128CBR/TEST/1.mp3
++ grep -i VBR
+ VBR_DETECTED=
+ '[' 340 -ne 128 ']'
+ ALL_128_CBR=false
++ basename /tank/mp3/#TOCONVERT_2_128CBR/TEST/1.mp3
+ MYTEMPNAME=1.mp3
+ echo '* file has a different bitrate 340 kb/s (filename: 1.mp3)'
+ [[ /tank/mp3/#TOCONVERT_2_128CBR/TEST/1.mp3 != /* ]]
+ '[' '!' -f /tank/mp3/#TOCONVERT_2_128CBR/TEST/1.mp3 ']'
+ echo 'file '\''/tank/mp3/#TOCONVERT_2_128CBR/TEST/1.mp3'\'''
+ '[' -z '' ']'
++ ffprobe -v error -select_streams v:0 -show_entries stream=codec_type -of csv=p=0 /tank/mp3/#TOCONVERT_2_128CBR/TEST/1.mp3
+ VIDEO_PRESENT=video
+ '[' video = video ']'
+ '[' -f /tank/mp3/#TOCONVERT_2_128CBR/TEST/myEmbeddedImage.jpg ']'
+ ffmpeg -y -i /tank/mp3/#TOCONVERT_2_128CBR/TEST/1.mp3 -map 0:v:0 -c copy /tank/mp3/ffmpeg version 5.1.6-0+deb12u1 Copyright (c) 2000-2024 the FFmpeg developers
  built with gcc 12 (Debian 12.2.0-14)
  configuration: --prefix=/usr --extra-version=0+deb12u1 --toolchain=hardened --libdir=/usr/lib/x86_64-linux-gnu --incdir=/usr/include/x86_64-linux-gnu --arch=amd64 --enable-gpl --disable-stripping --enable-gnutls --enable-ladspa --enable-libaom --enable-libass --enable-libbluray --enable-libbs2b --enable-libcaca --enable-libcdio --enable-libcodec2 --enable-libdav1d --enable-libflite --enable-libfontconfig --enable-libfreetype --enable-libfribidi --enable-libglslang --enable-libgme --enable-libgsm --enable-libjack --enable-libmp3lame --enable-libmysofa --enable-libopenjpeg --enable-libopenmpt --enable-libopus --enable-libpulse --enable-librabbitmq --enable-librist --enable-librubberband --enable-libshine --enable-libsnappy --enable-libsoxr --enable-libspeex --enable-libsrt --enable-libssh --enable-libsvtav1 --enable-libtheora --enable-libtwolame --enable-libvidstab --enable-libvorbis --enable-libvpx --enable-libwebp --enable-libx265 --enable-libxml2 --enable-libxvid --enable-libzimg --enable-libzmq --enable-libzvbi --enable-lv2 --enable-omx --enable-openal --enable-opencl --enable-opengl --enable-sdl2 --disable-sndio --enable-libjxl --enable-pocketsphinx --enable-librsvg --enable-libmfx --enable-libdc1394 --enable-libdrm --enable-libiec61883 --enable-chromaprint --enable-frei0r --enable-libx264 --enable-libplacebo --enable-librav1e --enable-shared
  libavutil      57. 28.100 / 57. 28.100
  libavcodec     59. 37.100 / 59. 37.100
  libavformat    59. 27.100 / 59. 27.100
  libavdevice    59.  7.100 / 59.  7.100
  libavfilter     8. 44.100 /  8. 44.100
  libswscale      6.  7.100 /  6.  7.100
  libswresample   4.  7.100 /  4.  7.100
  libpostproc    56.  6.100 / 56.  6.100
Input #0, mp3, from '/tank/mp3/#TOCONVERT_2_128CBR/TEST/1.mp3':
  Metadata:
    date            : 2013
  Duration: 00:00:43.42, start: 0.025056, bitrate: 340 kb/s
  Stream #0:0: Audio: mp3, 44100 Hz, stereo, fltp, 320 kb/s
    Metadata:
      encoder         : Lavc57.48
  Stream #0:1: Video: mjpeg (Progressive), yuvj420p(pc, bt470bg/unknown/unknown), 800x800 [SAR 1:1 DAR 1:1], 90k tbr, 90k tbn (attached pic)
    Metadata:
      comment         : Cover (front)
Output #0, image2, to '/tank/mp3/#TOCONVERT_2_128CBR/TEST/myEmbeddedImage.jpg':
  Metadata:
    date            : 2013
    encoder         : Lavf59.27.100
  Stream #0:0: Video: mjpeg (Progressive), yuvj420p(pc, bt470bg/unknown/unknown), 800x800 [SAR 1:1 DAR 1:1], q=2-31, 90k tbr, 90k tbn (attached pic)
    Metadata:
      comment         : Cover (front)
Stream mapping:
  Stream #0:1 -> #0:0 (copy)
Press [q] to stop, [?] for help
[image2 @ 0x560f869c86c0] The specified filename '/tank/mp3/#TOCONVERT_2_128CBR/TEST/myEmbeddedImage.jpg' does not contain an image sequence pattern or a pattern is invalid.
[image2 @ 0x560f869c86c0] Use a pattern such as %03d for an image sequence or use the -update option (with -frames:v 1 if needed) to write a single image.
frame=    1 fps=0.0 q=-1.0 Lsize=N/A time=00:00:00.00 bitrate=N/A speed=0.00798x
video:103kB audio:0kB subtitle:0kB other streams:0kB global headers:0kB muxing overhead: unknown
+ '[' -f /tank/mp3/#TOCONVERT_2_128CBR/TEST/myEmbeddedImage.jpg ']'
+ COVER_IMAGE=/tank/mp3/#TOCONVERT_2_128CBR/TEST/myEmbeddedImage.jpg
+ echo '* Cover picture found - will use this'
+ read -r -d '' myFile
++ echo tank/mp3/#TOCONVERT_2_128CBR/TEST/2.mp3
++ sed 's/['\''"]/_/g'
+ safe_file=tank/mp3/#TOCONVERT_2_128CBR/TEST/2.mp3
+ '[' tank/mp3/#TOCONVERT_2_128CBR/TEST/2.mp3 '!=' tank/mp3/#TOCONVERT_2_128CBR/TEST/2.mp3 ']'
+ false
+ [[ tank/mp3/#TOCONVERT_2_128CBR/TEST/2.mp3 != /* ]]
+ safe_file=/tank/mp3/#TOCONVERT_2_128CBR/TEST/2.mp3
+ echo '* had to correct the path of /tank/mp3/#TOCONVERT_2_128CBR/TEST/2.mp3'
+ '[' '!' -f /tank/mp3/#TOCONVERT_2_128CBR/TEST/2.mp3 ']'
+ echo 'file '\''/tank/mp3/#TOCONVERT_2_128CBR/TEST/2.mp3'\'''
+ '[' -z /tank/mp3/#TOCONVERT_2_128CBR/TEST/myEmbeddedImage.jpg ']'
+ IFS=
+ read -r -d '' myFile
++ echo /tank/mp3/#TOCONVERT_2_128CBR/TEST/3.mp3
++ sed 's/['\''"]/_/g'
+ safe_file=/tank/mp3/#TOCONVERT_2_128CBR/TEST/3.mp3
+ '[' /tank/mp3/#TOCONVERT_2_128CBR/TEST/3.mp3 '!=' /tank/mp3/#TOCONVERT_2_128CBR/TEST/3.mp3 ']'
+ false
+ [[ /tank/mp3/#TOCONVERT_2_128CBR/TEST/3.mp3 != /* ]]
+ '[' '!' -f /tank/mp3/#TOCONVERT_2_128CBR/TEST/3.mp3 ']'
+ echo 'file '\''/tank/mp3/#TOCONVERT_2_128CBR/TEST/3.mp3'\'''
+ '[' -z /tank/mp3/#TOCONVERT_2_128CBR/TEST/myEmbeddedImage.jpg ']'
+ IFS=
+ read -r -d '' myFile

if [ ! -s "$TEMP_LIST" ]; then
    echo "Error: No MP3 files found in '$INPUT_DIR'."
    rm -f "$TEMP_LIST"
    echo ""
    echo ""
    exit 1
fi
+ '[' '!' -s /tmp/tmp.iDboqi6sHy ']'

if [ -z "$COVER_IMAGE" ]; then
   COVER_IMAGE=$(find "$INPUT_DIR" -maxdepth 1 -type f -iname "*.jpg" | head -n 1)
   if [ ! -z "$COVER_IMAGE" ]; then
      MYTEMPSTRING="$(echo "$COVER_IMAGE" | sed 's#.*/\([^/]\+/[^/]\+\)$#\1#')"
      echo "* Using cover image found in directory: $MYTEMPSTRING"
   fi
fi
+ '[' -z /tank/mp3/#TOCONVERT_2_128CBR/TEST/myEmbeddedImage.jpg ']'


echo "start ffmpeg....."
+ echo 'start ffmpeg.....'
echo ""
+ echo ''
echo ""
+ echo ''


if $ALL_128_CBR; then
    echo "✅ All files are 128 kbit/s CBR. Merging without re-encoding..."
    if [ -n  "$COVER_IMAGE" ]; then
        ffmpeg -hide_banner -f concat -safe 0 -i "$TEMP_LIST" -i "$COVER_IMAGE" -map 0:a -map 1 -c copy -disposition:v attached_pic \
              -id3v2_version 3 -metadata:s:v title="Album cover" -metadata:s:v comment="Cover (front)" -y "$OUTPUT_FILE"
    else
        ffmpeg -hide_banner -f concat -safe 0 -i "$TEMP_LIST" -c copy -y "$OUTPUT_FILE"
    fi
else
    echo "⚠️ Some files are not 128 kbit/s CBR. Converting to 128 kbit/s..."

    if [ -n "$COVER_IMAGE" ]; then

       ffmpeg -hide_banner -f concat -safe 0 -i "$TEMP_LIST" \
         -i "$COVER_IMAGE" \
         -map 0:a -map 1 \
         -c:a libmp3lame -b:a 128k\
         -c:v copy \
         -disposition:v attached_pic \
         -metadata:s:v title="Album cover" \
         -metadata:s:v comment="Cover (front)" \
         -id3v2_version 3 \
         -write_id3v1 1 \
         -threads 0 \
         -y "$OUTPUT_FILE"


         #ffmpeg -i "$OUTPUT_FILE" -i "$COVER_IMAGE" -map 0:a -map 1 -c:a copy -disposition:v attached_pic \
         # -id3v2_version 3 -metadata:s:v title="Album cover" -metadata:s:v comment="Cover (front)" -y "$OUTPUT_FILE"

        #  ffmpeg -f concat -safe 0 -i "$TEMP_LIST" -i "$COVER_IMAGE" -map 0:a -map 1:v \
        #       -c:a libmp3lame -b:a 128k -c:v copy -metadata:s:v title="Album cover" \
        #       -metadata:s:v comment="Cover (front)" -id3v2_version 3 -write_id3v1 1 -y "$OUTPUT_FILE"
    else
        ffmpeg -hide_banner -f concat -safe 0 -i "$TEMP_LIST" -acodec libmp3lame -b:a 128k -id3v2_version 3 -write_id3v1 1 -y -threads 0 "$OUTPUT_FILE"
    fi
fi
+ false
+ echo '⚠️ Some files are not 128 kbit/s CBR. Converting to 128 kbit/s...'
+ '[' -n /tank/mp3/#TOCONVERT_2_128CBR/TEST/myEmbeddedImage.jpg ']'
+ ffmpeg -hide_banner -f concat -safe 0 -i /tmp/tmp.iDboqi6sHy -i /tank/mp3/#TOCONVERT_2_128CBR/TEST/myEmbeddedImage.jpg -map 0:a -map 1 -c:a libmp3lame -b:a 128k -c:v copy -disposition:v attached_pic -metadata:s:v 'title=Album cover' -metadata:s:v 'comment=Cover (front)' -id3v2_version 3 -write_id3v1 1 -threads 0 -y /tank/mp3/#CONVERTED_2_128CBR/TEST.mp3
Input #0, concat, from '/tmp/tmp.iDboqi6sHy':
  Duration: N/A, start: -0.025056, bitrate: 320 kb/s
  Stream #0:0: Audio: mp3, 44100 Hz, stereo, fltp, 320 kb/s
    Metadata:
      encoder         : Lavc57.48
  Stream #0:1: Video: mjpeg (Progressive), yuvj420p(pc, bt470bg/unknown/unknown), 800x800 [SAR 1:1 DAR 1:1], 90k tbr, 90k tbn
    Metadata:
      comment         : Cover (front)
Input #1, image2, from '/tank/mp3/#TOCONVERT_2_128CBR/TEST/myEmbeddedImage.jpg':
  Duration: 00:00:00.04, start: 0.000000, bitrate: 21073 kb/s
  Stream #1:0: Video: mjpeg (Progressive), yuvj420p(pc, bt470bg/unknown/unknown), 800x800 [SAR 1:1 DAR 1:1], 25 fps, 25 tbr, 25 tbn
Stream mapping:
  Stream #0:0 -> #0:0 (mp3 (mp3float) -> mp3 (libmp3lame))
  Stream #1:0 -> #0:1 (copy)
Press [q] to stop, [?] for help
Output #0, mp3, to '/tank/mp3/#CONVERTED_2_128CBR/TEST.mp3':
  Metadata:
    TSSE            : Lavf59.27.100
  Stream #0:0: Audio: mp3, 44100 Hz, stereo, fltp, 128 kb/s
    Metadata:
      encoder         : Lavc59.37.100 libmp3lame
  Stream #0:1: Video: mjpeg (Progressive), yuvj420p(pc, bt470bg/unknown/unknown), 800x800 [SAR 1:1 DAR 1:1], q=2-31, 25 fps, 25 tbr, 25 tbn (attached pic)
    Metadata:
      title           : Album cover
      comment         : Cover (front)
frame=    1 fps=0.3 q=-1.0 Lsize=    2149kB time=00:02:10.94 bitrate= 134.4kbits/s speed=45.8x
video:103kB audio:2045kB subtitle:0kB other streams:0kB global headers:0kB muxing overhead: 0.022593%

if [ -f "${INPUT_DIR}/${TEMPIMAGEFILE}" ]; then
   rm -f "${INPUT_DIR}/${TEMPIMAGEFILE}"
fi
+ '[' -f /tank/mp3/#TOCONVERT_2_128CBR/TEST/myEmbeddedImage.jpg ']'
+ rm -f /tank/mp3/#TOCONVERT_2_128CBR/TEST/myEmbeddedImage.jpg

if [ $? -eq 0 ]; then
    # Remove temporary file
    rm -f "$TEMP_LIST"

    NEW_NAME="$(dirname "$INPUT_DIR")/[CONVERTED] $DIR_NAME"
#    mv "$INPUT_DIR" "$NEW_NAME"

    echo "✅ Processing complete. Output file: $OUTPUT_FILE"
    echo "✅ Renamed directory to: $NEW_NAME"
else
    echo "❌ Error: Processing failed!"
fi
+ '[' 0 -eq 0 ']'
+ rm -f /tmp/tmp.iDboqi6sHy
++ dirname /tank/mp3/#TOCONVERT_2_128CBR/TEST
+ NEW_NAME='/tank/mp3/#TOCONVERT_2_128CBR/[CONVERTED] TEST'
+ echo '✅ Processing complete. Output file: /tank/mp3/#CONVERTED_2_128CBR/TEST.mp3'
+ echo '✅ Renamed directory to: /tank/mp3/#TOCONVERT_2_128CBR/[CONVERTED] TEST'

 

[R00kie]

Sieht man doch.

Das problematische Verhalten demonstrieren.

Wahrscheinlich wäre da was mit "xargs -0" fällig, wenn es um spaces geht.

Du konstruierst damit ein Problem, das im Code selbst liegt. Das demonstriert nicht unbedingt ein Problem mit der while-Schleife. Vor allem, war dein Statement zuvor, dass eine for-Schleife statt einer while-Schleife verwendet werden sollte, weil es weniger Gefummel macht.
Also müsste hier auch konkret while mit for verglichen und ein Vergleichscode aufgezeigt werden.

In dem Code wird gezielt stdin geleert, worauf read zugreift. Und das ist ein Bug im Code und kein Problem mit while. Wenn dann eher ein Problem mit "read".

Vielmehr zeigt das Beispiel sogar das Problem mit for, wenn man nicht mit IFS hantiert.
Hier mal mit dem gefixten, unnötigen cat:

$ for f in $(cat list); do cat < /dev/null > /dev/null; echo "$f"; done
a
b
c
d
e
$ while read -r f; do cat < /dev/null > /dev/null; echo "$f"; done < list
a
b
c
d e
$ cat list
a
b
c
d e
$

Ergänzung (5. März 2025)

@Uzer1510 Ich habe den Code noch mal besser lesbar formatiert und hier noch mal ganz deutlich:

 

[Uzer1510]

Oh stimmt -stdin habe ich wieder rausgeworfen gehabt als ich auf eine alten Stand zurückgegriffen habe weil ich so viel rumgedoktert habe.

das mir komplett entgangen


Dann jetzt aber danke an alle

 

[foofoobar]

Du konstruierst damit ein Problem, das im Code selbst liegt. Das demonstriert nicht unbedingt ein Problem mit der while-Schleife. Vor allem, war dein Statement zuvor, dass eine for-Schleife statt einer while-Schleife verwendet werden sollte, weil es weniger Gefummel macht.
Also müsste hier auch konkret while mit for verglichen und ein Vergleichscode aufgezeigt werden.

In dem Code wird gezielt stdin geleert, worauf read zugreift. Und das ist ein Bug im Code und kein Problem mit while. Wenn dann eher ein Problem mit "read".

Mein Beispiel ist gewollt kaputt, es soll ja auch das problematische Verhalten klarer ohne den Bloat und das erratische Verhalten von ffmpeg aufzeigen.

Ansonsten ist "xargs -0" dein Freund, damit fällt dann auch oft eine Parallelverarbeitung quasi nebenbei ab. :-)

Siehe auch hier:

• https://www.computerbase.de/forum/t...n-von-duplikatsdateien.2214317/#post-29888458
• https://www.computerbase.de/forum/t...n-von-duplikatsdateien.2214317/#post-29891246

 

[Uzer1510]

Hehe das Problem sit halt meistens tun die ja relativ problemlos. jetzt mit dem -stdin wieder drin hat auch alles super funkltioniert es musste nie ein "/" eingefügt werden.

Hab halt auf 2 Systemen parallel dran gewerkelt auf einem schnellen zum Testen und auf dem eigentlichen der das dann im Hintergrund mitmachen soll aber halt nicht sooooo fix ist (Celeron) da ändert man mal da was mal da und vergoisst dann alle Änderungen mitzunehmen.

Jedenfalls super dass sich was gelernt habe für die nächsten Scripte. in das scripten muss man halt auch erst mal reinkommen, die können ja echt viel.