Linux

2354 readers

2 users here now

Shit, just linux.

Use this community for anything related to linux for now, if it gets too huge maybe there will be some sort of meme/gaming/shitpost spinoff. Currently though… go nuts

founded 1 year ago

MODERATORS

carrot@sh.itjust.works

-1

mv *.jpg is complete bullshit. (sh.itjust.works)

submitted 9 months ago by mindbleach@sh.itjust.works to c/linux@sh.itjust.works

28 comments fedilink hide all child comments

'Well it's only passing mv a list of--' yeah yeah yeah, I know, and that's why I'm calling bullshit. It should be massively harder to execute filenames. Even if 1970s decisions make that the eternal hideous default: the lack of any idiot-proof standard workaround is incomprehensible.

StackOverflow's full of competing one-liners and people pointing out how each one is considered harmful. The least-skeezy options use exec. That sentence should make anyone recoil in horror.

This is not a filename problem. This is a tool problem. If a single printable character is going to silently expand into a list of names, then for god's sake, having it put each name in quotes should be fucking trivial.

you are viewing a single comment's thread
view the rest of the comments

[–] mindbleach@sh.itjust.works 1 points 9 months ago (2 children)

I expected people have tried to move files... using the move command.

I did not expect I'd have to specify that there's a destination, in an explanation of what *.jpg does, not an explanation of what mv does.

For clarity:

GNU bash, version 5.0.17(1)-release (x86_64-pc-linux-gnu)

mv *.jpg /mnt/Example/Pictures

Where it will then die with an error like mv: invalid option -- '1' depending on the contents of the current directory.

Every search about this has led back to this StackOverflow page sooner or later. All answers address wildcard expansion.

[–] NeatNit@discuss.tchncs.de 1 points 9 months ago* (last edited 9 months ago) (1 children)

Can you run printf %s\\n *.jpg in that same directory and share the result? I'm really intrigued. If there's private information in there, I'd be satisfied with just a handful of lines showing the issue with private info replaced with ****s

Edit: also run alias mv to check whether you have an alias messing things up

[–] mindbleach@sh.itjust.works 2 points 9 months ago

bash: alias: mv: not found

printf %s\\n *.jpg is just a list of filenames on newlines. There's 75,000 of them. You'll have to take my word for it.

Fortunately, GenderNeutralBro's aside about using ./* instead of * works as expected. Could have been files beginning with _-. None begin with -, exactly.

[–] FigMcLargeHuge@sh.itjust.works 1 points 9 months ago (1 children)

When I run into situations like this, I use the commands that work to write out a script. Eg, in your case the wildcard isn't working with the mv command, so do something like this:

ls -1 *.jpg | awk '{print "mv ""$1"" /mnt/example/Pictures"}' > /tmp/movefiles.sh

Then check the movefiles.sh and make sure it has all of the commands and files properly stated, make that executable, and then run that.

[–] mindbleach@sh.itjust.works 1 points 9 months ago (1 children)

ls | grep | mv would work, except the StackOverflow discussion also highlights how parsing ls can have the same issues.

I am moving thousands of files at once. If I have to check each one, it's still wrong.

The pragmatic answer turns out to be ./* instead of *.

[–] FigMcLargeHuge@sh.itjust.works 1 points 9 months ago* (last edited 9 months ago)

I saw that answer and was just offering another option. I am sure xargs might work, but you would need to test as you need a destination passed on each line. Back to my way, I have used it for a lot more than just the move command. I think I used it to do a chmod once where I wanted to check and make sure before I committed to actually running the command(s). You could also use find and the -exec option, which I think was also mentioned here.

Edit: also, you wouldn't need to check each one, just the first few and last few to make sure the syntax is correct. Maybe do a wc -l to make sure it's got the right number of entries and then let it run.