Bug 446639

Updated to the latest release

Spec URL: http://fedora.roving-it.com/rawhide/gupnp.spec
SRPM URL: http://fedora.roving-it.com/rawhide/gupnp-0.10.1-1.fc9.src.rpm

Updated to the latest release.

Spec URL: http://fedora.roving-it.com/rawhide/gupnp.spec
SRPM URL: http://fedora.roving-it.com/rawhide/gupnp-0.12-1.fc9.src.rpm

Updated to the latest release.

Spec URL: http://fedora.roving-it.com/rawhide/gupnp.spec
SRPM URL: http://fedora.roving-it.com/rawhide/gupnp-0.12.1-1.fc9.src.rpm

New location:

SPEC: http://pbrobinson.fedorapeople.org/gupnp.spec
SRPM: http://pbrobinson.fedorapeople.org/gupnp-0.12.1-1.fc9.src.rpm

I've taken this bug, but I'll wait to see a buildable package
from bug 446637 first.

I'm getting a build error on rawhide which looks like the same
problem we had with bug 446637:

gcc -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include -I/usr/include/gssdp-1.0 -I/usr/include/libsoup-2.4 -I/usr/include/libxml2 -I.. -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector --param=ssp-buffer-size=4 -m32 -march=i386 -mtune=generic -fasynchronous-unwind-tables -o .libs/light-client light-client.o  ../libgupnp/.libs/libgupnp-1.0.so
light-client.o: In function `main':
/home/rjones/rpmbuild/BUILD/gupnp-0.12.1/examples/light-client.c:79: undefined reference to `g_thread_init'
light-server.o: In function `main':
/homecollect2: ld returned 1 exit status
/rjones/rpmbuild/BUILD/gupnp-0.12.1/examples/light-server.c:96: undefined reference to `g_thread_init'
collect2: ld returned 1 exit status
make[2]: *** [light-client] Error 1
make[2]: *** Waiting for unfinished jobs....
make[2]: *** [light-server] Error 1

applied a patch to stop the compile of the examples directory as its not installed/packaged anyway. Maintainers know about the issue and it will be fixed in the next release shortly.

SPEC: http://pbrobinson.fedorapeople.org/gupnp.spec
SRPM: http://pbrobinson.fedorapeople.org/gupnp-0.12.1-2.fc9.src.rpm

- rpmlint output

  gupnp.i386: W: incoherent-version-in-changelog 0.12-2 0.12.1-2.fc10

  The %changelog version numbers are incorrect.

+ package name satisfies the packaging naming guidelines
+ specfile name matches the package base name
+ package should satisfy packaging guidelines
+ license meets guidelines and is acceptable to Fedora
+ license matches the actual package license
+ %doc includes license file
+ spec file written in American English
+ spec file is legible
+ upstream sources match sources in the srpm
  f8b0c3a759e4bf18380ed49f37e0b207
+ package successfully builds on at least one architecture
  i386
n/a ExcludeArch bugs filed
+ BuildRequires list all build dependencies
n/a %find_lang instead of %{_datadir}/locale/*
+ binary RPM with shared library files must call ldconfig in %post and %postun
+ does not use Prefix: /usr
+ package owns all directories it creates
+ no duplicate files in %files
+ %defattr line
+ %clean contains rm -rf $RPM_BUILD_ROOT
+ consistent use of macros
+ package must contain code or permissible content
n/a large documentation files should go in -doc subpackage
+ files marked %doc should not affect package
+ header files should be in -devel
n/a static libraries should be in -static
+ packages containing pkgconfig (.pc) files need 'Requires: pkgconfig'
+ libfoo.so must go in -devel
+ -devel must require the fully versioned base
+ packages should not contain libtool .la files
n/a packages containing GUI apps must include %{name}.desktop file
+ packages must not own files or directories owned by other packages
+ %install must start with rm -rf %{buildroot} etc.
+ filenames must be valid UTF-8

Optional:

n/a if there is no license file, packager should query upstream
n/a translations of description and summary for non-English languages, if available
- reviewer should build the package in mock
- the package should build into binary RPMs on all supported architectures
- review should test the package functions as described
+ scriptlets should be sane
+ pkgconfig files should go in -devel
+ shouldn't have file dependencies outside /etc /bin /sbin /usr/bin or /usr/sbin

Just fix those %changelog version numbers and resubmit for approval.

New spec and srpm fixing the changelog. BTW what rpmlint options did you use to get that reported so I can get more info for next time :)

SRPM: http://pbrobinson.fedorapeople.org/gupnp-0.12.1-3.fc9.src.rpm

OK, that fixes the rpmlint problem, so ===> APPROVED <===

I'm using rpmlint in Rawhide.  No special options.

New Package CVS Request
=======================
Package Name: gupnp
Short Description: GUPnP is an framework for creating UPnP devices & control points
Owners: pbrobinson
Branches: F-8 F-9
InitialCC: pbrobinson
Cvsextras Commits: yes

cvs done.

Committed to CVS. Coming to a rawhide near you soon. Thanks for your help yet again Richard, feel free to swap any more reviews that you need :-)