#!/bin/bash
#make working dir
TMP="$(mktemp -d -p /tmp/ youtube2mp3.XXXXXX || exit 1)"
cd $TMP
#download video(s)
for URL in $*; do
        youtube-dl -t $URL
done
#convert
for VIDEO in *.flv; do
        OUTPUT=$VIDEO".mp3"
        ffmpeg -i $VIDEO -ab 128k -ar 44100 $OUTPUT
done
#move video to Desktop
mv *.mp3 ~/Desktop
#clean-up
rm -rf $TMP

youtube-dl -t $1

youtube-dl -t "$1&fmt=18"

ffmpeg -i $VIDEO -ab 128k -ar 44100 $OUTPUT

#!/bin/bash

basename=$(basename $0)  # ie this is the name of the script that's being called

TMP="$(mktemp -d -p /tmp/ $basename.XXXXXX || exit 1)"
cd $TMP
#I like to print out what the script is doing:
echo
echo "======= Working directory is $TMP ======="
echo

#download video(s)
# using "$@" (list of arguments) instead of $* (line of arguments) is a good habit to get into,
# as some arguments don't get passed properly using $*, eg if the arguments have spaces
# (although that doesn't apply for this script). "$@" is the 'proper' way of doing it.
for URL in "$@"; do
        echo
        echo "======= Downloading $URL======="
        echo
        # note that I always use double quotes so that spaces and special chars are passed correctly
        # (again it probably doesn't matter with this script)
        youtube-dl -t "$URL"
done

#convert
for VIDEO in *.flv; do
        # change the CRAP.flv to .mp3 for the OUTPUT file (and I use double quotes again):
        OUTPUT=`echo "$VIDEO" | sed 's/-\w\+\.flv/.mp3/'`
        echo
        echo "======= Creating $OUTPUT ======="
        echo
        # (and I use double quotes again):
        ffmpeg -i "$VIDEO" -ab 128k -ar 44100 "$OUTPUT"
done

echo
echo "======= Moving to Desktop ======="
echo
mv *.mp3 ~/Desktop

echo
echo "======= Cleaning up ======="
echo
rm -rf $TMP

echo
echo "Done."
echo

VIDEO="example.flv"
OUTPUT="${VIDEO/%flv/mp3}"

OUTPUT="${VIDEO%-*flv}.mp3"