Bug 984750 - Review Request: lbzip2-utils - Utilities for working with bzip2 compressed files
Summary: Review Request: lbzip2-utils - Utilities for working with bzip2 compressed files
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Christopher Meng
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-07-15 21:25 UTC by Mikolaj Izdebski
Modified: 2013-07-22 11:35 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-07-16 12:36:54 UTC
Type: ---
Embargoed:
i: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Mikolaj Izdebski 2013-07-15 21:25:32 UTC
Spec URL: http://lbzip2.org/lbzip2-utils.spec
SRPM URL: http://lbzip2.org/lbzip2-utils-1.0-1.fc20.src.rpm
Description: This package provides a collection of utility programs to work with compressed files in bz2 format.  These utilities are supplementary to lbzip2 compression tool.
Fedora Account System Username: mizdebsk

Comment 1 Christopher Meng 2013-07-16 00:17:03 UTC
I'll take this.

Comment 2 Christopher Meng 2013-07-16 00:40:10 UTC
Let's first look into the spec itself.

Issue:

1. Please drop BuildRoot tag as modern RPM has already supported it.

2. Please drop rm -rf %{buildroot} in the %install section, same reason as above.

3. Please remove whole %clean section, same reason as #1.

4. Please remove %defattr(-,root,root,-), same reason...

Comment 3 Mikolaj Izdebski 2013-07-16 08:24:41 UTC
I'm sorry, I forgot to state that in the initial request, but I am going to submit this package for EPEL 5. All of the things mentioned in comment #2 are needed there.

If you insist I can remove them and for EPEL 5 use a different spec file, but I would rather keep the same spec across all branches.

Comment 4 Christopher Meng 2013-07-16 08:50:01 UTC
OK. I'll fill a bug when EL5 is terminated...

I've checked everything, fedora-review and manual, no problems.

APPROVED.

Comment 5 Mikolaj Izdebski 2013-07-16 09:18:43 UTC
Thank you.

New Package SCM Request
=======================
Package Name: lbzip2-utils
Short Description: Utilities for working with bzip2 compressed files
Owners: mizdebsk
Branches: f19 el5 el6

Comment 6 Gwyn Ciesla 2013-07-16 12:13:31 UTC
Git done (by process-git-requests).

Comment 7 Michael Schwendt 2013-07-22 11:29:43 UTC
> but I am going to submit this package for EPEL 5.
> All of the things mentioned in comment #2 are needed there.

RPM in EL5 is new enough, so you can drop %defattr.

Comment 8 Mikolaj Izdebski 2013-07-22 11:35:24 UTC
(In reply to Michael Schwendt from comment #7)
> > but I am going to submit this package for EPEL 5.
> > All of the things mentioned in comment #2 are needed there.
> 
> RPM in EL5 is new enough, so you can drop %defattr.

I didn't not know that.  I'll keep that in mind,  Thank you.


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