Bug 234330 - Review Request: ruby-cairo - ruby bindings for cairo
Review Request: ruby-cairo - ruby bindings for cairo
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
All Linux
medium Severity medium
: ---
: ---
Assigned To: Mamoru TASAKA
Fedora Package Reviews List
Depends On:
Blocks: 232160
  Show dependency treegraph
Reported: 2007-03-28 10:58 EDT by Allisson Azevedo
Modified: 2007-11-30 17:12 EST (History)
0 users

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2007-03-29 22:39:42 EDT
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)

  None (edit)
Description Allisson Azevedo 2007-03-28 10:58:37 EDT
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 12:21:38 EDT
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 03:08:22 EDT

   This package (ruby-cairo) is APPROVED by me.
Comment 4 Allisson Azevedo 2007-03-29 06:36:28 EDT
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.