Bug 801620

Summary: GtkBuilder loading issue
Product: Red Hat Enterprise Linux 6 Reporter: Marc-Andre Lureau <marcandre.lureau>
Component: gtk2Assignee: Cosimo Cecchi <ccecchi>
Status: CLOSED ERRATA QA Contact: Desktop QE <desktop-qa-list>
Severity: high Docs Contact:
Priority: unspecified    
Version: 6.3CC: bsanford, mclasen, tpelka
Target Milestone: rc   
Target Release: ---   
Hardware: Unspecified   
OS: All   
Whiteboard:
Fixed In Version: gtk2-2.18.9-10.el6 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-06-20 12:58:38 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Bug Depends On:    
Bug Blocks: 795437, 804307    
Attachments:
Description Flags
gtk2 patch fixing issue
none
gtk2 backported patch fixing issue
none
gtk2 patch with test none

Description Marc-Andre Lureau 2012-03-09 01:49:59 UTC
Created attachment 568792 [details]
gtk2 patch fixing issue

Description of problem:

accel_path aren't correctly registered when loading builder files.

See attached example patch against gtk2 that should run a simple UI with a customzied key-binding for view/fullscreen menu bound to shift-f8.

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

2.18.9-9

Actual results:

The customized keybinding is missing.

Expected results:

The keybinding should be working

Additional info:

The bug has been fixed in upstream and identified as commit (back ported patch attached)

commit a5b92540d039a3246db3f33842a363f3ce127c48
Author: Juan Pablo Ugarte <juanpablougarte>
Date:   Wed Jun 29 11:45:33 2011 -0300

    Make GtkBuilder do not delay construct properties that can be resolved
    (so construct is respected when possible) and skip construct only props that
    can not.

Comment 2 Marc-Andre Lureau 2012-03-09 01:54:14 UTC
Created attachment 568793 [details]
gtk2 backported patch fixing issue

Comment 3 Marc-Andre Lureau 2012-03-09 01:55:34 UTC
Created attachment 568794 [details]
gtk2 patch with test

This is just a patch for testing, not for gtk :)

Comment 5 Marc-Andre Lureau 2012-03-09 19:00:52 UTC
To verify that bug, start "virt-viewer -w foo" (virt-viewer 0.5.2)

Menu View/Fullscreen:
- doesn't have keybinding before fix
- has F11 keybinding after

Comment 6 Bill Sanford 2012-03-09 20:22:31 UTC
Also you need the libvirt-0.9.10-4 installed.

Comment 7 Cosimo Cecchi 2012-03-09 21:17:47 UTC
Built gtk2-2.18.9-10.el6 with a fix for this.

Comment 11 errata-xmlrpc 2012-06-20 12:58:38 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.

http://rhn.redhat.com/errata/RHBA-2012-0809.html