Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.

Bug 205504

Summary: Review Request: rum - rug-like interface for yum
Product: [Fedora] Fedora Reporter: James Willcox <snorp>
Component: Package ReviewAssignee: Ray Strode [halfline] <rstrode>
Status: CLOSED WONTFIX QA Contact: Fedora Package Reviews List <fedora-package-review>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: sundaram, tjb
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2007-03-13 14:47:03 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 163779    

Description James Willcox 2006-09-06 19:31:48 UTC
Spec URL: http://www.snorp.net/files/rum/rum.spec
SRPM URL: http://www.snorp.net/files/rum/rum-1.0.0-1.src.rpm
Description: rum is a rug-like interface for yum

Comment 1 James Willcox 2006-09-06 19:36:11 UTC
Oops, I should mention that this is my first package and I would need a sponsor,
etc.

Comment 2 Ray Strode [halfline] 2006-09-06 20:01:32 UTC
So at first glance, the package spec file looks great.

Did you run it through rpmlint?  If so, any spew?

Comment 3 James Willcox 2006-09-06 20:11:56 UTC
Yeah, it gives me this:

W: rum summary-not-capitalized rum is a rug-like interface for yum
E: rum non-executable-script /usr/share/rum/rummain.py 0644
E: rum non-executable-script /usr/share/rum/i18n.py 0644
E: rum zero-length /usr/share/doc/rum-1.0.0/ChangeLog

All of that seems fairly benign to me, but what do I know :)

Comment 4 Ray Strode [halfline] 2006-09-06 20:24:49 UTC
cool. And the srpm builds okay in mock? If so, seems fine.

Comment 5 Jason Tibbitts 2006-09-06 20:29:24 UTC
In general you shouldn't package zero length files.

Also, the description is somewhat lacking.  Even if it's supposed that I know
what "yum" is, I have little chance of knowing what "rug" might be.

Comment 6 James Willcox 2006-09-06 20:51:37 UTC
Ok, I changed the description to:

rum is a rug-like interface for yum (Yellowdog Updater Modified).  rug is the
command-line frontend for rcd (Red Carpet Daemon).

And removed the empty ChangeLog

Comment 7 James Willcox 2006-09-06 20:53:38 UTC
Bah, I meant:

rum is a rug-like interface for yum (Yellow dog Updater, Modified).  rug is the
command-line frontend for rcd (Red Carpet Daemon).

Comment 8 Ray Strode [halfline] 2006-09-07 19:34:34 UTC
Let me know when you've created an account, sent in the CLA, etc, and I'll move
you over to sponsored status.

Comment 9 Brian Pepple 2006-09-07 20:19:19 UTC
Quick note:  the Source should be a full url.


Comment 10 Chitlesh GOORAH 2006-09-08 09:34:42 UTC
** I've seen you have updated the src.rpm
Next time, do update the release version as well.

** 
%defattr(-,root,root,-)
%defattr(-,root,root)

Duplicates

**
chitlesh(rum-1.0.0-1)[0]$rpmlint /home/chitlesh/rpmbuild/SRPMS/rum-1.0.0-1.src.rpm
W: rum summary-not-capitalized rum is a rug-like interface for yum

Capitalized the "R" of "rum"

E: rum description-line-too-long rum is a rug-like interface for yum (Yellow dog
Updater, Modified).  rug is the command-line frontend for rcd (Red Carpet Daemon).

Divide it into 2 lines

** 
 /usr/bin/install -c -m 644 'i18n.py'
'/var/tmp/rum-1.0.0-1-root-chitlesh/usr/share/rum/i18n.py'
 /usr/bin/install -c -m 644 'rumcommand.py'
'/var/tmp/rum-1.0.0-1-root-chitlesh/usr/share/rum/rumcommand.py'
 /usr/bin/install -c -m 644 'rummain.py'
'/var/tmp/rum-1.0.0-1-root-chitlesh/usr/share/rum/rummain.py'                       

preserves the files' timestamps
make INSTALL="install -p" install DESTDIR=$RPM_BUILD_ROOT

** Like Brian said : the Source should be a full url.

Comment 11 Rahul Sundaram 2007-03-13 14:40:06 UTC
James Willcox,

Are you interested in this package anymore? Its been stalled for a long time now.

Comment 12 James Willcox 2007-03-13 14:47:03 UTC
No, we should close it.  I haven't taken the time to get the CLA approved by my
employer, and I haven't worked on rum in quite a while.