Bug 656010

rpmlint is not silent.

libsrtp.src: W: no-cleaning-of-buildroot %clean
libsrtp.src: W: no-buildroot-tag
libsrtp.src: W: no-%clean-section
libsrtp.src: W: invalid-url Source0: srtp-1.4.4-20101004cvs.tar.bz2
libsrtp.x86_64: W: shared-lib-calls-exit /usr/lib64/libsrtp.so.0.0.0 exit.5
2 packages and 0 specfiles checked; 0 errors, 5 warnings.

Exit call is not good. Could you provide any reasons why it might be leaved as is?

Is there any way to specify the revision for csv checkout in spec?

* The package named according to the Package Naming Guidelines.
* The spec file name match the base package %{name}. 
* The package meets the Packaging Guidelines.
* The package is licensed with a Fedora approved license.
* The License field in the package spec file matchs the actual license.
* File, containing the text of the license(s) for the package is included in %doc.
* The spec file is written in American English.
* The spec file for the package is legible.
* The sources used to build the package match the upstream source. MARK: md5sum on provided .tar.bz2 and obtained .tar.bz2 are MISMATCHED, but the sources are EQUAL.
* The package successfully compiles and builds into binary rpms on x86_64 F14.
* All build dependencies are listed in BuildRequires.
* There are no locales.
* Binary RPM package which stores shared library files calls ldconfig in %post and %postun.
* Packages do not bundle copies of system libraries.
* A package owns all directories that it creates.
* There are no files, listed more than once in the spec file's %files listings.
* Permissions on files are set properly.
* The package contains code, or permissable content.
* There is no large documentation.
* Header files are in a -devel package.
* There are no static libraries.
* Library files that end in .so (without suffix) is in a -devel package.
* Devel packages requires the base package.
* Packages doed not contain any .la libtool archives.
* All filenames in rpm packages are valid UTF-8.

All is need are the answers on two above questions.

Honestly, I'm not sure how to get any sort of sane revision for the tree from cvs. If you know how, I'd be happy to add it.

The exit() call is in crypto_kernel_status (in crypto/kernel/crypto_kernel.c)

  /* for each cipher type, describe and test */
  while(ctype != NULL) {
    printf("cipher: %s\n", ctype->cipher_type->description);
    printf("  instance count: %d\n", ctype->cipher_type->ref_count);
    printf("  self-test: ");
    status = cipher_type_self_test(ctype->cipher_type);
    if (status) {
      printf("failed with error code %d\n", status);
      exit(status);
    }
    printf("passed\n");
    ctype = ctype->next;
  }

Unfortunately, it is not trivial to remove that exit(), so I'm inclined to just leave it in.

With CVS all you have is a date, or a tag if upstream has made one.  There is no concept analogous to SVN's revision number or git's commit hash.

>Unfortunately, it is not trivial to remove that exit()

Ok. What about ping upstream?
Not a blocker. 

>With CVS all you have is a date

Yes, -D, found it. May be, just may be, it would be nice to be sure there is _this_ version that in the package. Not a blocker.

APPROVED.

New Package SCM Request
=======================
Package Name: libsrtp
Short Description: An implementation of the Secure Real-time Transport Protocol (SRTP)
Owners: spot jcollie
Branches: f13 f14 devel
InitialCC:

Git done (by process-git-requests).

libsrtp-1.4.4-1.20101004cvs.fc13 has been submitted as an update for Fedora 13.
https://admin.fedoraproject.org/updates/libsrtp-1.4.4-1.20101004cvs.fc13

libsrtp-1.4.4-1.20101004cvs.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/libsrtp-1.4.4-1.20101004cvs.fc14

libsrtp-1.4.4-1.20101004cvs.fc13 has been pushed to the Fedora 13 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update libsrtp'.  You can provide feedback for this update here: https://admin.fedoraproject.org/updates/libsrtp-1.4.4-1.20101004cvs.fc13

libsrtp-1.4.4-1.20101004cvs.fc13 has been pushed to the Fedora 13 stable repository.  If problems still persist, please make note of it in this bug report.

libsrtp-1.4.4-1.20101004cvs.fc14 has been pushed to the Fedora 14 stable repository.  If problems still persist, please make note of it in this bug report.

Package Change Request
======================
Package Name: libsrtp
New Branches: el6
Owners: jcollie
InitialCC: 

Wanted for Asterisk SRTP support.

Git done (by process-git-requests).

Package Change Request
======================
Package Name: libsrtp
New Branches: epel7
Owners: jcollie
InitialCC: 

For Asterisk SRTP support

Git done (by process-git-requests).