Red Hat Bugzilla – Bug 202033
Review Request: deltarpm
Last modified: 2007-12-29 16:29:16 EST
Spec URL: http://people.redhat.com/misa/rpms/deltarpm/deltarpm.spec SRPM URL: http://people.redhat.com/misa/rpms/deltarpm/deltarpm-3.3-2.src.rpm Description: Create deltas between rpms. A deltarpm contains the difference between an old and a new version of a rpm, which makes it possible to recreate the new rpm from the deltarpm and the old one. You don't have to have a copy of the old rpm, deltarpms can also work with installed rpms.
Good: + Tar ball matched with upstream. + Local Build works fine. + Local install works fine. + Local uninstall works fine. + License is OK. + Mock build works fine. Bad: - BuildRoot should be %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n) - Rpmlint on the source rpm complaints: W: deltarpm non-standard-group Applications/Utilitie s W: deltarpm setup-not-quiet - Rpmlint complaints on binary package: W: deltarpm non-standard-group Applications/Utilities
Do you have any good suggestion on a good group for this to live in? I can see it in: Development/Tools Applications/Archiving (well, maybe not, rpms are not generally for archives) Aplications/System
Even non-standard yum plugins are in System Environment/Base, so I guess this goes there as well.
Suggested changes were incorporated, please have a look (spec file path is the same). http://people.redhat.com/misa/rpms/deltarpm/deltarpm-3.3-3.src.rpm
You don't need to specify gcc as a BR. So, If you may remove this BR you are APPROVE.
Removed, in deltarpm 3.3-4 (I didn't like 3.3-3 anyway)
Package Change Request ====================== Package Name: deltarpm Updated Fedora Owners: jdieter@gmail.com As current owner, I'm happy to see this go to someone who's actively working on it instead.
cvs done.
Package Change Request ====================== Package Name: deltarpm New Branches: EL-4 EL-5 See https://bugzilla.redhat.com/show_bug.cgi?id=426959 Thanks. If I'm supposed to do this elsewhere, let me know.
You forgot to reset the fedora-cvs flag, but I noticed this request on the review list. ;) cvs done.