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 702862 - [Win32] gdbus-codegen : look for deps in default install path
[Win32] gdbus-codegen : look for deps in default install path
Status: RESOLVED FIXED
Product: glib
Classification: Platform
Component: gdbus
2.37.x
Other Windows
: Normal normal
: ---
Assigned To: David Zeuthen (not reading bugmail)
gtkdev
Depends on:
Blocks:
 
 
Reported: 2013-06-22 15:56 UTC by tarnyko
Modified: 2014-01-27 16:57 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
master-gdbuscodegen-win32.patch (512 bytes, patch)
2013-06-22 15:56 UTC, tarnyko 		none Details | Review

Description tarnyko 2013-06-22 15:56:31 UTC 
Created attachment 247519 [details] [review]
master-gdbuscodegen-win32.patch

Latest master. Makefiles install "gdbus-codegen" dependencies (codegen/codegen.py, etc) in "/share/glib-2.0" ; but gdbus-codegen itself, if run on Win32, looks for them in "lib/gdbus-2.0". Not sure why, though.

Attached patch harmonizes path with UNIX and so allows gdbus-codegen to work out of the box on this OS.
Comment 1 Martin Schlemmer 2013-07-31 10:29:54 UTC 
I can confirm the issue and vouch for this fix.
Comment 2 tarnyko 2013-12-09 04:14:58 UTC 
Hi Fan,

Could you do me a favour and confirm it allows gdbus-codegen to run flawlessly after a standard installation (which would greatly improve the experience) ?
Comment 3 Fan, Chun-wei 2013-12-11 04:56:28 UTC 
Hello Tarnyko,

I think it will, just let me know when this patch gets in so that I could update the property sheets that "installs" the scripts for gdbus-codegen.

So, I think one could go for it.

With blessings, thank you!
Comment 4 tarnyko 2013-12-11 10:14:32 UTC 
Thanks ! Committed, here it is :

https://git.gnome.org/browse/glib/commit/?id=36ef409591bf3c0f368c39d083c42ed029cea5e6
Comment 5 Milan Crha 2014-01-27 16:53:34 UTC 
(In reply to comment #4)
> Thanks ! Committed, here it is :

Close the bug too?
Comment 6 tarnyko 2014-01-27 16:57:25 UTC 
Yes, right. Done.