Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Links imported from Windows desktop do not work #103

Closed
ktamp opened this issue Nov 21, 2016 · 5 comments
Closed

Links imported from Windows desktop do not work #103

ktamp opened this issue Nov 21, 2016 · 5 comments
Assignees
Labels
Bug

Comments

@ktamp
Copy link

@ktamp ktamp commented Nov 21, 2016

E.g. installing game Syberia from GOG creates a desktop link which is imported by Q4Wine.
Trying to open this link causes error -1.

The actual command issued by Q4Wine to run Syberia is:

/usr/bin/env WINEPREFIX='/home/ktamp/.wine' WINESERVER='/usr/bin/wineserver' WINELOADER='/usr/bin/wine' WINEDLLPATH='/usr/lib32/wine/' WINEARCH='win32' WINEDEBUG='-all' /bin/sh -c "cd '/home/ktamp/.wine/dosdevices/c:/GOG Games/Syberia/' && '/usr/bin/wine' 'C:\windows\command\start.exe Unix homektamp.winedosdevicesc:usersPublicDesktopSyberia.lnk' 2>&1 "

The culprit here is in the last part:
'C:\windows\command\start.exe Unix homektamp.winedosdevicesc:usersPublicDesktopSyberia.lnk'

If this is split in two and the second part is simplified (while adding missing backslashes), it works:
'C:\windows\command\start.exe' 'c:\users\Public\Desktop\Syberia.lnk'

@brezerk brezerk added the Bug label Nov 21, 2016
@brezerk brezerk self-assigned this Nov 21, 2016
@brezerk
Copy link
Owner

@brezerk brezerk commented Nov 21, 2016

hi @ktamp

there should be Syberia.desktop file located at /home/ktamp/.wine/dosdevices/c:/users/ktamp/Desktop can you send me it please?

@brezerk
Copy link
Owner

@brezerk brezerk commented Nov 21, 2016

Ok.

Good news: I was able to reproduce this with the other game downloaded from GOG.
Bad news: now I have to walk through this game before I will create a fix for this issue... >_<

@brezerk
Copy link
Owner

@brezerk brezerk commented Nov 21, 2016

This issue was caused by odd regexp parser: the '/unix /home...' should be passed as arguments to start.exe.

This commit: 82de0db should fix the issue. Note you will need to remove imported icons in the import folder manually before testing the fix.

@ktamp
Copy link
Author

@ktamp ktamp commented Nov 22, 2016

I confirm it is working now. Thank you for this quick fix!

@brezerk
Copy link
Owner

@brezerk brezerk commented Nov 22, 2016

ur'welcome. thanks for bug report.

@brezerk brezerk closed this Nov 22, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.