Bug 234488

This is my first package, so I need a sponsor.

%{_datadir}/presto/* should be %{_datadir}/presto thus presto is known as the
owner of this directory.

Fixed in 0.3.2-2

Spec URL: http://www.lesbg.com/jdieter/presto/yum-presto.spec
SRPM URL: http://www.lesbg.com/jdieter/presto/yum-presto-0.3.2-2.src.rpm

New release: 0.3.3-1
Spec URL: http://www.lesbg.com/jdieter/presto/yum-presto.spec
SRPM URL: http://www.lesbg.com/jdieter/presto/yum-presto-0.3.3-1.src.rpm

rpmlint Output:

[tim@naboo ~]$ rpmlint yum-presto-0.3.3-1.noarch.rpm 
E: yum-presto only-non-binary-in-usr-lib
[tim@naboo ~]$ rpmlint yum-presto-0.3.3-1.src.rpm 
W: yum-presto macro-in-%changelog _datadir
   - Change %{_datadir} to %%{_datadir} in Chagelog entry
W: yum-presto no-%build-section
   - Not a blocker, but a empty %build section coulf be added to silence rpmlint

MUST:
* package is named appropriately
* it is legal for Fedora to distribute this
* license field matches the actual license.
* license is open source-compatible.
* specfile name matches %{name}
* verify source and patches (md5sum matches upstream, know what the patches do)
* summary and description fine
* correct buildroot
* %{?dist} is used
* license text included in package and marked with %doc
* package meets FHS (http://www.pathname.com/fhs/)
X changelog format fine (see earlier bug comments)
* Packager tag not used
* Vendor tag not used
* Distribution tag not used
* License used and not Copyright 
* Summary tag does not end in a period
* if possible, replace PreReq with Requires(pre) and/or Requires(
* Source match upstream
  $ md5sum yum-presto-0.3.3.tar.bz2 
  fd1984365bdbe61aca8114ab47eccefa  yum-presto-0.3.3.tar.bz2
  $ md5sum rpmbuild/SOURCES/yum-presto-0.3.3.tar.bz2
  fd1984365bdbe61aca8114ab47eccefa  rpmbuild/SOURCES/yum-presto-0.3.3.tar.bz2
* specfile is legible
* package successfully compiles and builds on at least x86
* BuildRequires are proper
* make sure lines are <= 80 characters
* specfile written in American English
* no -doc sub-package necessary
* no libraries
* no rpath
* config files uses %config(noreplace)
* not a GUI app
* no -devel sub-package necessary
* macros used appropriately and consistently
* no %makeinstall
* install section must begin with rm -rf $RPM_BUILD_ROOT or %{buildroot}
* no locale data
* no cp usage so no need to worry about -p
* split Requires(pre,post) into two separate lines
* package not relocatable
* package contains code
* package owns all directories and files
* no %files duplicates
* file permissions fine
? %defattrs present
 - should it be %defattr(-, root, root, -)
* %clean present
* %doc files do not affect runtime
* not a web app
* verify the final provides and requires of the binary RPMs
  $ rpm -q --requires -p rpmbuild/RPMS/noarch/yum-presto-0.3.3-1.noarch.rpm 
  config(yum-presto) = 0.3.3-1
  deltarpm >= 3.4
  python >= 2.4
  rpmlib(CompressedFileNames) <= 3.0.4-1
  rpmlib(PayloadFilesHavePrefix) <= 4.0-1
  yum >= 3.0


  $  rpm -q --provides -p rpmbuild/RPMS/noarch/yum-presto-0.3.3-1.noarch.rpm 
  config(yum-presto) = 0.3.3-1
  yum-presto = 0.3.3-1

* run rpmlint on the binary RPMs
 - see previous bug comments

SHOULD:
* package should include license text in the package and mark it with %doc
* package should build on i386
? package should build in mock
 - I haven't tried, but I don't think it'll be a problem

SUMMARY:
E: yum-presto only-non-binary-in-usr-lib
This is fine because yum plugins is stored in /usr/lib/yum-plugins.

W: yum-presto macro-in-%changelog _datadir
Change %{_datadir} to %%{_datadir} in Changelog entry

use %defattr(-, root, root, -)

Fix these issues and i will approve it, i'm not a sponsor, so we need somebody
to sponsor you.
 

New release: 0.3.4-1
Spec URL: http://www.lesbg.com/jdieter/presto/yum-presto.spec
SRPM URL: http://www.lesbg.com/jdieter/presto/yum-presto-0.3.4-1.src.rpm

This fixes all the mistakes you pointed out, Tim (including the %build one). 
Thanks much for the review!

The spec link is wrong, it point to 0.3.3 spec file, not the 0.3.4 one

$ rpmlint rpmbuild/RPMS/noarch/yum-presto-0.3.4-1.fc7.noarch.rpm 
E: yum-presto only-non-binary-in-usr-lib
$ rpmlint Download/yum-presto-0.3.4-1.src.rpm 
silent. 

The spec file in the SRPM is ok.

APPROVED.

Thanks!  The spec file has been changed on the server, but it may take a refresh
to come through.

Is there anything I should be doing to get a sponsor's attention, or should I
just wait patiently?

You should just wait for a sponsor to pop up. :-)

Sweet!  Thanks for the review!  I'll just wait and hope someone pops up.

Hey Jonathan. I'd be happy to sponsor you. 

Thanks for the nice review here Tim. 

I do see a build failure on x86_64 however... 

Processing files: yum-presto-0.3.4-1.fc7
error: File not found by glob:
/var/tmp/yum-presto-0.3.4-1.fc7-root-mockbuild/usr/lib64/yum-plugins/presto.py*

Looks like: 
%{_libdir}/yum-plugins/presto.py*

needs to be: 

/usr/lib/yum-plugins/presto.py*

Can you make that change? I have a x86_64 test box if you need one. 

(In reply to comment #14)
> Hey Jonathan. I'd be happy to sponsor you. 
> 
> Thanks for the nice review here Tim. 
> 
> I do see a build failure on x86_64 however... 
> 
> Processing files: yum-presto-0.3.4-1.fc7
> error: File not found by glob:
> /var/tmp/yum-presto-0.3.4-1.fc7-root-mockbuild/usr/lib64/yum-plugins/presto.py*
> 
> Looks like: 
> %{_libdir}/yum-plugins/presto.py*
> 
> needs to be: 
> 
> /usr/lib/yum-plugins/presto.py*
> 
> Can you make that change? I have a x86_64 test box if you need one. 
> 

It's right, it need to be '/usr/lib/yum-plugins/presto.py*', because yum look
for plugins in /usr/lib/yum-plugins/ for all archs.

Thanks for pointing that out!  I really should have tested this on my wife's
x86_64 system.

A note though: I haven't mirrored FC6 x86_64 updates or extras yet, so you won't
get any deltarpms through it yet.

New release: 0.3.5-1
Spec URL: http://www.lesbg.com/jdieter/presto/yum-presto.spec
SRPM URL: http://www.lesbg.com/jdieter/presto/yum-presto-0.3.5-1.src.rpm

Yeah, that looks good now here on x86_64... 

Are you looking for mirror space/BW for the x86_64 rpms? 
Or just haven't had time to generate them yet? Feel free to drop me an email 
about mirror space, I might be able to scare some up depending on what you need. 

Please continue the process from: 
http://fedoraproject.org/wiki/PackageMaintainers/Join#head-a601c13b0950a89568deafa65f505b4b58ee869b

Feel free to email me if you have any questions or problems, or find me on irc: 
nirik on #fedora-devel on irc.freenode.net. 

NOTE: (Just in case if it wasn't obvious or clear)  When/if imported into
Extras, the default config(s) must not point to repos/servers outside of the
(usual) fedoraproject infrastructure.  

Thank you for pointing that out.  I hadn't thought about it, but I guess it
should have been obvious to me.

When/if we get presto into extras, I don't think we'll point it at any
repositories.  Rather, it will be up to the repository owners (or the user) to
add the deltaurl line to their .repo files.

Thanks for the heads up.

Totally agreed. 

Jonathan: Can you make sure the version you check in has a commented presto.conf? 
ie, so it doesn't enable anything by default and the user must enable it, or
wait until it's enabled in their repo files?

It shouldn't be hard to comment the conf file.  The plugin will be enabled, but
there won't be any presto repositories in the conf file (as there are now).  If
there aren't any presto repositories in the conf file and none in the user's
.repo files, then it's just as if the plugin is disabled.

I had never originally planned on having the conf file contain repository
information at all.  It just turned out to be an easy feature to add and one
that was invaluable during testing.

I'm hoping that, as interest in yum-presto grows, Fedora Infrastructure will
start hosting deltarpms and modify their .repo files accordingly (the deltaurl
option will be ignored by yum if yum-presto isn't installed).

So, to reiterate, the conf file will not point to *any* repositories.  If
there's anything else you would like me to do with this, let me know.

Jonathan

New Package CVS Request
=======================
Package Name: yum-presto
Short Description: Deltarpm plugin for yum
Owners: jdieter
Branches: FC-6
InitialCC: 

CVS is approved, please wait about 10 minutes before checking in.