Bug 923460 - Review Request: libqmi - glib helper library for the Qualcomm MSM Interface (QMI) protocol
Summary: Review Request: libqmi - glib helper library for the Qualcomm MSM Interface (...
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Yanko Kaneti
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
: 914644 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-03-19 22:01 UTC by Dan Williams
Modified: 2013-05-13 16:04 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2013-05-13 16:04:04 UTC
Type: ---
Embargoed:
yaneti: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)
Always generate docs on dist time (1.72 KB, patch)
2013-05-06 18:26 UTC, Yanko Kaneti
no flags Details | Diff

Description Dan Williams 2013-03-19 22:01:58 UTC
Spec URL: http://people.redhat.com/dcbw/libqmi.spec
SRPM URL: http://people.redhat.com/dcbw/libqmi-1.2.0-1.fc17.src.rpm
Description: This package contains the libraries that make it easier to use QMI functionality from applications that use glib.
Fedora Account System Username: dcbw

Comment 1 Yanko Kaneti 2013-03-19 22:45:35 UTC
*** Bug 914644 has been marked as a duplicate of this bug. ***

Comment 2 Dan Williams 2013-03-20 12:42:40 UTC
Yanko: oops :)  Didn't realize there was another request, sorry about that.

Comment 3 Dan Williams 2013-05-06 17:49:12 UTC
Hey, can you post the docs patch you had again?  I have no idea where it went, and I'm sure the pastebin expired.  I'd like to apply it upstream since it's been reviewed and approved.

Comment 4 Yanko Kaneti 2013-05-06 18:26:32 UTC
Created attachment 744285 [details]
Always generate docs on dist time

Perhaps this was the one. At least that's whats git diff 'ing at the tree I have here.

Comment 5 Dan Williams 2013-05-06 22:18:59 UTC
Updated with Yanko's patches:

Spec URL: http://people.redhat.com/dcbw/libqmi.spec
SRPM URL: http://people.redhat.com/dcbw/libqmi-1.3.0-1.fc17.src.rpm
Description: This package contains the libraries that make it easier to use QMI functionality from applications that use glib.
Fedora Account System Username: dcbw

Comment 6 Yanko Kaneti 2013-05-07 06:59:04 UTC
The Source line needs an explanation or ideally a url for a new release, which would also help get rid of the ORIG-docs cruft.
There is also some legacy leftovers - BuildRoot, %define vs %global, %defattr, %{__rm} -rf $RPM_BUILD_ROOT

Comment 7 Dan Williams 2013-05-08 19:33:20 UTC
Updated again; I added a comment for the SOURCE line; there won't always be fixes, and it's much smoother to make dist a tarball from git than it is to git format-patch -X and add those all to the fedora package.  Thus there isn't an upstream source URL from the tarball I've used.

Comment 8 Dan Williams 2013-05-08 19:33:52 UTC
Spec URL: http://people.redhat.com/dcbw/libqmi.spec
SRPM URL: http://people.redhat.com/dcbw/libqmi-1.3.0-1.git20130508.fc17.src.rpm
Description: This package contains the libraries that make it easier to use QMI functionality from applications that use glib.
Fedora Account System Username: dcbw

Comment 9 Yanko Kaneti 2013-05-08 21:36:46 UTC
wrt Source: I prefer the exact opposite approach, but given you'll be the maintainer there is little point in blocking on it.

Looking at the source I think the main package license should be LGPLv2+ and only the -utils  GPLv2+

Some minor issues:
libqmi.x86_64: W: incoherent-version-in-changelog 1.2.0-1 ['1.3.0-1.git20130508.fc20', '1.3.0-1.git20130508']
libqmi.x86_64: E: zero-length /usr/share/doc/libqmi-1.3.0/README
libqmi-devel.x86_64: E: zero-length /usr/share/doc/libqmi-devel-1.3.0/ChangeLog
libqmi-devel.x86_64: W: summary-ended-with-dot C Header files for adding QMI support to applications that use glib.
libqmi-utils.x86_64: W: summary-ended-with-dot C Utilities to use the QMI protocol from the command line.

Builds in mock.
Reasonably named and versioned.
Does not conflict with anything existing.

Please fix the license before importing.

APPROVED

Comment 10 Dan Williams 2013-05-10 16:35:32 UTC
Fixed all those errors, thanks!

Comment 11 Dan Williams 2013-05-10 16:39:42 UTC
New Package SCM Request
=======================
Package Name: libqmi
Short Description: helper library for WWAN modems implement the QMI protocol
Owners: dcbw
Branches: f18 f19

Comment 12 Gwyn Ciesla 2013-05-10 17:14:14 UTC
Git done (by process-git-requests).

Comment 13 Dan Williams 2013-05-13 16:04:04 UTC
Imported and built.  Thanks everyone!


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