Bug 1507661 - gdbus-codegen fails
Summary: gdbus-codegen fails
Alias: None
Product: Red Hat Enterprise Linux 7
Classification: Red Hat
Component: glib2
Version: 7.5
Hardware: Unspecified
OS: Unspecified
Target Milestone: rc
: ---
Assignee: Colin Walters
QA Contact: Desktop QE
Depends On: 1508056
TreeView+ depends on / blocked
Reported: 2017-10-30 20:36 UTC by Matěj Cepl
Modified: 2018-05-17 14:39 UTC (History)
4 users (show)

Fixed In Version: glib2-2.54.1-3.el7
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Last Closed: 2018-04-10 13:06:05 UTC
Target Upstream Version:

Attachments (Terms of Use)
proposed patch (577 bytes, patch)
2017-10-30 21:40 UTC, Matěj Cepl
no flags Details | Diff

System ID Priority Status Summary Last Updated
Red Hat Product Errata RHBA-2018:0770 None None None 2018-04-10 13:07:01 UTC

Description Matěj Cepl 2017-10-30 20:36:33 UTC
Description of problem:
matej@mitmanek: ~$ gdbus-codegen --help
Traceback (most recent call last):
  File "/bin/gdbus-codegen", line 41, in <module>
    from codegen import codegen_main
ImportError: No module named codegen
matej@mitmanek: ~$

Version-Release number of selected component (if applicable):

How reproducible:

Additional info:
The problem lies in the /bin/gdbus-codegen script itself:

elif os.path.basename(filedir) == 'bin':
    # Make the prefix containing gdbus-codegen 'relocatable' at runtime by
    # adding /some/prefix/bin/../share/glib-2.0 to the python path
    path = os.path.join(filedir, '..', 'share', 'glib-2.0')

must fail to do The Right Thing™ when it is in /bin/ instead of /usr/bin/ as apparently expected.

Comment 2 Matěj Cepl 2017-10-30 21:40:04 UTC
Created attachment 1345637 [details]
proposed patch

Comment 3 Colin Walters 2017-10-31 14:25:58 UTC
Thanks for the bug report and patch!  I did a quick annotate on git upstream and


Let's backport that instead?

Comment 4 Matěj Cepl 2017-10-31 15:10:11 UTC
(In reply to Colin Walters from comment #3)
> Thanks for the bug report and patch!  I did a quick annotate on git upstream
> and saw:
> https://git.gnome.org/browse/glib/commit/
> ?id=b9f2ea423526735f7fe7371fb1339eae91a618c2
> Let's backport that instead?

Sounds a way better. That algorithm of looking for the proper dir with libraries is in my opinion too brittle, and after all when building the package we know where all those files will go.

Comment 5 Colin Walters 2017-10-31 16:54:51 UTC
We actually need https://bugzilla.gnome.org/show_bug.cgi?id=789723 too.

Comment 7 Colin Walters 2017-10-31 17:54:43 UTC
Not going to go through the hoops to get a pm_ack for this, it should be fixed in the current build, please reopen/comment/send smoke signal it still doesn't work or there are any other issues.  Thanks!

Comment 10 Mats Wichmann 2017-11-08 17:33:21 UTC
Note the same issue appeared in bug 1485853 on the Fedora side.

Comment 16 errata-xmlrpc 2018-04-10 13:06:05 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.


Note You need to log in before you can comment on or make changes to this bug.