Bug 771485 - Review Request: libcxgb4 - Chelsio T4 iWARP HCA Userspace Driver
Summary: Review Request: libcxgb4 - Chelsio T4 iWARP HCA Userspace Driver
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Albert Strasheim
QA Contact: Fedora Extras Quality Assurance
URL: http://people.redhat.com/dledford/Pac...
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-01-03 21:21 UTC by Doug Ledford
Modified: 2012-02-01 19:28 UTC (History)
4 users (show)

Fixed In Version: libcxgb4-1.2.0-1.fc16
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-02-01 19:23:43 UTC
Type: ---
fenlason: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Doug Ledford 2012-01-03 21:21:38 UTC
This package is a hardware driver that enables Chelsio T4 hardware to be used as an iWARP capable device in combination with the libibverbs package.  The libibverbs package is the hardware independent RDMA library, and the libcxgb4 package is one of multiple possible hardware driver modules that are abstracted out by libibverbs.

The package for review can be found at the URL above.

Comment 1 Albert Strasheim 2012-01-09 13:30:07 UTC
$ rpmlint libcxgb4*
libcxgb4.x86_64: W: spelling-error Summary(en_US) Chelsio -> Chelsea
libcxgb4.x86_64: W: spelling-error Summary(en_US) iWARP -> i Warp, warp, antiwar
libcxgb4.x86_64: W: spelling-error %description -l en_US Userspace -> User space, User-space, Users pace
libcxgb4.x86_64: W: spelling-error %description -l en_US libibverbs -> verbalizes
libcxgb4.x86_64: W: spelling-error %description -l en_US iWARP -> i Warp, warp, antiwar
libcxgb4.x86_64: W: obsolete-not-provided libcxgb4-devel
libcxgb4.x86_64: W: non-conffile-in-etc /etc/modprobe.d/libcxgb4.conf
libcxgb4.x86_64: W: non-conffile-in-etc /etc/libibverbs.d/cxgb4.driver
libcxgb4.spec:13: W: unversioned-explicit-obsoletes %{name}-devel
libcxgb4.spec:15: W: unversioned-explicit-provides libibverbs-driver.%{_arch}
libcxgb4-static.x86_64: W: no-documentation
2 packages and 1 specfiles checked; 0 errors, 11 warnings.

I don't have this hardware, so I can't test further unfortunately.

I noticed that libmlx4-static provides libmlx4-devel-static. What's up with that? Should libcxgb4-static do the same?

Comment 2 Doug Ledford 2012-01-09 19:29:05 UTC
libmlx4 provides libmlx4-devel-static because early versions of the package provided that.  We switched the package naming over to the more standard compliant base base-devel base-static long ago and the old provides is in libmlx4 to keep any software that didn't get updated from breaking.  Libcxgb4 has no such broken history and need not provide -devel-static as a package.

Comment 3 Doug Ledford 2012-01-11 19:08:11 UTC
Albert, are you good with signing off on this package at this point?

Comment 4 Albert Strasheim 2012-01-11 19:23:24 UTC
Looks good to me.

Comment 5 Doug Ledford 2012-01-11 20:10:53 UTC
New Package SCM Request
=======================
Package Name: libcxgb4
Short Description: Driver for Chelsio T4 hardware
Owners: dledford
Branches: f15 f16
InitialCC:

Comment 6 Gwyn Ciesla 2012-01-11 20:36:16 UTC
Git done (by process-git-requests).

Comment 7 Fedora Update System 2012-01-11 20:52:27 UTC
libcxgb4-1.2.0-1.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/libcxgb4-1.2.0-1.fc15

Comment 8 Fedora Update System 2012-01-11 20:52:36 UTC
libcxgb4-1.2.0-1.fc16 has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/libcxgb4-1.2.0-1.fc16

Comment 9 Fedora Update System 2012-01-15 20:05:21 UTC
libcxgb4-1.2.0-1.fc16 has been pushed to the Fedora 16 testing repository.

Comment 10 Fedora Update System 2012-02-01 19:23:43 UTC
libcxgb4-1.2.0-1.fc15 has been pushed to the Fedora 15 stable repository.

Comment 11 Fedora Update System 2012-02-01 19:28:07 UTC
libcxgb4-1.2.0-1.fc16 has been pushed to the Fedora 16 stable repository.


Note You need to log in before you can comment on or make changes to this bug.