LinuxQuestions.org - [SOLVED] shell script issue

- Programming (https://www.linuxquestions.org/questions/programming-9/)

- - shell script issue (https://www.linuxquestions.org/questions/programming-9/shell-script-issue-4175460103/)

shell script issue

hey guys,
i'am sorry, this is not a fedora issue but i use to used this forum ... the question is about escaping the special blank character. Please take a look at this simple script:

Code:

echo 11111111111111111111111111111111

ls ./\[VOCALOID2\ SONiKA\]\ THE\ VOCALOIDS\ \(KRAFTWERK\'s\ THE\ ROBOTS\ Vocaloid\ Cover\)-KyIpSKp8lh0.mp3



echo 22222222222222222222222222222222

`ls ./\[VOCALOID2\ SONiKA\]\ THE\ VOCALOIDS\ \(KRAFTWERK\'s\ THE\ ROBOTS\ Vocaloid\ Cover\)-KyIpSKp8lh0.mp3 > tmp`

cat tmp



echo 33333333333333333333333333333333

for fileName in `cat tmp`

do 

        echo fileName= "$fileName"

done

and its execution:

Code:

hduque#./test.sh

11111111111111111111111111111111

./[VOCALOID2 SONiKA] THE VOCALOIDS (KRAFTWERK's THE ROBOTS Vocaloid Cover)-KyIpSKp8lh0.mp3

22222222222222222222222222222222

./[VOCALOID2 SONiKA] THE VOCALOIDS (KRAFTWERK's THE ROBOTS Vocaloid Cover)-KyIpSKp8lh0.mp3

33333333333333333333333333333333

fileName= ./[VOCALOID2

fileName= SONiKA]

fileName= THE

fileName= VOCALOIDS

fileName= (KRAFTWERK's

fileName= THE

fileName= ROBOTS

fileName= Vocaloid

fileName= Cover)-KyIpSKp8lh0.mp3

I am waiting to have THE SAME output for steps 1,2 and 3. The 3rd step doesn't interpret the escaped blank character ... i'am waiting for just one line into the for command. Like this:

Code:

fileName= ./[VOCALOID2 SONiKA] THE VOCALOIDS (KRAFTWERK's THE ROBOTS Vocaloid Cover)-KyIpSKp8lh0.mp3

does somebody knows whats going on ?

thanks in advance!!!

héctor

(Unnecessary use of temporary files, unsafe tempfile creation,) "for" versus "while" loops:

Code:

doSomethingWith() { echo "$1"; }



find /some/directory -type f -iname \*.mp3 | while read ITEM; do

 doSomethingWith "${ITEM}"

done

---------- Post added 30-04-13 at 01:36 ----------

Moved: This thread is more suitable in the Programming forum and has been moved accordingly to help your thread/question get the exposure it deserves.

In this instance you could still use a for loop, but as pointed out above, not by reading it from a file as this should almost always be done with a while loop.

Using for:

Code:

for fileName in *.mp3

do

    echo "file name = $fileName"

done

There are a few common problems in scripting that are easy avoidable if you keep to simple rules, and one of them is using "for" instead of "while" loops. If you don't want to take it it from me, David The H. or other LQ members well-versed in scripting see http://mywiki.wooledge.org/DontReadLinesWithFor

Or even:

Code:

doSomethingWith() { printf '%s\n' "$1"; }

(It works even if the filename is ./-n and you use basename to strip ./)

Note "doSomethingWith()" means just that: do something. The most important thing to take away from this thread is using a "while" loop.

Quote:

Originally Posted by hDUQUE (Post 4941593)

Code:

`ls ./\[VOCALOID2\ SONiKA\]\ THE\ VOCALOIDS\ $KRAFTWERK\'s\ THE\ ROBOTS\ Vocaloid\ Cover$-KyIpSKp8lh0.mp3 > tmp`

In addition to what has already been said, what's the purpose of backticks here? They do absolutely nothing other than confuse people.

Quote:

Originally Posted by unSpawn (Post 4941646)

Code:

doSomethingWith() { echo "$1"; }



find /some/directory -type f -iname \*.mp3 | while read ITEM; do

 doSomethingWith "${ITEM}"

done

Beware it'll break if file name contains new line characters (yeah, find is kinda useless that way).

(Those who use newlines in the file-names should be killed painfully anyways:|)

I agree. While it's a nice find, I've been bitten by new lines a few times, I dread the day we feel we need to include new lines in default exclusion clauses here.

Quote:

Originally Posted by NevemTeve (Post 4941872)

(Those who use newlines in the file-names should be killed painfully anyways:|)

Maybe so, but before they are killed, they may destroy your system if they use new lines for malicious purposes. Imagine the following being run via crontab with root privilages:

Code:

find /tmp -mtime +7 | while read file; do rm -rf "$file"; done

All one needs to do is: mkdir -p "$(printf '/tmp/\n/\n')" and than wait a week to delete all the files on the system.
(In this instance, find /tmp -mtime +7 -delete solves the issue).

Quote:

Originally Posted by mina86 (Post 4941905)

crontab with root privilages

...which 0) requires root rights to edit and therefore 1) an entry vector (let's forego talking about insider jobs), followed by 2) successful elevation of privileges and 3) the humor to "hide" stuff in roots crontab in the first place...

You have a point here, mina86, there should be a kernel-parameter/mount-option against such filenames.

Quote:

Originally Posted by unSpawn (Post 4941714)

Sorry to be the stick in the mud here unSpawn, but if you look closer at my example you will see that I am not reading from a file nor reading lines so the example is
not relevant. Furthermore if we look at Mr. Wooledges site we will see the following http://mywiki.wooledge.org/BashPitfa...8ls_.2A.mp3.29 indicates this is a
good way to read from a directory.

Quote:

Originally Posted by grail (Post 4942019)

Sorry to be the stick in the mud here unSpawn, but

No need to, it's perfectly fine to point out you've got a good solution to a problem.
+1

GNU find (-print0) and bash (read -d '') can handle files with newlines:

Code:

#!/bin/bash

# list-files.bash

touch $'a\nfile' 'another file'



find . -print0 | while read -d '' file ; do

    echo "[$file]"

done

Code:

$ ./list-files.bash

[.]

[./a

file]

[./another file]

[./list-files.bash]