Bug 665877

* Informed upstream to use %{name}-%{version}.tar.gz naming convention.

$ rpmlint freeDiameter.spec
freeDiameter.spec: W: invalid-url Source0: http://www.freediameter.net/hg/freeDiameter/archive/freeDiameter-1.0.3.tar.gz HTTP Error 500: Internal Server Error
0 packages and 1 specfiles checked; 0 errors, 1 warnings.

$ rpmlint ../RPMS/i686/freeDiameter-1.0.3-1.fc14.i686.rpm 
freeDiameter.i686: W: no-manual-page-for-binary freeDiameterd-1.0.3
freeDiameter.i686: W: no-manual-page-for-binary freeDiameterd
1 packages and 0 specfiles checked; 0 errors, 2 warnings.

$ rpmlint ../RPMS/i686/freeDiameter-devel-1.0.3-1.fc14.i686.rpm 
freeDiameter-devel.i686: W: no-documentation
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

$ rpmlint ../SRPMS/freeDiameter-1.0.3-1.fc14.src.rpm 
freeDiameter.src: W: invalid-url Source0: http://www.freediameter.net/hg/freeDiameter/archive/freeDiameter-1.0.3.tar.gz HTTP Error 500: Internal Server Error
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

Successful Koji builds for F-13, F-14 and EL-6 respectively:
http://koji.fedoraproject.org/koji/taskinfo?taskID=2690177
http://koji.fedoraproject.org/koji/taskinfo?taskID=2690180
http://koji.fedoraproject.org/koji/taskinfo?taskID=2690181

Looks like new version 1.1.0 is already available. Can you package it?

Looking at the spec file
1) Good preserve timestamp by changing
make install DESTDIR=%{buildroot}
to
make install DESTDIR=%{buildroot} INSTALL="install -p"

2) Avoid macro usage for command rm. See http://fedoraproject.org/wiki/Packaging/Guidelines#Macros

Preserved timestamp. Removed use of macro for command rm. Updated upstream package:

SPEC: http://shakthimaan.fedorapeople.org/SPECS/freeDiameter.spec
SRPM: http://shakthimaan.fedorapeople.org/SRPMS/freeDiameter-1.1.0-1.fc14.src.rpm

$  rpmlint freeDiameter.spec 
0 packages and 1 specfiles checked; 0 errors, 0 warnings.

$  rpmlint freeDiameter-1.1.0-1.fc14.src.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

$  rpmlint freeDiameter-1.1.0-1.fc14.i686.rpm 
freeDiameter.i686: W: shared-lib-calls-exit /usr/lib/libfdcore.so.1.1.0 exit
freeDiameter.i686: W: no-manual-page-for-binary freeDiameterd
freeDiameter.i686: W: no-manual-page-for-binary freeDiameterd-1.1.0
1 packages and 0 specfiles checked; 0 errors, 3 warnings.

$  rpmlint freeDiameter-devel-1.1.0-1.fc14.i686.rpm 
freeDiameter-devel.i686: W: no-documentation
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

Will inform upstream to remove exit() call. Successful Koji builds for F-15, F-16, EL-6:

http://koji.fedoraproject.org/koji/taskinfo?taskID=3115184
http://koji.fedoraproject.org/koji/taskinfo?taskID=3115189
http://koji.fedoraproject.org/koji/taskinfo?taskID=3115192

Review:
1) Upstream source verified as (sha1sum)
ddd683b90f2755163605064deb8fab5f15d22414  1.1.0.tar.gz
ddd683b90f2755163605064deb8fab5f15d22414  ../SOURCES/freeDiameter-1.1.0.tar.gz

2) rpmlint output on rpms gave
freeDiameter-devel.x86_64: W: no-documentation
freeDiameter.x86_64: W: shared-lib-calls-exit /usr/lib64/libfdcore.so.1.1.0 exit.5
freeDiameter.x86_64: W: no-manual-page-for-binary freeDiameterd
freeDiameter.x86_64: W: no-manual-page-for-binary freeDiameterd-1.1.0
3 packages and 0 specfiles checked; 0 errors, 4 warnings.

3) Package follows packaging guidelines.

Suggestions:
1) The only working download link I found is http://www.freediameter.net/hg/freeDiameter/archive/1.1.0.tar.gz
so change the Source0 tag to

Source0:        http://www.freediameter.net/hg/%{name}/archive/%{version}.tar.gz

and add a comment before this Source tag that upstream tarball is renamed to follow %{name}-%{version}.tar.gz conventions.

APPROVED.

New Package SCM Request
=======================
Package Name: freeDiameter
Short Description: A Diameter protocol open implementation
Owners: shakthimaan
Branches: F-15 F-16 EL-6
InitialCC: shakthimaan

Git done (by process-git-requests).

devel==f16, created by default.

http://www.freediameter.net/hg/%{name}/archive/%{name}-%{version}.tar.gz works, otherwise rpmlint would have complained as well.

I am not able to switch to f-15 or el-6 branch. I get:

  $ fedpkg switch-branch f15
  Unable to switch to another branch: Unknown remote branch f15

The commit access is not approved? See:

  https://admin.fedoraproject.org/pkgdb/acls/name/freeDiameter

No problem switching branch for me, and the ACLs appear quite fine on the server.  Everything looks fine in pkgdb as well; I don't see anything wrong at the link you gave.  Perhaps you could indicate what you think is incorrect there.

Never mind. I am able to switch to f15 and el6 branches now, and able to upload the package. Thanks for your reply.

freeDiameter-1.1.0-1.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/freeDiameter-1.1.0-1.el6

freeDiameter-1.1.0-1.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/freeDiameter-1.1.0-1.fc15

freeDiameter-1.1.0-1.fc15 has been pushed to the Fedora 15 testing repository.

freeDiameter-1.1.0-1.fc15 has been pushed to the Fedora 15 stable repository.

Package Change Request
======================
Package Name: freeDiameter
New Branches: F-14
Owners: shakthimaan
InitialCC: shakthimaan

Git done (by process-git-requests).

freeDiameter-1.1.0-1.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/freeDiameter-1.1.0-1.fc14

freeDiameter-1.1.0-1.el6 has been pushed to the Fedora EPEL 6 stable repository.

freeDiameter-1.1.0-1.fc14 has been pushed to the Fedora 14 stable repository.