Mrthdrb

How do I define a right arrow with bar in LaTeX?

Why does John Bercow say “unlock” after reading out the results of a vote?

quarter to five p.m

What defines a dissertation?

Is it correct to write "is not focus on"?

Tiptoe or tiphoof? Adjusting words to better fit fantasy races

What would be the benefits of having both a state and local currencies?

Is there a problem with hiding "forgot password" until it's needed?

The plural of 'stomach"

Is it okay / does it make sense for another player to join a running game of Munchkin?

Modify casing of marked letters

What's a natural way to say that someone works somewhere (for a job)?

Trouble understanding overseas colleagues

How to verify if g is a generator for p?

Go Pregnant or Go Home

Efficiently merge handle parallel feature branches in SFDX

How will losing mobility of one hand affect my career as a programmer?

voltage of sounds of mp3files

How do I rename a LINUX host without needing to reboot for the rename to take effect?

Lay out the Carpet

Why Were Madagascar and New Zealand Discovered So Late?

Why are all the doors on Ferenginar (the Ferengi home world) far shorter than the average Ferengi?

How to avoid InDesign adding pages automatically?

Can I convert a rim brake wheel to a disc brake wheel?

Find fails if filename contains brackets

Find: Reference to Current DirectoryHow do I use find when the filename contains spaces?recursive search for a pattern, then for each match print out the specific SEQUENCE: line number, file name, and no file contentsAppend and write out variable-named file from list of variablesRemove all brackets in filenameFind a directory which contains all the files in my listFind filename using list inside a fileFind words after specific symbol on linedesktop action with bash command and terminalHow to add values to an array which contains a variable in the array name in bash?

2

I'm trying to use find inside a loop to create a variable that contains a file matching the filename + desired string

Example:

file1.en.srt
file1.mkv
file1.pt.srt

This is the relevant part of the code:

shopt -s nullglob
shopt -s nocaseglob

if [ -d "$1" ]; then
 for file in "$1%//"*mkv; do
 # Get filename to match against subs and audios
 filename="$(basename "$file" .mkv)"

 # Find matching subtitle file
 engsubs="$(find . -name "$filename*en.srt*" | sed -e 's,^./,,')" 

 # Find matching audio file
 engaudio="$(find . -iname "$filename*en.ac3" -o -iname "$filename*en.eac3" -o -iname "$filename*en.dts" | sed -e 's,^./,,')" 
 done
fi

It works if files don't contain brackets, but the find commands don't find anything for files whose names contain brackets. Why this is happening? I want to create a variable like $en that would contain file1.en.srt

bash shell-script find filenames

share|improve this question

edited yesterday

wjandrea

504413

asked yesterday

FreedoFreedo

450520

• 
  
  
  

  
  5
  

  
  
  
  
  
  

  
  
  Please don't just tell us "it works" and "it doesn't work". How does it fail? What error message do you get? Which part of it fails? How do you call this script? What is $1?
  
  – terdon♦
  yesterday
  

  
  
  
  


• 
  
  
  

  
  

  
  
  
  
  
  

  
  
  No error message, it just won't find anything if the filenames contains brackets []... I call this script from command line... $1 would be the path
  
  – Freedo
  yesterday
  
  

  
  
  
  


• 
  
  
  

  
  1
  

  
  
  
  
  
  

  
  
  please try to create a minimal code example for us to reproduce without your whole script.
  
  – RoVo
  yesterday
  
  

  
  
  
  


• 
  
  
  

  
  1
  

  
  
  
  
  
  

  
  
  Related: Why is looping over find's output bad practice?
  
  – Kusalananda♦
  yesterday
  

  
  
  
  


• 
  
  
  

  
  

  
  
  
  
  
  

  
  
  Also related: Why does my shell script choke on whitespace or other special characters?
  
  – Kusalananda♦
  yesterday
  

  
  
  
  

 | 
show 1 more comment

2

I'm trying to use find inside a loop to create a variable that contains a file matching the filename + desired string

Example:

file1.en.srt
file1.mkv
file1.pt.srt

This is the relevant part of the code:

shopt -s nullglob
shopt -s nocaseglob

if [ -d "$1" ]; then
 for file in "$1%//"*mkv; do
 # Get filename to match against subs and audios
 filename="$(basename "$file" .mkv)"

 # Find matching subtitle file
 engsubs="$(find . -name "$filename*en.srt*" | sed -e 's,^./,,')" 

 # Find matching audio file
 engaudio="$(find . -iname "$filename*en.ac3" -o -iname "$filename*en.eac3" -o -iname "$filename*en.dts" | sed -e 's,^./,,')" 
 done
fi

It works if files don't contain brackets, but the find commands don't find anything for files whose names contain brackets. Why this is happening? I want to create a variable like $en that would contain file1.en.srt

bash shell-script find filenames

share|improve this question

edited yesterday

wjandrea

504413

asked yesterday

FreedoFreedo

