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.
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
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
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
Now, the srpm is missing ( 404 ).
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.
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.
Just in case it's not clear from my previous comment, appmenu-qt will work *without* any patches to other packages :)
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 ?
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.
FWIW, there's also at least one plasmoid which uses those app menus.
I'll try to start reviewing this today.
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.
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.
oh, and if you don't mind, when making the cvsadmin request, please add me (rdieter) as comaintainer.
Damian, ping? are you still able and interested in maintaining appmenu-qt?
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.
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).
I will take over this package.
*** This bug has been marked as a duplicate of bug 882508 ***
thanks.