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 597890 - wrong argument passing to Meld with dirnames containing white spaces
wrong argument passing to Meld with dirnames containing white spaces
Status: RESOLVED FIXED
Product: gnome-commander
Classification: Other
Component: application
1.2.x
Other Linux
: Normal normal
: 1.2.9
Assigned To: epiotr
epiotr
Depends on:
Blocks:
 
 
Reported: 2009-10-09 10:19 UTC by Danny Schneider
Modified: 2009-10-17 12:52 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Bug 597890.patch (1.12 KB, patch)
2009-10-10 23:00 UTC, epiotr 		none Details | Review

Description Danny Schneider 2009-10-09 10:19:19 UTC 
using synchronize Directories fails when the pathes contain white spaces
Error Message is produced by Meld then...
Comment 1 Danny Schneider 2009-10-09 10:25:09 UTC 
just confirmed it myself again...without white spaces it works fine
Comment 2 epiotr 2009-10-09 17:27:39 UTC 
What version of gcmd? And what's your setup for differ (Settings...|Programs|Diff)?
Comment 3 Danny Schneider 2009-10-09 19:40:27 UTC 
Gnome Commander Version was 1.2.5 installed from Ubuntu Repository for Ubuntu 8.04LTS

differ is configured as meld %s

today I updated to 1.2.8 (compiled from source) - I will test if bug still exists and report later
Comment 4 Danny Schneider 2009-10-10 15:12:15 UTC 
now updated to GCMDer 1.2.8

Path1: ~/workspace = /home/danny/workspace
Path2: /media/disk/Dokumente und Einstellungen/Danny/workspace

Meld answers with Error Message "Wrong number of Arguments (found 4)"

suppose the whitespaces are handled wrong the 2 pathes will split up like this:

/home/danny/workspace
/media/disk/Dokumente
und
Einstellungen/Danny/workspace

and now we have 4 Arguments!
Comment 5 epiotr 2009-10-10 22:12:52 UTC 
Could you, please, run gcmd from terminal and attach the output when comparing files with white spaces in the path?
Comment 6 Danny Schneider 2009-10-10 22:29:05 UTC 
danny@PC01Linux:~$ gnome-commander

(gnome-commander:10916): Gtk-WARNING **: unable to find signal handler for object(GnomeCmdDir:0x845c218) with func(0x809b530) and data(0x8325d68)

(gnome-commander:10916): Gtk-WARNING **: unable to find signal handler for object(GnomeCmdDir:0x845c218) with func(0x809b1d0) and data(0x8325d68)

(gnome-commander:10916): Gtk-WARNING **: unable to find signal handler for object(GnomeCmdDir:0x845c218) with func(0x809c660) and data(0x8325d68)

(gnome-commander:10916): Gtk-WARNING **: unable to find signal handler for object(GnomeCmdDir:0x845c218) with func(0x8097e00) and data(0x8325d68)
»meld  /media/disk/Dokumente und Einstellungen/Danny/workspace /home/danny/workspace« wird ausgeführt
Comment 7 epiotr 2009-10-10 23:00:47 UTC 
Created attachment 145230 [details] [review]
Bug 597890.patch

Please verify if the attached patch is working for you.
Comment 8 Danny Schneider 2009-10-10 23:24:01 UTC 
sorry for asking - how to apply the patch?
I normaly only code for myself and do not need patching :-)
Comment 9 epiotr 2009-10-10 23:34:26 UTC 
patch -p1 < 'Bug 597890.patch'

in your src root dir
Comment 10 epiotr 2009-10-17 12:52:54 UTC 
This problem has been fixed in our software repository. The fix will go into the next software release. Thank you for your bug report.