450520

• 
  
  
  

  
  5
  

  
  
  
  
  
  

  
  
  Please don't just tell us "it works" and "it doesn't work". How does it fail? What error message do you get? Which part of it fails? How do you call this script? What is $1?
  
  – terdon♦
  yesterday
  

  
  
  
  


• 
  
  
  

  
  

  
  
  
  
  
  

  
  
  No error message, it just won't find anything if the filenames contains brackets []... I call this script from command line... $1 would be the path
  
  – Freedo
  yesterday
  
  

  
  
  
  


• 
  
  
  

  
  1
  

  
  
  
  
  
  

  
  
  please try to create a minimal code example for us to reproduce without your whole script.
  
  – RoVo
  yesterday
  
  

  
  
  
  


• 
  
  
  

  
  1
  

  
  
  
  
  
  

  
  
  Related: Why is looping over find's output bad practice?
  
  – Kusalananda♦
  yesterday
  

  
  
  
  


• 
  
  
  

  
  

  
  
  
  
  
  

  
  
  Also related: Why does my shell script choke on whitespace or other special characters?
  
  – Kusalananda♦
  yesterday
  

  
  
  
  

 | 
show 1 more comment

I'm trying to use find inside a loop to create a variable that contains a file matching the filename + desired string

Example:

file1.en.srt
file1.mkv
file1.pt.srt

This is the relevant part of the code:

shopt -s nullglob
shopt -s nocaseglob

if [ -d "$1" ]; then
 for file in "$1%//"*mkv; do
 # Get filename to match against subs and audios
 filename="$(basename "$file" .mkv)"

 # Find matching subtitle file
 engsubs="$(find . -name "$filename*en.srt*" | sed -e 's,^./,,')" 

 # Find matching audio file
 engaudio="$(find . -iname "$filename*en.ac3" -o -iname "$filename*en.eac3" -o -iname "$filename*en.dts" | sed -e 's,^./,,')" 
 done
fi

It works if files don't contain brackets, but the find commands don't find anything for files whose names contain brackets. Why this is happening? I want to create a variable like $en that would contain file1.en.srt

bash shell-script find filenames

share|improve this question

edited yesterday

wjandrea

504413

asked yesterday

FreedoFreedo

450520

file1.en.srt
file1.mkv
file1.pt.srt

shopt -s nullglob
shopt -s nocaseglob

if [ -d "$1" ]; then
 for file in "$1%//"*mkv; do
 # Get filename to match against subs and audios
 filename="$(basename "$file" .mkv)"

 # Find matching subtitle file
 engsubs="$(find . -name "$filename*en.srt*" | sed -e 's,^./,,')" 

 # Find matching audio file
 engaudio="$(find . -iname "$filename*en.ac3" -o -iname "$filename*en.eac3" -o -iname "$filename*en.dts" | sed -e 's,^./,,')" 
 done
fi

share|improve this question

edited yesterday

wjandrea

504413

asked yesterday

FreedoFreedo

450520

share|improve this question

edited yesterday

wjandrea

504413

asked yesterday

FreedoFreedo

450520

edited yesterday

wjandrea

504413

edited yesterday

wjandrea

504413

asked yesterday

FreedoFreedo

450520

asked yesterday

FreedoFreedo

450520

1 Answer
1

active

oldest

votes

8

The problem is that [ and ] are glob characters. For example, consider this file:

ba[r].mkv

When running your script on that file, $filename will be: ba[r] and therefore your find command will be:

find . -name 'ba[r]*pt-BR.srt*'

Since [r] is a single letter character class, it means r. So your command is looking for a filename starting with ba, then an r, then any character(s), and pt-BR.srt and any characters again. You need to escape the brackets:

find . -name 'ba[r]*pt-BR.srt*'

The simplest way to do that is to use printf and %q. So just change this line:

filename="$(basename "$file" .mkv)"

To this:

filename=$(printf '%q' "$(basename "$file" .mkv)")

share|improve this answer

answered yesterday

terdon♦terdon

133k32264444

• 
  
  
  

  
  

  
  
  
  
  
  

  
  
  Yeah I just noticed now that I didn't need that * at the end of srt or other file extensions lol. And it worked...thanks so much <3. Do you think this could mess with other filenames containing space, (), ç or other special characters?
  
  – Freedo
  yesterday
  

  
  
  
  


