After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 332835 - gimp-remote does not pass URIs properly
gimp-remote does not pass URIs properly
Status: RESOLVED FIXED
Product: GIMP
Classification: Other
Component: General
2.3.x
Other All
: Normal normal
: 2.4
Assigned To: GIMP Bugs
GIMP Bugs
Depends on:
Blocks:
 
 
Reported: 2006-02-28 12:30 UTC by Jason Quigley
Modified: 2008-01-15 14:05 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Simple URI format checking (1.37 KB, patch)
2006-06-22 21:01 UTC, Eric Lamarque
committed Details | Review

Description Jason Quigley 2006-02-28 12:30:08 UTC
Please describe the problem:
GIMP is able to open remote files through the open dialog perfectly. However,
gimp-remote seems to pass URIs in the form of a straight filename when used to
open a file from Nautilus on a Windows share (or sftp).

Changing the default action to call gimp directly results in the file opening as
expected - obviously with the minor problem of opening a new instance of GIMP
each time.

Steps to reproduce:
1. Mount a remote share via Nautilus
2. Set up an action to call gnome-remote with the GnomeVFS URI
3. Open the file


Actual results:
An error dialog displays saying that "/home/jason/smb://jason@blah/blah/blah"
can't be found

Expected results:
The image would open just like calling gimp directly.

Does this happen every time?
Yes

Other information:
Comment 1 Sven Neumann 2006-02-28 13:25:54 UTC
The check in gimp-remote whether a passed filename is an URI or not probably needs to be adjusted. It is somewhat naive and only deals with a fixed set of schemes.
Comment 2 Sven Neumann 2006-02-28 13:27:47 UTC
The better fix however is to finally get rid of gimp-remote and build this functionality into the gimp application (see bug #52866).
Comment 3 Sven Neumann 2006-06-19 10:29:14 UTC
Raising severity, we should really try to do something about this.
Comment 4 Eric Lamarque 2006-06-22 21:01:42 UTC
Created attachment 67868 [details] [review]
Simple URI format checking

This patch adds a new function in gimp-remote. This tryes to validate URI as <scheme>:<something>. <scheme> is defined as in RFC 2396 (part 3.1). <something> should not be empty.
Comment 5 Sven Neumann 2006-06-23 08:15:21 UTC
I'd really have preferred if we folded gimp-remote into the core application. But let's apply this patch then and postpone that task to 2.6.
Comment 6 Sven Neumann 2006-06-23 08:34:06 UTC
Applied with some minor coding style modifications. Thanks a lot for your patch.

2006-06-23  Sven Neumann  <sven@gimp.org>

	* tools/gimp-remote.c: applied patch from Eric Lamarque that adds
	a new function to checks for a valid URI. Fixes bug #332835.