bottlesdevs / Bottles

Run Windows software and games on Linux
https://usebottles.com
GNU General Public License v3.0
6.2k stars 258 forks source link

[BUG] Can't launch executable with xfce4-terminal (Text ended before matching quote was found) #1097

Closed 1nikolas closed 2 years ago

1nikolas commented 2 years ago

Describe the bug More bugs with the terminal integration (yay) Executables with "launch with terminal" option don't run on xfce4-terminal (see log bellow). I suspect that it has to do with the file name/path having spaces. If #1095 gets fixed I'll try for xterm and Konsole too

Installation

To Reproduce Steps to reproduce the behavior:

  1. Have xfce4-terminal
  2. Create a bottle
  3. Open an executable with "launch with terminal"
  4. Neither terminal nor the executable runs

Expected behavior The terminal and the executable runs

Screenshots image

Desktop (please complete the following information):

Additional context Log:

(18:14:53) INFO Launching an executable��� 
(18:14:53) INFO Setting Key: [Latest_Executables] to [[{'name': 'Firefox Setup 97.0.1.exe', 'file': '/home/nick/Downloads/Firefox Setup 97.0.1.exe', 'args': ''}]] for bottle: [test]��� 
(18:14:53) WARNING No GPU vendor found, keep going without setting VK_ICD_FILENAMES.. 
xfce4-terminal: Text ended before matching quote was found for ". (The text was ?sh -c "/home/nick/.local/share/bottles/runners/caffe-7.2/bin/wine64 /home/nick/Downloads/Firefox?)
mirkobrombin commented 2 years ago

Can you attach the bottle.yml file?

1nikolas commented 2 years ago

Can you attach the bottle.yml file?

I think this is an integration issue but sure I will when I get home

1nikolas commented 2 years ago

okay here it is

Arch: win64
Creation_Date: '2022-02-27 15:21:29.156351'
Custom_Path: false
DLL_Overrides: {}
DXVK: dxvk-1.9.4
Environment: Application
Environment_Variables: {}
External_Programs: {}
Installed_Dependencies: []
Latest_Executables:
-   args: ''
    file: /home/nick/Downloads/Firefox Setup 97.0.1.exe
    name: Firefox Setup 97.0.1.exe
NVAPI: dxvk-nvapi-v0.5.2
Name: test
Parameters:
    aco_compiler: false
    custom_dpi: 96
    discrete_gpu: false
    dxvk: true
    dxvk_hud: false
    dxvk_nvapi: false
    fixme_logs: false
    fsr: false
    fsr_level: 5
    fullscreen_capture: false
    gamemode: false
    gamescope: false
    gamescope_borderless: false
    gamescope_fps: 0
    gamescope_fps_no_focus: 0
    gamescope_fullscreen: true
    gamescope_game_height: 0
    gamescope_game_width: 0
    gamescope_scaling: false
    gamescope_window_height: 0
    gamescope_window_width: 0
    pulseaudio_latency: false
    sync: wine
    take_focus: false
    use_runtime: false
    virtual_desktop: false
    virtual_desktop_res: 1280x720
    vkd3d: true
Path: test
Runner: caffe-7.2
State: 0
Uninstallers: {}
Update_Date: '2022-02-27 15:21:29.156381'
VKD3D: vkd3d-proton-2.5
Versioning: false
Windows: win10
WorkingDir: ''
mirkobrombin commented 2 years ago

Can you attach the bottle.yml file?

I think this is an integration issue but sure I will when I get home

Ya, I was just checking for env vars and maybe other things which can cause these problems. So it is an implementation problem but we fixed this more then 2 times :| I'll check again

1nikolas commented 2 years ago

well this commit introduces a new issue

image

Should I open a new issue?

mirkobrombin commented 2 years ago

Ok i got it

1nikolas commented 2 years ago

Now it doesn't like the parenthesis in the program files folder

/bin/sh: -c: line 1: syntax error near unexpected token `('
/bin/sh: -c: line 1: `xfce4-terminal -e 'sh -c "/home/nick/.local/share/bottles/runners/caffe-7.2/bin/wine64 '/home/nick/.local/share/bottles/bottles/c-test/drive_c/Program Files (x86)/Program/program.exe'"''
mirkobrombin commented 2 years ago

Now it doesn't like the parenthesis in the program files folder

/bin/sh: -c: line 1: syntax error near unexpected token `('
/bin/sh: -c: line 1: `xfce4-terminal -e 'sh -c "/home/nick/.local/share/bottles/runners/caffe-7.2/bin/wine64 '/home/nick/.local/share/bottles/bottles/c-test/drive_c/Program Files (x86)/Program/program.exe'"''

And now I hate it :(

mirkobrombin commented 2 years ago

image

Seem fixed now (xfce4-terminal).