Batch renaming using shell script

Question

I have a folder with files named as

input (1).txt
input (2).txt
input (3).txt
...
input (207).txt

How do I rename them to

input_1.in
input_2.in
input_3.in
...
input_207.in

I am trying this

for f in *.txt ; do mv $f `echo $f | sed -e 's/input\ (\(\d*\))\.txt/input_\1.in/'` ; done

But it gives me

mv: target `(100).txt' is not a directory
mv: target `(101).txt' is not a directory
mv: target `(102).txt' is not a directory
...

Where did I go wrong?

I have put in the quotes now, but I get this now

mv: `input (90).txt' and `input (90).txt' are the same file

It is somehow trying to rename the file to the same name. How is that happening?

NawaMan · Accepted Answer · 2010-08-22 07:21:16Z

3

That is because bash for split the element with space ' ' so you are commanding it to move 'input' to '(1)'.

The way to solve this is to tell bash to split by new line using IFS variable.

Like this:

IFS=$'\n'

Then do your command.

However, I suggest you to use find to do this instead using -exec command.

For example:

find *.txt -exec mv "{}" `echo "{}" | sed -e 's/input\ (\([0-9]*\))\.txt/input_\1.in/'` \;

NOTE: I write this from memory and I did test this so let try and adjust it.

Hope this helps.

edited Aug 22, 2010 at 7:21

answered Aug 22, 2010 at 6:58

NawaMan

25.8k11 gold badges54 silver badges77 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

Lazer Over a year ago

Still this mv: 'input (90).txt' and 'input (90).txt' are the same file

NawaMan Over a year ago

I don't think you can use '\d' (from what I rememeber), try using '[0-9]'.

Dennis Williamson Over a year ago

@Lazer: in sed the escape \d introduces a decimal character representation \d97 == "a"

Ignacio Vazquez-Abrams · Accepted Answer · 2010-08-22 06:50:39Z

3

You're forgetting to quote your arguments.

... mv "$f" "$(echo "$f" | ... )" ; done

answered Aug 22, 2010 at 6:50

Ignacio Vazquez-Abrams

804k160 gold badges1.4k silver badges1.4k bronze badges

Comments

ghostdog74 · Accepted Answer · 2010-08-22 07:26:34Z

3

no need to call external commands

#!/bin/bash
shopt -s nullglob
shopt -s extglob
for file in *.txt
do
  newfile="${file//[)]/}"
  newfile="${file// [(]/_}"
  mv "$file" "${newfile%.txt}.in"
done

answered Aug 22, 2010 at 7:26

ghostdog74

346k62 gold badges264 silver badges349 bronze badges

Comments

Brian Campbell · Accepted Answer · 2010-08-23 17:26:54Z

1

As you've already fixed, you need to quote the $f argument to mv.

As to your second problem, sed doesn't support \d. You could use [0-9] instead.

answered Aug 23, 2010 at 17:26

Brian Campbell

335k59 gold badges367 silver badges344 bronze badges

Comments

Omar Ali · Accepted Answer · 2010-08-22 09:52:25Z

0

for f in *.txt ; do mv "$f" `echo $f | sed -e 's/input\ (\(\d*\))\.txt/input_\1.in/'` ; done

answered Aug 22, 2010 at 9:52

Omar Ali

8,6274 gold badges35 silver badges58 bronze badges

Comments

Ole Tange · Accepted Answer · 2010-08-22 19:41:23Z

0

If you have GNU Parallel http://www.gnu.org/software/parallel/ installed you can do this:

seq 1 207 | parallel -q mv 'input ({}).txt' input_{}.in

Watch the intro video for GNU Parallel to learn more: http://www.youtube.com/watch?v=OpaiGYxkSuQ

answered Aug 22, 2010 at 19:41

Ole Tange

34.2k9 gold badges93 silver badges112 bronze badges

Collectives™ on Stack Overflow

Batch renaming using shell script

6 Answers 6

3 Comments

Comments

Comments

Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

6 Answers 6

3 Comments

Comments

Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related