Re: a small patch to wineshelllink

[Michael Cardenas]

forgot this:

Changelog
- Added a check for file type to wineshelllink. If the file is an
executable, add "Type=application" to the kde entry.

----- Original Message -----
From: "Michael Cardenas" <michaelc(a)lindows.com>
To: <wine-patches(a)winehq.com>
Sent: Thursday, February 14, 2002 3:24 PM
Subject: a small patch to wineshelllink
> I put this patch together, but once I did the diff against the current
> cvs, I see that somebody has already fixed it. Here's a patch that is a
> little more correct.
>
> It checks when an application is creating a desktop link and adds the
> "Type=application" line if the link points to an executable.

----------------------------------------------------------------------------
----
> --- wineshelllink Mon Jan 14 10:32:12 2002
> +++ /home/michael/wineshelllink Thu Feb 14 11:42:29 2002
> @@ -70,9 +70,11 @@
>  [KDE Desktop Entry]
>  Name=$xname
>  Exec=wine "$path" -- $args
> -Type=Application
>  Comment=$descr
>  EOF
> +
> +(file "$path" | grep -q 'MS-DOS executable (EXE)') && echo
> 'Type=Application'
> +
>      [ -z "$workdir" ] || echo "Path=\"$workdir\""
>      [ -z "$xpmicon" ] || echo "Icon=$xpmicon"
>  }
> @@ -187,3 +189,4 @@
>  fi
>
> exit 0
> +