Bug 646181 - Review Request: rubygem-gio2 - Ruby binding of gio-2.0.x
Summary: Review Request: rubygem-gio2 - Ruby binding of gio-2.0.x
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Michael Stahnke
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2010-10-24 19:52 UTC by Mamoru TASAKA
Modified: 2010-10-29 16:58 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2010-10-29 16:58:19 UTC
Type: ---
Embargoed:
mastahnke: fedora-review+
petersen: fedora-cvs+


Attachments (Terms of Use)

Comment 1 Michael Stahnke 2010-10-26 21:29:22 UTC
stahnma@olive /var/lib/mock/fedora-rawhide-i386/result> rpmlint *rpm
rubygem-gio2.i686: W: spelling-error Summary(en_US) gio -> goo, gi, go
rubygem-gio2.i686: W: spelling-error %description -l en_US gio -> goo, gi, go
rubygem-gio2.src: W: spelling-error Summary(en_US) gio -> goo, gi, go
rubygem-gio2.src: W: spelling-error %description -l en_US gio -> goo, gi, go
>> rubygem-gio2.src:96: W: macro-in-comment %{geminstdir}
>> rubygem-gio2.src:96: W: macro-in-comment %{ruby_sitearch}
rubygem-gio2.src: W: no-buildroot-tag
rubygem-gio2-devel.i686: W: no-documentation
rubygem-gio2-doc.i686: W: no-documentation
rubygem-gio2-doc.i686: W: unexpanded-macro /usr/lib/ruby/gems/1.8/doc/gio2-0.90.4/ri/GLib/InputStream/pending%3d-i.yaml %3d
rubygem-gio2-doc.i686: W: unexpanded-macro /usr/lib/ruby/gems/1.8/doc/gio2-0.90.4/ri/GLib/File/eql%3f-i.yaml %3f
rubygem-gio2-doc.i686: W: unexpanded-macro /usr/lib/ruby/gems/1.8/doc/gio2-0.90.4/ri/GLib/DesktopAppInfo/desktop_env%3d-c.yaml %3d
rubygem-gio2-doc.i686: W: unexpanded-macro /usr/lib/ruby/gems/1.8/doc/gio2-0.90.4/ri/GLib/FileInfo/directory%3f-i.yaml %3f
rubygem-gio2-doc.i686: W: unexpanded-macro /usr/lib/ruby/gems/1.8/doc/gio2-0.90.4/ri/GLib/Resolver/default%3d-c.yaml %3d
rubygem-gio2-doc.i686: W: unexpanded-macro /usr/lib/ruby/gems/1.8/doc/gio2-0.90.4/ri/GLib/Icon/eql%3f-i.yaml %3f
ruby-gio2.i686: W: no-documentation
>> ruby-gio2.i686: W: dangling-relative-symlink /usr/lib/ruby/site_ruby/1.8/gio2.rb ../../gems/1.8/gems/gio2-0.90.4/lib/gio2.rb
6 packages and 0 specfiles checked; 0 errors, 17 warnings.

The only things to look into are noted with ">>".  

* Omitted most not applicable guidelines
Package Name +
License +
Spec file in English and readable + 
Source MD5 matches upstream +
Builds in mock and koji +
No bundles of lib + 
Devel package requires base package +
Doc package usage and will allow running without doc +
PC files in devel package +
* Ruby Specific
Ruby ABI + 
Use of sitelib and sitearch +
Gem setup +


MUST: Every binary RPM package (or subpackage) which stores shared library files (not just symlinks) in any of the dynamic linker's default paths, must call ldconfig in %post and %postun.

It appears to me that you install /usr/lib/ruby/site_ruby/1.8/i386-linux/gio2.so

My only caveat is that I am unsure if the path is a default linker's path.  I don't think it is. 


The package looks good.  Do you have any good examples of using it?  I was unable to test it's usage.  

Package rubygem-gio2 APPROVED.

Comment 2 Mamoru TASAKA 2010-10-27 18:48:59 UTC
Thank you for review!

- I will fix up rpmlint issue
- dangling warning is harmless here (symlink is surely resolved)
- calling /sbin/ldconfig is not needed here.

New Package SCM Request
=======================
Package Name:       rubygem-gio2
Short Description:  Ruby binding of gio-2.0.x
Owners:             mtasaka
Branches:           f13 f14

Comment 3 Jens Petersen 2010-10-28 09:27:17 UTC
Git done (by process-git-requests).

Comment 4 Mamoru TASAKA 2010-10-29 16:58:19 UTC
Closing this one, thank you.


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