remove brackets from files name 1

• Sep 22, 2016

1
• #2

feherke

Programmer

Aug 5, 2002

9,541

RO

Hi

You wrote "remove the brackets", but in your example you removed the parenthesis and everything between them. This code does the later :

Code:

[b]for[/b] file [b]in[/b] [teal]*(*)*;[/teal] [b]do[/b] echo mv [i][green]"$file"[/green] [green]"${file//[/green][/i][lime]\([/lime][i][green]*[/green][/i][lime]\)[/lime][i][green]}"[/green][/i][teal];[/teal] [b]done[/b]

As it is, this only outputs the commands to rename the files. If satisfied, remove the [tt]echo[/tt] from the code and run it again to actually rename them.

Feherke.
feherke.ga

 

Upvote 0 Downvote

• Sep 26, 2016

• Thread starter
• #3

kaisalhassani

Programmer

Sep 2, 2007

8

GB

Hi

Thank you for that but I had an error see below.

#for file in *(*)*; do echo mv "$file" "${file//\(*\)}"; done
sh: "${file//\(*\)}": The specified substitution is not valid for this command.

 

Upvote 0 Downvote

• Sep 26, 2016

• #4

feherke

Programmer

Aug 5, 2002

9,541

RO

Hi

It works for me with Ksh93 and MirBSD Ksh. I assume you have Ksh88, but I not have that version. :-(

This one works in Dash, so should work in any Bourne-compatible shell :

Code:

[b]for[/b] file [b]in[/b] [teal]*(*)*;[/teal] [b]do[/b] echo mv [i][green]"$file"[/green] [green]"`echo "$file" | sed 's/(.*)//'`"[/green][/i][teal];[/teal] [b]done[/b]

But better see if you have [tt]rename[/tt] ( sometimes [tt]rename.pl[/tt] ) from the perl package ( the one from util-linux package replaces only fixed strings, but that one you probably not have anyway ) :

Code:

rename -n [green][i]'s/\(.*\)//'[/i][/green] [teal]*(*)*[/teal]

As it is, this only outputs a list of what would be renamed. If satisfied, remove the [tt]-n[/tt] from the code and run it again to actually rename them.

Feherke.
feherke.ga

 

Upvote 0 Downvote

• Sep 27, 2016

• Thread starter
• #5

kaisalhassani

Programmer

Sep 2, 2007

8

GB

Hi
Thank you again.
rename command is not available.
I had to amend the above because I have a file without the brackets too.
for file in *(*)*
do
nfile="`echo "$file" | sed 's/(.*)//'`"
if [[ $file != $nfile ]]; then
mv $file $nfile
fi
done

 

Upvote 0 Downvote

• Sep 27, 2016

• #6

feherke

Programmer

Aug 5, 2002

9,541

RO

Hi

Oops. You are right, Kais. Either that, or fix my glob :

Code:

[b]for[/b] file [b]in[/b] [teal]*[highlight]\[/highlight](*[highlight]\[/highlight])*;[/teal] [b]do[/b] echo mv [i][green]"$file"[/green] [green]"`echo "$file" | sed 's/(.*)//'`"[/green][/i][teal];[/teal] [b]done[/b]

Feherke.
feherke.ga

 

Upvote 0 Downvote