Bug 842124 - Review Request: appmenu-qt - This package allows Qt to export its menus over DBus.
Summary: Review Request: appmenu-qt - This package allows Qt to export its menus over ...
Keywords:
Status: CLOSED DUPLICATE of bug 882508
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
unspecified
medium
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: kde-reviews
TreeView+ depends on / blocked
 
Reported: 2012-07-22 11:21 UTC by Damian Ivanov
Modified: 2012-12-01 14:59 UTC (History)
8 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-12-01 14:56:28 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Damian Ivanov 2012-07-22 11:21:27 UTC
Spec URL: https://build.opensuse.org/package/view_file?file=appmenu-qt-Fedora_17.spec&package=appmenu-qt&project=GNOME%3AAyatana&rev=56bb87142451d9f5d21c15676f27e5e1
SRPM URL: http://download.opensuse.org/repositories/GNOME:/Ayatana/Fedora_17/src/appmenu-qt-0.2.6-6.1.src.rpm
Description: This package allows Qt to export its menus over DBus. 
Fedora Account System Username:

NOTE: This package is for unity on fedora and/or indicators usable by gnome and xfce. It requires dbusmenu which will have a review request too.

Comment 1 Damian Ivanov 2012-07-22 11:23:23 UTC
Please note one additional difference: When including to Fedora .spec file name must be renamed to appmenu-qt.spec and the moment name is %name-Fedora_17.spec
correctly named spec file can be found here:
https://raw.github.com/chenxiaolong/Unity-for-Fedora/master/appmenu-qt/appmenu-qt.spec

Comment 2 Michael S. 2012-07-22 17:09:43 UTC
Hi,

a few notes :

- BuildRequires:	libXfixes
BuildRequires:	libXfixes-devel

this seems redundant ( ie, the -devel should pull the regular library )

- No need to clean in %install
%install
rm -rf $RPM_BUILD_ROOT

that's already done by rpm since a long time

- unowned directory

I am npot sure, but it seems that the direcotry 
%{_libdir}/qt4/plugins/menubar/libappmenu
is unowned. This should be fixed :
https://fedoraproject.org/wiki/Packaging:Guidelines#File_and_Directory_Ownership

Comment 3 Andrew Gunnerson 2012-07-24 18:33:23 UTC
The reason I added both libXfixes and libXfixes-devel is to workaround some issues in the Open Build Service's dependency resolver. If appmenu-qt is accepted into the Fedora repos, any dependencies where I put "Satisfy OBS conflict" are not needed.

I have updated the spec file to check for %{?opensuse_bs}.

New spec file can be found here: https://raw.github.com/chenxiaolong/Unity-for-Fedora/master/appmenu-qt/appmenu-qt.spec

Comment 4 Michael S. 2012-07-24 21:04:13 UTC
Now, the srpm is missing ( 404 ).

Comment 5 Michael S. 2012-07-24 21:09:47 UTC
In fact, I do not understand, can you explain in more detail what are such conflict, what do you see, how it fail the build ?

I see no reason that fedora on OBS should be different from koji or mock, that's basically the same rpm, installed by similar tools.

Comment 6 Andrew Gunnerson 2012-07-25 02:21:02 UTC
Sure. Somewhere along the chain of build dependencies, qt-ubuntu needs libXfixes-devel. The conflict comes from us providing a libXfixes-ubuntu-devel package (for Unity), which provides and obsoletes libXfixes-devel. OBS uses its own dependency solver that ignores the obsoletes tag, so it fails with something like: "have choice for libXfixes-devel, required by qt-devel: libXfixes-ubuntu-devel libXfixes-devel". In OBS, we must BuildRequires the preferred package to use.

No workarounds will ever be needed because koji uses mock, which uses yum, which works just fine.

The issue is really only relevant in our OBS repo because we need patched system packages for Unity.

Hope I explained the issue a little better :)

By the way, the SRPM is here: http://download.opensuse.org/repositories/GNOME:/Ayatana/Fedora_17/src/

The SRPM will 404 pretty quickly as OBS bumps the release number with every rebuild.

Comment 7 Andrew Gunnerson 2012-07-25 02:22:22 UTC
Just in case it's not clear from my previous comment, appmenu-qt will work *without* any patches to other packages :)

Comment 8 Michael S. 2012-07-25 05:01:43 UTC
so if appmenu-qt package work without patchs, why is there the need for libXfixes-ubuntu-devel, and if there is no need for it, why do you keep it in the fedora sources ?

Comment 9 Andrew Gunnerson 2012-07-25 05:03:57 UTC
You're right. That is the current source for the GNOME:Ayatana. Damian (if he will maintain the package) should copy the spec file to somewhere else and remove that part of the spec.

Comment 10 Kevin Kofler 2012-07-31 15:28:15 UTC
FWIW, there's also at least one plasmoid which uses those app menus.

Comment 11 Rex Dieter 2012-08-06 16:10:47 UTC
I'll try to start reviewing this today.

Comment 12 Rex Dieter 2012-08-06 16:14:09 UTC
using http://download.opensuse.org/repositories/GNOME:/Ayatana/Fedora_17/src/appmenu-qt-0.2.6-14.1.src.rpm

the original link didn't work.

Comment 13 Rex Dieter 2012-08-06 16:26:34 UTC
OK, using https://raw.github.com/chenxiaolong/Unity-for-Fedora/master/appmenu-qt/appmenu-qt.spec as referenced...

1. MUST drop extraneous BuildRequires:  gcc-c++
(see http://fedoraproject.org/wiki/Packaging/Guidelines#Exceptions_2 )

naming: ok

sources: ok
md5sum *.bz2
90cce750e5412d43ce075b0a9bdb6782  appmenu-qt-0.2.6.tar.bz2

license: ok

macros: ok (but could use some polish, like using fedora's %_qt4_plugindir macro instead of hard-coding %{_libdir}/qt4/plugins)

scriptlets: n/a

otherwise, it's a relatively simple package, I see nothing else that needs work.

As far as I'm concerned, item 1 is relatively cosmetic, and it's ok with me as long as it is addressed prior to doing any official fedora builds in koji.


APPROVED.

Comment 14 Rex Dieter 2012-08-06 16:32:08 UTC
oh, and if you don't mind, when making the cvsadmin request, please add me (rdieter) as comaintainer.

Comment 15 Rex Dieter 2012-11-08 15:55:51 UTC
Damian, ping?  are you still able and interested in maintaining appmenu-qt?

Comment 16 Damian Ivanov 2012-11-13 14:45:20 UTC
Rex, appmenu-qt is an important package. All desktop environments need that package if they want Qt applications to export their menus. I did (and will continue if missed something / guidelines get updated) change the spec file to conform to Fedora policies, though 
I consider Koji & co inferior to OBS and use exclusively OBS for building packages. If anybody wants to have it in mainline Fedora just get the spec from OBS and remove the -Fedora_17 (or appropriate) suffix.

Comment 17 Rex Dieter 2012-11-13 16:58:16 UTC
Damian, submitting something for review here means that you want to take responsibility for owning and maintaining it in fedora.  Based on your statement, it's not clear to me if you're willing to do that or not.  I'd appreciate if you could clarify.

(If not, this review is good for posterity, but will have to be closed, and a new one submitted by someone who *is* willing).

Comment 18 Mario Blättermann 2012-12-01 14:41:49 UTC
I will take over this package.

Comment 19 Mario Blättermann 2012-12-01 14:56:28 UTC

*** This bug has been marked as a duplicate of bug 882508 ***

Comment 20 Rex Dieter 2012-12-01 14:59:02 UTC
thanks.


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