Bug 529517

*** Bug 527860 has been marked as a duplicate of this bug. ***

FYI, Ryan is sponsored now, so any sponsored contributor can review and approve this package.

In any case, the firs thing I notice is that this has redundant "Requires: kdelibs4 >= 4.1 qt4 >= 4.4.0", please remove those. (Those versions have been current since "forever", so the automatic soname deps should be sufficient.)

(In reply to comment #2)
> FYI, Ryan is sponsored now, so any sponsored contributor can review and approve
> this package.  

Kevin,
I'll take it as I'm now helping some guy on local Linux forum to set up synaptic. I've already asked him not to report bugs/feature requests to forum but to bugzilla or IRC ;-)

Package name OK
Summary OK
License OK
URL & Sources FIXME
Sources MD5 FIXME (new version, sources URL, see comment)
Buildroot OK
Requires - do not require kdelibs4 >= 4.1 qt4 >= 4.4.0
Description OK
File attributes OK
Documentation installed OK
Desktop file OK (KDE service desktop file, no need for validation)
Changelog OK

Rpmlint
SPEC & SRPM OK
RPMS OK

Builds 
Scratch OK
Local OK

Fixme:
Group should be User Interfaces/Desktops to match other KControl module packages.
New version available - 0.3.0.
Sources URL - http://github.com/mishaaq/kcm_touchpad/tarball/kcm_touchpad-0.3.0 or 
  http://download.github.com/mishaaq-kcm_touchpad-000be4c.tar.gz.
Could you please update package, so I can check missing checks, thanks.
Do not use Requires.

I will update tonight and post latest SRPM etc.

Thanks Jaroslav :)

Updated.
http://rrix.fedorapeople.org/kcm_touchpad/kcm_touchpad-0.3.0-1.fc12.src.rpm
http://rrix.fedorapeople.org/kcm_touchpad/kcm_touchpad.spec

http://koji.fedoraproject.org/koji/taskinfo?taskID=1786723

Ryan,
please fix remaining issues - group and source url.

http://rrix.fedorapeople.org/kcm_touchpad/kcm_touchpad-0.3.0-2.fc12.src.rpm
http://rrix.fedorapeople.org/kcm_touchpad/kcm_touchpad.spec

Source checksum unchanged, no koji scratch build submitted.

Hmm, looks like http://github.com/mishaaq/kcm_touchpad/tarball/kcm_touchpad-0.3.0 is better Source URL as they do some magic with waitdownload, download URLs... So the question is how to handle correctly GitHub downloads? Could you update it? Sorry, I thought it would work :(

Checksum is OK, once you change Source URL again, I'll approve it. Thanks.

That's not a good source URL, as it doesn't have the .tar.bz2. Another issue is that these tarballs are generated on demand and thus don't have reproducible checksums.

I think in this case it's safer to just leave the URL as a comment and add only a specific downloaded tarball as Source0.

We also need to complain to upstream about this, those "autogenerate tarball" features in repository browsers are not intended to serve as a substitute for actual releases!

To be more precise, that URL is a redirect, so it can't be used as a Source URL at all. The last component of the Source URL MUST match the file name on disk.

Checksums are same - these released files are not regenerated, only redirected. That's why asked my question. Maybe bringing it to fedora-devel is now the best we can do regarding this issue. So thanks Kevin, I think your proposed solution with comment is really best right now, but it's not solution as Source URLs are checked occasionaly.

Unfortunately, there are several upstream servers where the download URLs are unusable due to weird redirects like that. Our guidelines for source URLs suggest using a comment for the real URL in such cases. Of course this means they can't be automatically checked, but that's just life.

Apologies, the bugzilla mail fell into my oblivion of an inbox. I am confused in regards to what exactly I should be putting for Source0?

I currently have:
Source0:        http://download.github.com/mishaaq-kcm_touchpad-000be4c.tar.gz

As far as I can tell, this is a direct download (There is not a 301 or anything reported in wget)

Jreznik suggested http://github.com/mishaaq/kcm_touchpad/tarball/kcm_touchpad-0.3.0 which is a 302 to the above Source0

Which should I use to keep this review moving? The folder both of these tar.gz extract to is mishaaq-kcm_touchpad-000be4c leading to ridiculous %setup -q -n mishaaq-kcm_touchpad-000be4c in either case.

This situation in general sucks, and I'll ask upstream to clarify this in the future.

Again, sorry for the delay

If the first one is a direct download, then I think it's OK to let it there. The second one is better from name point of view - it's in %{name}-%{version} scheme. But it's redirect :( And it extracts to that not very nice directory... It would be nice to ask upstream for real and proper release going with usual release standards.

Other issues are solved, I think I can approve it. Kevin what do you think about letting current sources URL?

http://download.github.com/mishaaq-kcm_touchpad-000be4c.tar.gz is the correct Source URL to use.

Ok, thanks!

APPROVED

New Package CVS Request
=======================
Package Name: kcm_touchpad
Short Description: Synaptics driver based touchpads kcontrol module
Owners: rrix
Branches: F-10 F-11 F-12
InitialCC:

cvs done.

kcm_touchpad-0.3.0-5.fc12 has been submitted as an update for Fedora 12.
http://admin.fedoraproject.org/updates/kcm_touchpad-0.3.0-5.fc12

kcm_touchpad-0.3.0-5.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/kcm_touchpad-0.3.0-5.fc11

kcm_touchpad-0.3.0-5.fc12 has been pushed to the Fedora 12 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 kcm_touchpad'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F12/FEDORA-2009-11601

kcm_touchpad-0.3.0-5.fc11 has been pushed to the Fedora 11 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 kcm_touchpad'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F11/FEDORA-2009-11605

kcm_touchpad-0.3.0-5.fc11 has been pushed to the Fedora 11 stable repository.  If problems still persist, please make note of it in this bug report.

kcm_touchpad-0.3.0-5.fc12 has been pushed to the Fedora 12 stable repository.  If problems still persist, please make note of it in this bug report.