Bug 975316 - Review Request: libodb-qt - Qt ODB runtime library
Summary: Review Request: libodb-qt - Qt ODB runtime library
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Peter Lemenkov
QA Contact: Fedora Extras Quality Assurance
Depends On: 975312
TreeView+ depends on / blocked
Reported: 2013-06-18 05:08 UTC by Dave Johansen
Modified: 2013-11-19 01:22 UTC (History)
7 users (show)

Fixed In Version: libodb-qt-2.2.1-1.el5
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2013-11-09 03:36:43 UTC
Type: ---
lemenkov: fedora-review+
gwync: fedora-cvs+

Attachments (Terms of Use)

Description Dave Johansen 2013-06-18 05:08:32 UTC
Spec URL: https://docs.google.com/file/d/0B9bQ41XLJ8QuZlVlNGh2b2pzUjA/edit?usp=sharing
SRPM URL: https://docs.google.com/file/d/0B9bQ41XLJ8QuRTI1eWNkbTJNdGM/edit?usp=sharing
Description: This package contains the Qt profile library. The Qt profile provides support for persisting Qt smart pointers, containers, and value types with the ODB system.
Fedora Account System Username: daveisfera

Comment 1 Peter Lemenkov 2013-06-18 14:48:55 UTC
Unblocking FE-NEEDSPONSOR - I've just sponsored Dave.

Comment 3 Dave Johansen 2013-10-06 03:34:23 UTC
This package is ready for review and a scratch build on rawhide is available at: http://koji.fedoraproject.org/koji/taskinfo?taskID=6028297

Comment 4 Dave Johansen 2013-10-06 04:22:21 UTC
Sorry, I forgot to include the link to the current source RPM in the last comment. It can be found at: http://daveisfera.fedorapeople.org/odb_2.2/SRPMS/libodb-qt-2.2.1-1.el6.src.rpm

Comment 5 Rex Dieter 2013-10-06 13:08:25 UTC
Small suggestion:

# If building on Fedora or RHEL 6
%if 0%{?rhel}%{?fedora} >= 6
BuildRequires: qt-devel
BuildRequires: qt4-devel

Can be simplified to just
BuildRequires: qt4-devel
this should work everywhere (rhel/fedora qt pkg has a virtual provides for qt4- namespace to handle this use-case)

Comment 6 Dave Johansen 2013-10-06 19:08:01 UTC
Ok, thanks for the advice on that. I made the change an uploaded the simplified spec file and the corresponding source RPM.

Comment 7 Boris Kolpackov 2013-10-07 04:53:54 UTC
Wouldn't changing qt-devel to qt4-devel mean that we will always depend on Qt4 even though the default (qt-devel) might actually be Qt5? ODB supports both and I think if the default is/becomes Qt5 in some version of Fedora, then we should depend on that.

Comment 8 Christopher Meng 2013-10-07 08:59:21 UTC
Suggestion, why not rm -rf %{_datadir}/doc/libodb-qt in %install and then mark:
%doc GPLv2

in %files?

No need to waste 5 lines in %files IMHO.

Comment 9 Rex Dieter 2013-10-07 14:39:58 UTC
re: comment #7

There will (likely) never be a Qt5-based qt-devel pkg.  IMO, better to specify exactly what you want, so you know what you're getting and there are no surprises.

To depend on Qt5, you can use (something like) one of the following:
BuildRequires: qt5-qtbase-devel
BuildRequires: pkgconfig(Qt5Core)

The Qt4 versions of those being:
BuildRequires: qt4-devel
BuildRequires: pkgconfig(QtCore)

Comment 10 Peter Lemenkov 2013-10-17 14:44:49 UTC
Looks good.


Legend: + = PASSED, - = FAILED, 0 = Not Applicable

+ rpmlint is almost silent

work ~/Desktop: rpmlint libodb-qt-*
libodb-qt.src: W: spelling-error Summary(en_US) runtime -> run time, run-time, rudiment
libodb-qt.x86_64: W: spelling-error Summary(en_US) runtime -> run time, run-time, rudiment
libodb-qt.x86_64: W: no-documentation
4 packages and 0 specfiles checked; 0 errors, 3 warnings.
work ~/Desktop: 

