[Bug 49415] New: .desktop files (from shortcuts) broken in wine-staging

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=49415

            Bug ID: 49415
           Summary: .desktop files (from shortcuts) broken in wine-staging
           Product: Wine-staging
           Version: unspecified
          Hardware: x86
                OS: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: -unknown
          Assignee: wine-bugs at winehq.org
          Reporter: nyanpasu64 at tuta.io
                CC: leslie_alistair at hotmail.com, z.figura12 at gmail.com
      Distribution: ---

In wine-staging, .desktop files are created incorrectly.

The correct (wine Github) .desktop file contains the following:

env WINEPREFIX="/home/nyanpasu64/.wine" wine start /Unix
/home/nyanpasu64/.wine/dosdevices/c:/users/nyanpasu64/Start\
Menu/Programs/OpenMPT.lnk

The incorrect (wine-staging) .desktop file contains the following, and fails to
start:

env WINEPREFIX="/home/nyanpasu64/.wine" wine /usr/bin/wine /Unix
/home/nyanpasu64/.wine/dosdevices/c:/users/nyanpasu64/Start\
Menu/Programs/OpenMPT.lnk

This is caused by the following patch:
https://github.com/wine-staging/wine-staging/blob/029c249e789fd8b05d8c1eeda48deb8810bbb751/patches/winemenubuilder-Desktop_Icon_Path/0001-winemenubuilder-Create-desktop-shortcuts-with-absolu.patch

I think the problem is the following line (and similar lines):

-        fprintf(file, "Exec=env WINEPREFIX=\"%s\" wine %s %s\n", prefix, path,
args);
+        fprintf(file, "Exec=env WINEPREFIX=\"%s\" wine %s %s\n", prefix,
wine_path, args);

path is not the location of the wine binary, but instead contains "start
...lnk". I think the patch should be closer to:

-        fprintf(file, "Exec=env WINEPREFIX=\"%s\" wine %s %s\n", prefix, path,
args);
+        fprintf(file, "Exec=env WINEPREFIX=\"%s\" %s %s %s\n", prefix,
wine_path, path, args);

Note that I have not tested this patch.

-- 
Do not reply to this email, post in Bugzilla using the
above URL to reply.
You are receiving this mail because:
You are watching all bug changes.