On Fri, Jul 26, 2024 at 09:44:52 +0700, Max Nikulin wrote:
> https://mywiki.wooledge.org/BashPitfalls#for_f_in_.24.28ls_.2A.mp3.29
> for f in $(ls *.mp3)
> No 1 in Bash Pitfalls
I added nicer anchors, which you can use if you prefer:
https://mywiki.wooledge.org/BashPitfalls#pf1
The auto-generated anchor also works, of course, but is far less pleasing
to read.
This particular pitfall isn't an exact match for the OP's situation,
but it's tangentially related. The OP is explicitly insisting that
the /tmp directory will always and forever have exactly one entry
containing "apache" in its name. This is a poor assumption; *anyone*
can make another file or subdirectory in /tmp, and they could put
the word apache in its name.
Therefore, it's possible that $(ls /tmp | grep apache) might return
more than one result. This will break the OP's "/tmp/$(ls ...)"
solution. The glob alternative will still work, however.
So, in summary, the glob solution:
* Is shorter.
* Is easier to read and understand.
* Is more efficient.
* Doesn't break if someone creates /tmp/apache420.
