this post was submitted on 16 Sep 2023
38 points (95.2% liked)
Asklemmy
43133 readers
1325 users here now
A loosely moderated place to ask open-ended questions
Search asklemmy π
If your post meets the following criteria, it's welcome here!
- Open-ended question
- Not offensive: at this point, we do not have the bandwidth to moderate overtly political discussions. Assume best intent and be excellent to each other.
- Not regarding using or support for Lemmy: context, see the list of support communities and tools for finding communities below
- Not ad nauseam inducing: please make sure it is a question that would be new to most members
- An actual topic of discussion
Looking for support?
Looking for a community?
- Lemmyverse: community search
- sub.rehab: maps old subreddits to fediverse options, marks official as such
- [email protected]: a community for finding communities
~Icon~ ~by~ ~@Double_[email protected]~
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
Edit: the other commenter is right, I fucked up the usage of basename.
newfile=$(echo $file|sed βs/..*//β)
That's a bit dangerous for a few reasons:
cat
is the wrong command, because it outputs the file's content, not the file's name.my.awesome.file.txt
would become an empty string, leading to errors. (The regex is not anchored to the end of the string ($
), the.
is not escaped, so it becomes a wild card, ...)My awesome file.txt
would trip up the loop and lead to unwanted results.I'd suggest this:
for file in * ; do mv β$fileβ $(echo β$fileβ | sed -r 's/(\.tar)?\.[^.]*$//') ; done