Spec URL: http://www.lovesunix.net/fedora/ndesk-dbus-glib.spec SRPM URL: http://www.lovesunix.net/fedora/ndesk-dbus-glib-0.3-1.fc8.src.rpm Description: ndesk-dbus-glib is glib mainloop integration for the C# implementation of D-Bus ndesk-dbus. This is required to make DBus communication work for many Mono apps, such as Banshee where the outdated dbus-sharp currently prevents correct functioning. *Note* this does not currently build in Mock, please consider pointing out my mistake to me.
block on 245491 since this depends on ndesk-dbus which is also under review.
Spec URL: http://www.lovesunix.net/fedora/ndesk-dbus-glib.spec SRPM URL: http://www.lovesunix.net/fedora/ndesk-dbus-glib-0.3-2.fc8.src.rpm Okay small thinko, .mdb files aren't MonoDevelop project files and thus don't go in -devel.
Spec URL: http://www.lovesunix.net/fedora/ndesk-dbus-glib.spec SRPM URL: http://www.lovesunix.net/fedora/ndesk-dbus-glib-0.3-3.fc8.src.rpm Significantly less hacky. Also let's not attempt to build that debug stuff.
Spec URL: http://www.lovesunix.net/fedora/ndesk-dbus-glib.spec SRPM URL: http://www.lovesunix.net/fedora/ndesk-dbus-glib-0.3-4.fc8.src.rpm Fix a few problems found during the ndesk-dbus review which also applied to this one.
Spec URL: http://www.lovesunix.net/fedora/ndesk-dbus-glib.spec SRPM URL: http://www.lovesunix.net/fedora/ndesk-dbus-glib-0.3-5.fc8.src.rpm And let's not build this on ppc64, it lacks the proper dependencies.
I'll review this shortly.
I'm dropping this as part of orphaning my packages. I'll leave the package and spec on the server if someone else wants to pick it up at least this might save them some time.
Okay, I got the personal issues that caused me to abandon Fedora contribution worked out, as nobody has expressed an interest in this package in my regretable absence I'm picking it up again. ndesk-dbus-glib will be required to build the dependencies for Banshee in the near future so we'll probably need to put this in the repos anyways soon. According to the rules since 3 months has passed it has to undergo a new review. Changes: * Build this as noarch since that upstream recommended this. Spec URL: http://www.lovesunix.net/fedora/ndesk-dbus-glib.spec SRPM URL: http://www.lovesunix.net/fedora/ndesk-dbus-glib-0.3-6.fc8.src.rpm
Revert noarch change to confort to the guidelines, Upstream 0 : Peter Gordon 1 Spec URL: http://www.lovesunix.net/fedora/ndesk-dbus-glib.spec SRPM URL: http://www.lovesunix.net/fedora/ndesk-dbus-glib-0.3-7.fc8.src.rpm
Okey dokey....sorry about the lateness of this. Here we go! == GOOD == + Naming and version/release are appropriate. + Source tarball matches that of upstream: 319d423fa3dfe8bfddba69e2b7c58df8 dbus-sharp-glib-0.3-upstream.tar.gz 319d423fa3dfe8bfddba69e2b7c58df8 dbus-sharp-glib-0.3-srpm.tar.gz + License (MIT) is Fedora-acceptable and matches the licenses of the source files. + Spec file is written in American English, and is legible. + Builds fine in mock (F7 and Development, x86_64); all BuildRequires are listed without duplicates or repeats via dependencies. + ExcludeArch bug filed as appropriate (PPC64) + Final file/directory ownership is good, and does not conflict with system packages. No duplicate files are listed. + Files listed properly with an appropriate %defattr(...) line. + A %clean section exists and includes only "rm -rf $RPM_BUILD_ROOT" as required. + Macro usage is consistent. + Package contains code. + pkgconfig data (.pc file) is in a devel subpackage as required; and the devel subpackage properly hardcodes a dependency on pkgconfig. + devel subpackage properly hardcodes a dependency on its parent (ndesk-dbus-glib) of the same Version/Release. + The buildroot is properly cleaned at the beginning of %install (via "rm -rf $RPM_BUILD_ROOT"). + All filenames are valid UTF-8. == MINOR == (1) The "Url" tag in your spec file should be all-uppercase: "URL: ..." (2) rpmlint complains about the source RPM, though the binary > ndesk-dbus-glib.src: W: mixed-use-of-spaces-and-tabs (spaces: line 17, tab: line 1) Just an inconsistency between using spaces and tabs for indenting. Choose one or the other and stick to it, please. :) (3) Your spec contains: > Source1: include.mk > Patch0: build-fix.patch Please prefix any extra files with %{name}, so that file conflicts are kept to a minimum when installing multiple source RPMs to the same build tree. (4) You're using %{?buildsubdir} in the various MONO_SHARED_DIR settings. Is that really needed? There seems to be no definition for it in the standard rpm macros. (Then again, this was also in the ndesk-dbus package, so if you remove it from here, please remove it in that package as well to keep consistent.) (5) While understandable, the ndesk-dbus dependency is not needed, as RPM Mono-handling is sane enough that it registers the library as a mono() provides, and knows that this is linked to it so uses that for it's dependency. During their builds, the following are output: ndesk-dbus: > Provides: mono(NDesk.DBus) = 1.0.0.0 ndesk-dbus-glib: > Provides: mono(NDesk.DBus.GLib) = 1.0.0.0 > Requires: mono(NDesk.DBus) = 1.0.0.0 mono(mscorlib) = 2.0.0.0 ndesk-dbus Unless another alternate package also provides this, you should not need to manually specify the ndesk-dbus dep. === BLOCKER === - Since the upstream source tarball includes a copy of the license text (COPYING), please add to the %doc of the package. == NOT APPLICABLE == * Includes no i18n, so %find_lang is not needed. * Does not install shared libs; /sbin/ldconfig invocations are not needed. * Package is not relocatable. * Package does not need a doc subpackage, since no documentation is included. * No %doc files are listed; thus nothing in runtime breaks if they are not present. * No headers or static libraries; and no native-code shared libraries. * Package contains no .la (libtool) or GUI stuff. Also, would you please bump the package to 0.4.1 (available from their website)? Once we can filter through these small issues, it should be golden. :)
I fixed the release blockers.. and I upgraded to 0.4.1, as discussed on jabber this meant switching to a much cleaner autofoo build process. I added a dirty hack to both this and the base package to correctly set the libdir, I'm sure there's a cleaner way to do it but this one seemed good (ifarch really needs a ifarch 64bit thingy). * updated to 0.4.1 * Patches are gone * COPYING is in %doc * Dependency removed * Url -> URL * spaces -> tabs (all hail the mighty tab) Spec URL: http://www.lovesunix.net/fedora/ndesk-dbus-glib.spec SRPM URL: http://www.lovesunix.net/fedora/ndesk-dbus-glib-0.4.1-1.fc8.src.rpm
It's likely you'll need ndesk-dbus 0.6.0 for this, I have updated that spec and it builds locally on x86_64 but throw an error on koji. I'm trying to figure out where in the code the /usr/lib/mono/2.0/mscorlib.dll reference comes from, but I'd naturally love your expertise Senor Gordon. http://koji.fedoraproject.org/koji/taskinfo?taskID=230519
The Mono/multlib stuff should be resolved with recent updates. I'll try to build this locally later today and finish the review from that. Thanks.
And it hit the build system so here's a package: http://koji.fedoraproject.org/koji/buildinfo?buildID=23677
Mock build is good (testThe spec looks properly cleaned up as requested, except for one tiny detail: Is there a reason for the %ifarch conditional in the update spec's %install section? It looks to me that you're only using it to get the proper lib or lib64 directory; in which case you can simply use %{_libdir} instead of hardcoding /usr/lib or /usr/lib64. The reason I ask this is that other 64-bit arches (secondary, AFAIK) exist that you'd probably want to add to it if you choose to leave it as a conditional: ppc64, sparc64, alpha, et al. Keeping it as a simple %{_libdir} macro would preclude the need for any such conditional. Other than that, ndesk-dbus-glib-0.4.1-1 is APPROVED. Please fix that up before you import it. :)
Thank you Peter. New Package CVS Request ======================= Package Name: ndesk-dbus-glib Short Description: glib mainloop integration for ndesk-dbus Owners: dnielsen Branches: devel F-8 F-7 InitialCC: (none) Cvsextras Commits: Yes
cvs done.
This is, finally, in devel - closing http://koji.fedoraproject.org/koji/taskinfo?taskID=237147