• 
  
  
  

  
  1
  

  
  
  
  
  
  

  
  
  @Freedo no, the spaces and ( will also be escaped. Try running printf '%qn' "a ho[r]i(b)le file".
  
  – terdon♦
  yesterday
  

  
  
  
  

add a comment | 

Your Answer

StackExchange.ready(function()
var channelOptions =
tags: "".split(" "),
id: "106"
;
initTagRenderer("".split(" "), "".split(" "), channelOptions);

StackExchange.using("externalEditor", function()
// Have to fire editor after snippets, if snippets enabled
if (StackExchange.settings.snippets.snippetsEnabled)
StackExchange.using("snippets", function()
createEditor();
);

else
createEditor();

);

function createEditor()
StackExchange.prepareEditor(
heartbeatType: 'answer',
autoActivateHeartbeat: false,
convertImagesToLinks: false,
noModals: true,
showLowRepImageUploadWarning: true,
reputationToPostImages: null,
bindNavPrevention: true,
postfix: "",
imageUploader:
brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
allowUrls: true
,
onDemand: true,
discardSelector: ".discard-answer"
,immediatelyShowMarkdownHelp:true
);



);

draft saved

draft discarded

Sign up or log in

StackExchange.ready(function ()
StackExchange.helpers.onClickDraftSave('#login-link');
);

Sign up using Google

Sign up using Facebook

Sign up using Email and Password

Post as a guest

Name

Email

Required, but never shown

StackExchange.ready(
function ()
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2funix.stackexchange.com%2fquestions%2f508462%2ffind-fails-if-filename-contains-brackets%23new-answer', 'question_page');

);

Post as a guest

Name

Email

Required, but never shown

8

The problem is that [ and ] are glob characters. For example, consider this file:

ba[r].mkv

When running your script on that file, $filename will be: ba[r] and therefore your find command will be:

find . -name 'ba[r]*pt-BR.srt*'

Since [r] is a single letter character class, it means r. So your command is looking for a filename starting with ba, then an r, then any character(s), and pt-BR.srt and any characters again. You need to escape the brackets:

find . -name 'ba[r]*pt-BR.srt*'

The simplest way to do that is to use printf and %q. So just change this line:

filename="$(basename "$file" .mkv)"

To this:

filename=$(printf '%q' "$(basename "$file" .mkv)")

share|improve this answer

answered yesterday

terdon♦terdon

133k32264444

• 
  
  
  

  
  

  
  
  
  
  
  

  
  
  Yeah I just noticed now that I didn't need that * at the end of srt or other file extensions lol. And it worked...thanks so much <3. Do you think this could mess with other filenames containing space, (), ç or other special characters?
  
  – Freedo
  yesterday
  

  
  
  
  


• 
  
  
  

  
  1
  

  
  
  
  
  
  

  
  
  @Freedo no, the spaces and ( will also be escaped. Try running printf '%qn' "a ho[r]i(b)le file".
  
  – terdon♦
  yesterday
  

  
  
  
  

add a comment | 

8

The problem is that [ and ] are glob characters. For example, consider this file:

ba[r].mkv

When running your script on that file, $filename will be: ba[r] and therefore your find command will be:

find . -name 'ba[r]*pt-BR.srt*'

Since [r] is a single letter character class, it means r. So your command is looking for a filename starting with ba, then an r, then any character(s), and pt-BR.srt and any characters again. You need to escape the brackets:

find . -name 'ba[r]*pt-BR.srt*'

The simplest way to do that is to use printf and %q. So just change this line:

filename="$(basename "$file" .mkv)"

To this:

filename=$(printf '%q' "$(basename "$file" .mkv)")

share|improve this answer

answered yesterday

terdon♦terdon

133k32264444

• 
  
  
  

  
  

  
  
  
  
  
  

  
  
  Yeah I just noticed now that I didn't need that * at the end of srt or other file extensions lol. And it worked...thanks so much <3. Do you think this could mess with other filenames containing space, (), ç or other special characters?
  
  – Freedo
  yesterday
  

  
  
  
  


• 
  
  
  

  
  1
  

  
  
  
  
  
  

  
  
  @Freedo no, the spaces and ( will also be escaped. Try running printf '%qn' "a ho[r]i(b)le file".
  
  – terdon♦
  yesterday
  

  
  
  
  

add a comment | 

The problem is that [ and ] are glob characters. For example, consider this file:

ba[r].mkv

When running your script on that file, $filename will be: ba[r] and therefore your find command will be:

find . -name 'ba[r]*pt-BR.srt*'

Since [r] is a single letter character class, it means r. So your command is looking for a filename starting with ba, then an r, then any character(s), and pt-BR.srt and any characters again. You need to escape the brackets:

find . -name 'ba[r]*pt-BR.srt*'

The simplest way to do that is to use printf and %q. So just change this line:

filename="$(basename "$file" .mkv)"

To this:

filename=$(printf '%q' "$(basename "$file" .mkv)")

share|improve this answer

answered yesterday

terdon♦terdon

133k32264444

ba[r].mkv

find . -name 'ba[r]*pt-BR.srt*'

find . -name 'ba[r]*pt-BR.srt*'

filename="$(basename "$file" .mkv)"

filename=$(printf '%q' "$(basename "$file" .mkv)")

share|improve this answer

answered yesterday

terdon♦terdon

133k32264444

answered yesterday

terdon♦terdon

133k32264444

answered yesterday

terdon♦terdon

133k32264444