this post was submitted on 24 Jan 2025
14 points (100.0% liked)
Linux
49393 readers
1654 users here now
From Wikipedia, the free encyclopedia
Linux is a family of open source Unix-like operating systems based on the Linux kernel, an operating system kernel first released on September 17, 1991 by Linus Torvalds. Linux is typically packaged in a Linux distribution (or distro for short).
Distributions include the Linux kernel and supporting system software and libraries, many of which are provided by the GNU Project. Many Linux distributions use the word "Linux" in their name, but the Free Software Foundation uses the name GNU/Linux to emphasize the importance of GNU software, causing some controversy.
Rules
- Posts must be relevant to operating systems running the Linux kernel. GNU/Linux or otherwise.
- No misinformation
- No NSFW content
- No hate speech, bigotry, etc
Related Communities
Community icon by Alpár-Etele Méder, licensed under CC BY 3.0
founded 5 years ago
MODERATORS
you are viewing a single comment's thread
view the rest of the comments
view the rest of the comments
Don't use
ls
if you want to get filenames, it does a bunch of stuff to them. Use a shell glob orfind
.Also, because filenames can have newlines, if you want to loop over them, it's best to use one these:
When reading newline-delimited stuff with
while read
, you want to use:The
IFS=
prevents trimming of whitespace, and-r
prevents interpretation of backslashes.Some additional thoughts to be aware of by looking closer to each line (previously I just glanced over).
This point is not directly affecting your example, but I want to make you aware of something I fall into myself. Its one of those Bash quirks. Other shells might handle it differently, only speaking about Bash here. For a regular for loop over files, its important to note that if no file exists, the variable will be set to the search string. So example
for x in *.png; do
, if no .png file is found, thenx
will be set to*.png
literally. So depending on what you do in the loop this could be catastrophic. But Bash has an option for this specifically:shopt -s nullglob
. Using this option, if no file is found, thenx
will be set to an empty string. More about Bash options: https://www.gnu.org/software/bash/manual/html_node/The-Shopt-Builtin.htmlBTW one can also do a read line by line without cat, by reading the file directly: (for some reasons Beehaw won't let me type the lower than character, so replace that, here a screenshot too):
Those find and ls commands were just to illustrate how the actual alias work, to get a sense of. I'm aware of those issues with filenames. It's not about ls or find here.
Yeah sorry then. It would be good to not use
ls
in your example though, someone who doesn't know about that might read this discussion and think that's reasonable.As for your original question, doing the
foreach
as a personal alias is fine. I wouldn't use it in any script, since if anyone else reads that, they probably already know aboutxargs
. So using yourforeach
would be more confusing to any potential reader I think.I guess you are right. I even point such things out on others, so fair enough. I will update the example, as I don't want someone to see this and take it as a good example. For the alias, I never use aliases in scripts anyway. These are always for interactive usage for me at least. In scripts i follow some other rules, such as use longer names for options (unless it is a really common one).