Bug 234330 - Review Request: ruby-cairo - ruby bindings for cairo
Summary: Review Request: ruby-cairo - ruby bindings for cairo
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review   
(Show other bugs)
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Mamoru TASAKA
QA Contact: Fedora Package Reviews List
Depends On:
Blocks: 232160
TreeView+ depends on / blocked
Reported: 2007-03-28 14:58 UTC by Allisson Azevedo
Modified: 2007-11-30 22:12 UTC (History)
0 users

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2007-03-30 02:39:42 UTC
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
mtasaka: fedora-review+
jwboyer: fedora-cvs+

Attachments (Terms of Use)

Description Allisson Azevedo 2007-03-28 14:58:37 UTC
Spec URL: http://fedora.allisson.eti.br/ruby-cairo/ruby-cairo.spec
SRPM URL: http://fedora.allisson.eti.br/ruby-cairo/ruby-cairo-1.4.1-1.src.rpm

Description: Ruby bindings for cairo. Cairo is a 2D graphics library with
support for multiple output devices. Currently supported output targets include the 
X Window System, win32, and image buffers.

Comment 1 Mamoru TASAKA 2007-03-28 16:21:38 UTC
For 1.4.1-1:

* License
  - Well as Ruby uses "Ruby License/GPL", I would prefer
    to use it.

* Group
  - "Development/Languages" is for -devel package.
    For libraries, this should be "System Environment/Libraries"

* Release number specific dependency
  - Usually, provides or Requires for main package should be
    release number specific.

* Ownership
  - The following directories are not owned by any packages.
    When you just write
    (where foo is a directory), this means the directory foo itself
    and all files/directories/etc under foo directory.

    While if you write
%dir foo/
   This only contains the directory foo itself.

* Timestamps
  - Again keep timestamps on non-generated files.
    For this package, the following works.
make install DESTDIR=$RPM_BUILD_ROOT INSTALL="%{__install} -c -p"

* %check
  - Empty %check is not needed and please remove this.

* Requires
   - %{ruby_sitearch}/rb_cairo.h contains the line:
    39  #include "ruby.h"
     This means that -devel package needs "Requires: ruby-devel".

- When you update spec file, please describe briefly how
  you changed it in %changelog entry, not just write "Fix .spec".

Comment 3 Mamoru TASAKA 2007-03-29 07:08:22 UTC

   This package (ruby-cairo) is APPROVED by me.

Comment 4 Allisson Azevedo 2007-03-29 10:36:28 UTC
New Package CVS Request
Package Name: ruby-cairo
Short Description: ruby bindings for cairo
Owners: allisson@gmail.com
Branches: FC-6

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