Bug 595637

I have a slow internet connection atm.

Will do a full review, when I can download the qt stuff this week...

A few comments for now:
- Is there a reason why you use %{?isa}? I don't think leaving them out will cause problems, but maybe I'm wrong...?
This makes only sense, when using e.g:
%ifarch %ix86
Requires: %{name}.(i?86|athlon|geode)
%endif
%ifarch x86_64 amd64 ia32e
Requires: %{name}.(x86_64|amd64|ia32e)
%endif

Or is this common in KDE?
(This is my first look at a KDE/qt package ever^^)

- You aren't consistend with tabbing:
  4 Name:           qoauth
  5 Version:                1.0.1
  6 Release:                0.1.%{gitdate}git%{githash}%{?dist}
  7 Summary:                Qt-based C++ library for OAuth authorization scheme
  8 Group:          System Environment/Libraries
  9 License:                LGPLv2+
 10 URL:                    http://github.com/ayoy/qoauth

Would be more readable otherwise.

- checksum does not match (which is excpected, because of the timestamps):
  diff -r is clean

- name ok
- koji successfull: http://koji.fedoraproject.org/koji/taskinfo?taskID=2259994
- rpmlint ok:
$ rpmlint ./qoauth-1.0.1-0.1.20100525gitec7e4d5.fc13.src.rpm ./x86_64/qoauth-*
qoauth.src:11: W: macro-in-comment %{version}
qoauth.src:11: W: macro-in-comment %{name}
qoauth.src:11: W: macro-in-comment %{version}
qoauth.src:14: W: macro-in-comment %h
qoauth.src:14: W: macro-in-comment %h
qoauth.src: W: no-cleaning-of-buildroot %install
qoauth.src: W: no-buildroot-tag
qoauth.src: W: invalid-url Source0: qoauth-ec7e4d5.tar.bz2
4 packages and 0 specfiles checked; 0 errors, 8 warnings.

macro-in-comment could be avoided when using s/%/%%/g.

- libs correctly packaged
- no static libs
- no *.la
- parallel make is there
- BR/R ok
- doc is not too big, to need to split in a subpackage

#############################################################################

Only cosmetic issues: macro-in-comment + tabbing + %{?isa}.

#############################################################################

APPROVED

New Package CVS Request
=======================
Package Name: qoauth
Short Description: Qt-based C++ library for OAuth authorization scheme
Owners: supercyper
Branches: F-12 F-13
InitialCC:

(In reply to comment #1)
> 
> A few comments for now:
> - Is there a reason why you use %{?isa}? I don't think leaving them out will
> cause problems, but maybe I'm wrong...?
> This makes only sense, when using e.g:
> %ifarch %ix86
> Requires: %{name}.(i?86|athlon|geode)
> %endif
> %ifarch x86_64 amd64 ia32e
> Requires: %{name}.(x86_64|amd64|ia32e)
> %endif
> 
> Or is this common in KDE?
> (This is my first look at a KDE/qt package ever^^)
It's common in KDE packages, it's useless under most circumstance, but I still suggest to use it when requires multiarch libs explicitly.

See https://fedoraproject.org/wiki/PackagingDrafts/ArchSpecificRequires

> 
> - You aren't consistend with tabbing:
>   4 Name:           qoauth
>   5 Version:                1.0.1
>   6 Release:                0.1.%{gitdate}git%{githash}%{?dist}
>   7 Summary:                Qt-based C++ library for OAuth authorization scheme
>   8 Group:          System Environment/Libraries
>   9 License:                LGPLv2+
>  10 URL:                    http://github.com/ayoy/qoauth
> 
> Would be more readable otherwise.
> 
My Editor is wrong some days ago: ), will be fixed soon.

CVS done (by process-cvs-requests.py).

Thanks a lot for the review. I fixed the following issues already except macro-in-comment and add %check section to the spec.

See http://cvs.fedoraproject.org/viewvc/devel/qoauth/qoauth.spec?revision=1.3&view=markup