Bug 427954

Summary: Review Request: gtk2-aqd - Global Menu Patch for GTK
Product: [Fedora] Fedora Reporter: Feng Yu <rainwoodman>
Component: Package ReviewAssignee: Nobody's working on this, feel free to take it <nobody>
Status: CLOSED WONTFIX QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: low    
Version: rawhideCC: fedora-package-review, mclasen, mtasaka, notting, peter, splewako
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2008-01-21 18:44:40 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 427955    

Description Feng Yu 2008-01-08 05:59:01 UTC
Spec URL: http://gnome2-globalmenu.googlecode.com/files/gtk2-aqd-svn221.spec
SRPM URL: http://gnome2-globalmenu.googlecode.com/files/gtk2-aqd-0.3.221.2.12.3-3.fc8.src.rpm

Description: patched gtk2 version to allow global menu for gnome applications
Project host: http://code.google.com/p/gnome2-globalmenu/

Related package: gnome2-globalmenu-applet

Comment 1 Feng Yu 2008-01-08 06:00:19 UTC
This is my first package.


Comment 2 manuel wolfshant 2008-01-08 11:10:28 UTC
Don't you need a sponsor ?

Comment 3 Feng Yu 2008-01-08 19:24:19 UTC
Yes, I need.


Comment 4 Mamoru TASAKA 2008-01-11 16:18:29 UTC
Umm.. gtk2 maintainers, would you watch this?

Comment 5 Matthias Clasen 2008-01-11 16:59:18 UTC
I don't think we'll ship any patched gtk versions in parallel to gtk2.

Comment 6 Mamoru TASAKA 2008-01-13 16:40:18 UTC
(In reply to comment #5)
> I don't think we'll ship any patched gtk versions in parallel to gtk2.
Well, I don't think either so I cc-ed to gtk2 maintainers.

Feng, how do you think? IMO if you want to add some functions to GTK
you should ask GTK upstream directly to import your patch sets.

Comment 7 Feng Yu 2008-01-13 23:37:52 UTC
Yes, you're right shipping two gtk in one distribution tends to confuse users.

i thought the benefit is if we can install gtk2 by default, and if we leave
gtk2-aqd in package list, then when people want mac-like menubar they can choose
to install gtk2-aqd.

anyway, thank you still. 


Comment 8 Peter Gordon 2008-01-14 00:25:44 UTC
Personally, I'm a fan of the Mac-workaline global menu bar. However, I would
also dislike shipping two different GTK+ versions with that as the only change
since it would effectively double (or more) the work required to keep them
updated and in sync with one another. 

My thinking is similar to that of the Freeworld FreeType build included in
Livna/RPMfusion (http://bugzilla.livna.org/show_bug.cgi?id=1473). Could we add
an ld.so config for the patched GTK+, then have it in its own %_libdir
subdirectory or similar?

Alternatively, and mostly for the sake of my own curiosity: GTK+ maintainers,
would it possible to add this patch to the mainline gtk2 spec file, so we could
easily rebuild it "--with global_menu" or similar (though it would be disabled
by default)?  

Comment 9 Peter Gordon 2008-01-14 00:27:11 UTC
> [...] of the Mac-workaline global menu bar. [...]

That should be "workalike." Sorry for the typo.

Comment 10 Matthias Clasen 2008-01-14 02:11:50 UTC
> My thinking is similar to that of the Freeworld FreeType build included in
> Livna/RPMfusion (http://bugzilla.livna.org/show_bug.cgi?id=1473). Could we add
> an ld.so config for the patched GTK+, then have it in its own %_libdir
> subdirectory or similar?

All kinds of odd hacks are possible, and I am not looking forward to the bugs
that result from any such scenario. 

> Alternatively, and mostly for the sake of my own curiosity: GTK+ maintainers,
> would it possible to add this patch to the mainline gtk2 spec file, so we
> could
> easily rebuild it "--with global_menu" or similar (though it would be 
> disabled by default)?  

I don't see any benefit of that, honestly. People who absolutely cannot live
without this can just patch the srpm and rebuild. The difference between
"add config option to spec file, rebuild" and 
"add patch to spec file, rebuild" is really marginal.

For any further discussion of global menubar patches, please refer to the
upstream bug on this patch.

Comment 11 Mamoru TASAKA 2008-01-21 18:31:33 UTC
Well, should we once close this bug (as WONTFIX)?

Comment 12 Feng Yu 2008-01-21 18:38:01 UTC
I think so.