linuxmint / mint21.1-beta

BETA Bug Squash Rush
7 stars 0 forks source link

nemo: actions with quotes in the Exec line fail when a target file's path contains spaces. #8

Closed haggen88 closed 1 year ago

haggen88 commented 1 year ago

via USB image writer works correctly [tested on live usb]

https://user-images.githubusercontent.com/68134588/205680715-8a2b69de-b8ba-4a4a-b4d0-6590d5f5f01d.mp4

logs .zip

mtwebster commented 1 year ago

Works here, can you run:

NEMO_DEBUG=Actions nemo --debug

It's pretty verbose, but it should show what exactly is being executed for that action.

haggen88 commented 1 year ago

@mtwebster terminal output.zip

carlosmintfan commented 1 year ago

But there's another issue here: When you right-click and choose to make a bootable usb stick, then you have to select the iso manually again!

mtwebster commented 1 year ago

Move the iso to a folder with no spaces and try it. I can reproduce this issue when there are spaces in the path.

haggen88 commented 1 year ago

Move the iso to a folder with no spaces and try it. I can reproduce this issue when there are spaces in the path.

I confirm that

hitobashira commented 1 year ago

nemo actions. I don't know if it will be helpful, but it will also fail in the following cases.

Exec=bash -c '7zz a -mx0 -mmt12 "%F".cb7 "%F" && sleep 1'

I use about 60 nemo actions, but most of them fail. What they have in common is

cd "%F"

Exec=alacritty -e bash -c 'cd "%F" && my image_file_cutter'

If there is Most of my directories contain spaces.

clefebvre commented 1 year ago

Fixed in mintstick 1.5.4.

clefebvre commented 1 year ago

@hitobashira make sure you remove quotes around %F in your actions.

Exec=bash -c '7zz a -mx0 -mmt12 "%F".cb7 "%F" && sleep 1'

should be

Exec=bash -c '7zz a -mx0 -mmt12 %F.cb7 %F && sleep 1'

veractor commented 1 year ago

For @hitobashira's case, even if the quotes around %F were removed, there will still be problems with some paths, which when actioned on, may not work at all. I've filed this in https://github.com/linuxmint/nemo/issues/3138.