! Please note that this is a snapshot of our old Bugzilla server, which is read only since May 29, 2020. Please go to gitlab.xfce.org for our new server !
dbus-glib-0.72 requires the --prefix switch for dbus-binding-tool, breaking p...
Status:
CLOSED: FIXED

Comments

Description Gergan Penkov 2006-11-06 20:12:30 CET
this new dbus-binding-tool behaviour breaks plugins/thunar-tpa
from the Makefile.am
---
 thunar-tpa-bindings.h: $(srcdir)/thunar-tpa-bindings.xml Makefile
       dbus-binding-tool --mode=glib-client $(srcdir)/thunar-tpa-bindings.xml > thunar-tpa-bindings.h
 endif
---
essentially sth like --prefix=thunar_tpa_icon should be added... 
it seems that xfdesktop is also broken, but I'll open a new bug for it...
Comment 1 Gergan Penkov 2006-11-06 21:29:02 CET
hm --prefix=org_xfce_Trash seems to be the correct one (strange the previous one somehow worked)...
Comment 2 Benedikt Meurer editbugs 2006-11-10 19:09:53 CET
Created attachment 866 
Suggested fix

The prefix doesn't seem to be used for glib-client mode.
Comment 3 Gergan Penkov 2006-11-10 19:32:18 CET
(In reply to comment #2)
> The prefix doesn't seem to be used for glib-client mode.
> 
Yes, no matter what it is fed with, it seems to create the same header, I probably should go and file bug against dbus-binding-tool, as this most probably is not the desired behaviour...
Should I file a separate bug for xfdesktop? also this is the patch I use in the moment for it (well probably with incorrect prefix :) ):
Index: src/Makefile.am
===================================================================
--- src/Makefile.am     (revision 23763)
+++ src/Makefile.am     (working copy)
@@ -113,10 +113,10 @@
        xfdesktop-dbus-bindings-trash.h
 
 xfdesktop-dbus-bindings-filemanager.h: $(srcdir)/xfdesktop-dbus-bindings-filemanager.xml Makefile
-       dbus-binding-tool --mode=glib-client $(srcdir)/xfdesktop-dbus-bindings-filemanager.xml > xfdesktop-dbus-bindings-filemanager.h
+       dbus-binding-tool --prefix=org_xfce_Thrash --mode=glib-client $(srcdir)/xfdesktop-dbus-bindings-filemanager.xml > xfdesktop-dbus-bindings-filemanager.h
 
 xfdesktop-dbus-bindings-trash.h: $(srcdir)/xfdesktop-dbus-bindings-trash.xml Makefile
-       dbus-binding-tool --mode=glib-client $(srcdir)/xfdesktop-dbus-bindings-trash.xml > xfdesktop-dbus-bindings-trash.h
+       dbus-binding-tool --prefix=org_xfce_FileManager --mode=glib-client $(srcdir)/xfdesktop-dbus-bindings-trash.xml > xfdesktop-dbus-bindings-trash.h
 
 endif
 endif
Comment 4 Benedikt Meurer editbugs 2006-11-11 10:26:18 CET
Yes, file a separate one for xfdesktop.
Comment 5 Benedikt Meurer editbugs 2006-11-11 10:28:30 CET
Committed with revision 23820.

2006-11-11	Benedikt Meurer <benny@xfce.org>

	* plugins/thunar-tpa/Makefile.am: Starting with version 0.72
	  dbus-binding-tool now requires the --prefix command line option
	  even though it isn't used for anything. Bug #2528.

Bug #2528

Reported by:
Gergan Penkov
Reported on: 2006-11-06
Last modified on: 2009-07-17

People

Assignee:
Jannis Pohlmann
CC List:
0 users

Version

Version:
0.5.1svn

Attachments

Suggested fix (592 bytes, patch)
2006-11-10 19:09 CET , Benedikt Meurer
no flags

Additional information