+ The package is named according to the  Package Naming Guidelines.
+ The spec file name matches the base package %{name}, in the format %{name}.spec.
+ The package meets the Packaging Guidelines.
+ The package is licensed with a Fedora approved license and meets the Licensing Guidelines.
+ The License field in the package spec file matches the actual license (strict GPLv2).
+ The 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 package successfully compiles and builds into binary rpms on at least one primary architecture. See Koji link above.
+ All build dependencies are listed in BuildRequires.
0 No need to handle locales.
+ The package stores shared library files in some of the dynamic linker's default paths, and it calls ldconfig in %post and %postun.
+ The package does NOT bundle copies of system libraries.
0 The package is not designed to be relocatable.
+ The package owns all directories that it creates.
+ The package does not list a file more than once in the spec file's %files listings.
+ Permissions on files are set properly.
+ The package has a %clean section, which contains rm -rf %{buildroot} (or $RPM_BUILD_ROOT).
+ The package consistently uses macros.
+ The package contains code, or permissible content.
0 No extremely large documentation files.
+ Anything, the package includes as %doc, does not affect the runtime of the application. Well, since we're moving to unversioned docdirs this will definitely cause issues. Please be careful with EL5 and EL6.
+ Header files are stored in a -devel package.
0 No static libraries.
+ The pkgconfig(.pc) files are stored in a -devel package. A necessary runtime requirement is picked up automatically in Fedora, but EL5 (and maybe even EL6) will require explicit "Requires: pkgconfig"
+ The library file(s) that end in .so (without suffix) is(are) stored in a -devel package.
+ The -devel package requires the base package using a fully versioned dependency: Requires: %{name}%{?_isa} = %{version}-%{release}
+ The package does NOT contain any .la libtool archives.
0 Not a GUI application.
+ The package does not own files or directories already owned by other packages.
+ At the beginning of %install, the package runs rm -rf %{buildroot} (or $RPM_BUILD_ROOT).
+ All filenames in rpm packages are valid UTF-8.


Comment 11 Dave Johansen 2013-10-22 14:57:31 UTC
New Package SCM Request
Package Name: libodb-qt
Short Description: Qt ODB runtime library from Code Synthesis
Owners: daveisfera
Branches: el5 el6 f18 f19 f20 
InitialCC: peter

Comment 12 Gwyn Ciesla 2013-10-22 15:10:15 UTC
Git done (by process-git-requests).

Comment 13 Fedora Update System 2013-10-30 03:01:43 UTC
libodb-qt-2.2.1-1.fc18 has been submitted as an update for Fedora 18.

Comment 14 Fedora Update System 2013-10-30 03:02:05 UTC
libodb-qt-2.2.1-1.fc19 has been submitted as an update for Fedora 19.

Comment 15 Fedora Update System 2013-10-31 03:00:54 UTC
libodb-qt-2.2.1-1.fc18 has been pushed to the Fedora 18 testing repository.

Comment 16 Fedora Update System 2013-11-02 23:46:02 UTC
libodb-qt-2.2.1-1.el5 has been submitted as an update for Fedora EPEL 5.

Comment 17 Fedora Update System 2013-11-02 23:46:18 UTC
libodb-qt-2.2.1-1.el6 has been submitted as an update for Fedora EPEL 6.

Comment 18 Fedora Update System 2013-11-03 04:33:52 UTC
libodb-pgsql-2.2.0-1.fc20 has been submitted as an update for Fedora 20.

Comment 19 Fedora Update System 2013-11-09 03:36:43 UTC
libodb-qt-2.2.1-1.fc18 has been pushed to the Fedora 18 stable repository.

Comment 20 Fedora Update System 2013-11-09 03:38:29 UTC
libodb-qt-2.2.1-1.fc19 has been pushed to the Fedora 19 stable repository.

Comment 21 Fedora Update System 2013-11-10 07:13:28 UTC
libodb-pgsql-2.2.0-1.fc20 has been pushed to the Fedora 20 stable repository.

Comment 22 Fedora Update System 2013-11-19 01:21:31 UTC
libodb-qt-2.2.1-1.el6 has been pushed to the Fedora EPEL 6 stable repository.

Comment 23 Fedora Update System 2013-11-19 01:22:33 UTC
libodb-qt-2.2.1-1.el5 has been pushed to the Fedora EPEL 5 stable repository.

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