Seriously. There doesn’t seem to be a way to do this. Every thing I ever try I just get bad substitution errors. The internet is full of people posting code that’s supposed to compare file extensions but none of it works. I’ve spent all morning trying everything I could find. I already gave up and I’m making this progeam in python instead but now I’m curious. How tf do you actually compare file extensions? If I have a folder fill of files and I want to run a command only on the png files, there seems to be no way to actually do this.

If someone posts “[[ $file == *.txt ]]” I’m going to fucking scream because THAT DOES NOT WORK. IT’S NOT VAILD BASH CODE.

  • @[email protected]
    link
    fedilink
    English
    7
    edit-2
    4 months ago

    That works in bash 3.2 for me. So does this:

    [[ $file = *.txt ]]
    

    Maybe, the error is somewhere else in your code.

  • Korthrun
    link
    fedilink
    English
    64 months ago

    File extensions are just a text based convention. Renaming a file to end in .PDF does not make the file a valid PDF.

    You’re really saying there’s no way for a posix shell to take the text to the right of the last “.” character and then do simple string comparison on the result?

    Also why can’t you just use normal globbing to feed arguments to your command? Why do you need to involve flow control?

  • @[email protected]
    link
    fedilink
    English
    4
    edit-2
    4 months ago

    Maybe a little more context of what you want to run would help here. Find would work.

    find . -name “*.png” -exec whateveryouwanttodohere {} \;

    Or you could find, and then pass the arguments to xargs:

    find . -name “*.png” -print | xargs whateveryouwanttodohere

  • @[email protected]
    link
    fedilink
    English
    34 months ago

    Let’s not forget that Linux filenames are case sensitive, so “.txt” ≠ “.TXT” ≠ “.Txt”

  • Nomecks
    link
    fedilink
    English
    3
    edit-2
    4 months ago

    for i in ``ls *.png``; do something $i; done;

    (not formatting correctly)

    Is this sort of what you mean? You can tune the loop, but essentially you build a list of the files you want to do something against, then loop through it.

    • Korthrun
      link
      fedilink
      English
      5
      edit-2
      4 months ago

      What’s the benefit of spawning a subshell and executing “ls” here instead of just passing a glob to your loop?

      $ for lol in /usr/share/*.lm;do printf "I found a file named '%s'\n" "$lol";done
      
      I found a file named '/usr/share/out-go.lm'
      I found a file named '/usr/share/ragel.lm'
      I found a file named '/usr/share/ril.lm'
      I found a file named '/usr/share/rlhc-c.lm'
      I found a file named '/usr/share/rlhc-crack.lm'
      I found a file named '/usr/share/rlhc-csharp.lm'
      I found a file named '/usr/share/rlhc-d.lm'
      I found a file named '/usr/share/rlhc-go.lm'
      I found a file named '/usr/share/rlhc-java.lm'
      I found a file named '/usr/share/rlhc-js.lm'
      I found a file named '/usr/share/rlhc-julia.lm'
      I found a file named '/usr/share/rlhc-main.lm'
      I found a file named '/usr/share/rlhc-ocaml.lm'
      I found a file named '/usr/share/rlhc-ruby.lm'
      I found a file named '/usr/share/rlhc-rust.lm'
      
      • Nomecks
        link
        fedilink
        English
        74 months ago

        The benefit is I get taught something new!

  • @[email protected]
    link
    fedilink
    English
    34 months ago

    Are you actually using Bash? You might be in a different shell. You can check with echo $SHELL

  • @[email protected]
    link
    fedilink
    English
    2
    edit-2
    4 months ago

    If someone posts “[[ $file == *.txt ]]” I’m going to fucking scream because THAT DOES NOT WORK. IT’S NOT VAILD BASH CODE.

    This is valid bash code.

    Do you understand how string substitution works? In this example, “file” is the name of a variable, and $file substitutes its value. If you have not set the value of file, then it won’t work.

    Edit: you should, as a rule of thumb, quote your variables. So "$file" instead of just $file. Quoting prevents some weird behavior with whitespace and special characters in the value. But either way, this is valid code and will work in the general case.

  • @[email protected]
    link
    fedilink
    English
    24 months ago

    How about instead of starting with a list of all files in the directory, you use find to filter just the ones ending in .txt (or .TXT, it’s case insensitive)

    find $dir -maxdepth 1 -type f -iname "*.txt"