icculus / mojosetup

A standalone installer for Linux software.
https://icculus.org/mojosetup/
Other
67 stars 7 forks source link

Installation fails if pathname contains whitespace, % or " #18

Closed SDLBugzilla closed 2 years ago

SDLBugzilla commented 2 years ago

This bug report was migrated from our old Bugzilla tracker.

These attachments are available in the static archive:

Reported in version: unspecified Reported for operating system, platform: Linux, PC

Comments on the original bug report:

On 2009-06-05 20:01:50 -0400, Reto Schneider wrote:

Installation fails if pathname contains whitespace, % or "

On 2009-06-05 20:04:34 -0400, Reto Schneider wrote:

Created attachment 2074 Escapes % and " in the .desktop file

On 2009-06-05 20:07:16 -0400, Reto Schneider wrote:

Created attachment 2075 changes to the installdir before calling xdg-desktop-menu

changes to the install directory before calling xdg-desktop-menu. This avoids having whitespace in the arguments for xdg-desktop-menu.

On 2009-06-05 20:07:55 -0400, Reto Schneider wrote:

Again: I tried my best, but take care...

On 2009-07-07 18:51:29 -0400, Ryan C. Gordon wrote:

Comment on attachment 2074 Escapes % and " in the .desktop file

This is now hg changeset 689:0c5e12c9c3a9.

--ryan.

On 2009-07-07 19:44:03 -0400, Ryan C. Gordon wrote:

Comment on attachment 2075 changes to the installdir before calling xdg-desktop-menu

Ugh, the xdg-desktop-menu problem is a bug in XDG's shell script, not MojoSetup. :(

I'd really rather not chdir() to work around it...let me think about it awhile longer.

--ryan.

On 2009-07-07 20:42:03 -0400, Ryan C. Gordon wrote:

Comment on attachment 2075 changes to the installdir before calling xdg-desktop-menu

hg changeset 695:682e60bf3a0f implements a chdir() strategy...the xdg-utils scripts are basically hopeless for fixing, so we'll have to work around it.

--ryan.

On 2009-07-07 20:42:34 -0400, Ryan C. Gordon wrote:

This bug is now fixed, as of the latest in revision control.

--ryan.