Bug 254019 - Review Request: rubygem-daemons - A toolkit to create and control ruby daemons
Summary: Review Request: rubygem-daemons - A toolkit to create and control ruby daemons
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: David Lutterkort
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 254015
TreeView+ depends on / blocked
 
Reported: 2007-08-23 17:21 UTC by Scott Seago
Modified: 2013-04-30 23:40 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2007-11-01 03:52:22 UTC
Type: ---
Embargoed:
lutter: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Scott Seago 2007-08-23 17:21:40 UTC
Spec URL: http://people.redhat.com/sseago/fedora/spec/rubygem-daemons.spec
SRPM URL: http://people.redhat.com/sseago/fedora/SRPMS/rubygem-daemons-1.0.7-1.fc7.src.rpm
Description: 
Daemons provides an easy way to wrap existing ruby scripts (for example a
self-written server)  to be run as a daemon and to be controlled by simple
start/stop/restart commands.  You can also call blocks as daemons and control
them from the parent or just daemonize the current process.  Besides this
basic functionality, daemons offers many advanced features like exception 
backtracing and logging (in case your ruby script crashes) and monitoring and
automatic restarting of your processes if they crash.

Comment 1 David Lutterkort 2007-08-24 01:27:42 UTC
OK - Package name
OK - Specfile name
OK - Specfile is legible
OK - No prebuilt binaries included
OK - BuildRoot value (one of the recommended values)
OK - PreReq not used
OK - Source md5sum matches upstream
OK - No hardcoded pathnames
OK - Package owns all the files it installs
OK - Package requires create needed unowned directories
OK - Package builds successfully on i386 and x86_64 (mock)
OK - BuildRequires sufficient
OK - File permissions set properly
OK - Macro usage is consistent
OK - rpmlint is silent

OK - Package is named rubygem-%{gemname}
OK - Source points to full URL of gem
OK - Package version identical with gem version
OK - Package Requires and BuildRequires rubygems
OK - Package provides rubygem(%{gemname}) = %version
OK - Package requires gem dependencies correctly
OK - %prep and %build are empty
OK - %gemdir defined properly, and gem installed into it
OK - Package owns its directories under %gemdir
OK - No arch-specific content in %{gemdir}
OK - Package is noarch

FIX - Mark the LICENSE file as %doc

FIX - License info is not accurate
    The license seems to be BSD, but not quite. Needs to be clarified


Comment 2 David Lutterkort 2007-08-24 01:31:54 UTC
Contrary to what I said above, rpmlint is not silent:

E: rubygem-daemons non-executable-script
/usr/lib/ruby/gems/1.8/gems/daemons-1.0.7/examples/run/myserver.rb 0644

Since it's just an example, nothing to worry about.

Comment 3 David Lutterkort 2007-08-24 02:00:47 UTC
Actually, the license for most of the code is MIT; for daemonize.rb it's (GPLv2+
or Ruby) ... the overall license tag therefore is 'MIT and (GPLv2+ or Ruby)' and
daemonize.rb's license should be pointed out according to
http://fedoraproject.org/wiki/Packaging/LicensingGuidelines?highlight=%28Packaging%29#head-5dcaa7704b32aabaddc2e709f328f48eea6c91de



Comment 4 Scott Seago 2007-08-24 17:58:13 UTC
License and spec issues sorted out, SRPM is:
 http://people.redhat.com/sseago/fedora/SRPMS/rubygem-daemons-1.0.7-2.fc7.src.rpm

Comment 5 David Lutterkort 2007-08-27 18:39:49 UTC
Excellent.

APPROVED

Please follow http://fedoraproject.org/wiki/CVSAdminProcedure and import
the package. Close this bug as RAWHIDE once it's been successfully imported
and built.


Comment 6 Scott Seago 2007-08-27 21:36:17 UTC
New Package CVS Request
=======================
Package Name: rubygem-daemons
Short Description: A toolkit to create and control ruby daemons
Owners: sseago
Branches: F-7, EL-5
InitialCC: 
Cvsextras Commits: No

Comment 7 Kevin Fenzi 2007-08-28 04:41:42 UTC
cvs done.

Comment 8 Jeroen van Meeuwen 2008-10-16 23:02:05 UTC
Package Change Request
======================
Package Name: rubygem-daemons
New Branches: EL-4
Owners: sseago, kanarip

Approved by sseago in private email

Comment 9 Kevin Fenzi 2008-10-19 23:03:27 UTC
cvs done.


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