Bug 426149

Yay! I was looking forward to see this package in Fedora!

 + Package is named according to the Package Naming Guidelines.
 + Spec file name must match the base package %{name}, in the format %{name}.spec.
 + Package meets the Packaging Guidelines.
 + Package successfully compiles and builds into binary rpms on at least one
supported architecture.
     Tested on: Fedora rawhide, Fedora 8
 + Buildroot is correct
(%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n))
 + Package is licensed with an open-source compatible license and meets other
legal requirements as defined in the legal section of Packaging Guidelines.
 + License field in the package spec file matches the actual license.
     License type: BSD
 + Spec file is legible and written in American English.
 + Package is not known to require ExcludeArch
 + Package must own all directories that it creates.
 + Package does not contain duplicates in %files.
 + Permissions on files are set properly.
 + Package has a %clean section, which contains rm -rf of the buildroot
 + Package consistently uses macros.
 + File based requires are sane.
 - Rpmlint output is not sane.

    sshmenu.src:59: E: hardcoded-library-path in
%{_prefix}/lib/bonobo/servers/sshmenu-applet.server
    sshmenu.src:60: E: hardcoded-library-path in %{_prefix}/lib/ruby/*/*.rb

Not sure if this is an issue, can you check it?

I've tried it on Fedora 8 and it's working correctly.

Those "hardcoded library paths" should be harmless. It's because this is a
noarch package, and those files need to be in /usr/lib even on 64bit
architectures (not in /usr/lib64, as it would mean that the noarch package built
on a 64bit machine wouldn't work on a 32bit one).
I've tested my builds on an x86_64, and the bonobo server file is picked up fine
(see the comment in the spec about it). Maybe the ruby files could use some
macro of some kind, that I don't know for sure, but it should be fine like this
nevertheless.

I understand Matt, Thank you!


APPROVED.

ping?

I've a comment about the name of the package.
I think the name should be follow other packages names of gnome applet tools.
so gnome-applet-sshmenu, should be more appropriate.

i'm talkin' about the summary of the bug as well.

- Well, ruby script should not be installed under rubylibdir, it should
  be moved to sitelibdir.
  rubylibdir should be used by ruby scripts in ruby rpm (and its subrpms,
  like ruby-libs, ruby-tcltk,...)

  Also, for refering to the directories related to ruby, please
  refer to
  http://fedoraproject.org/wiki/Packaging/Ruby

- Also, as the directory where ruby script is installed is version
  specific, this package should have "Requires: ruby(abi) = 1.8".

Thanks for these comments! I had never packaged a ruby application for Fedora
before, and completely missed the ruby specific packaging Wiki page. I'll fix
this ASAP.

About the package name, I'm actually all for "gnome-applet-sshmenu", but the
thing is that this program doesn't necessarily require a GNOME panel to run, as
it includes both a simple program, and the gnome applet. Maybe it would be
possible to split it into "sshmenu" which would be the simple non-applet
application (still possibly requiring some ruby-gnome stuff), and
"gnome-applet-sshmenu" which would require "sshmenu" but also provide the
panel/bonobo/etc. stuff which is GNOME applet specific.

Further thoughts?

I've updated the package to 3.15-2.
http://thias.fedorapeople.org/review/sshmenu/
* Mon Jan 14 2008 Matthias Saou <http://freshrpms.net/> 3.15-2
- Follow ruby guidelines : Put ruby files in ruby_sitelib.
- Add required hardcoded ruby abi version requirement.

I'll wait for further comments regarding the name change and/or applet split.

* Wed Jan 16 2008 Matthias Saou <http://freshrpms.net/> 3.15-4
- Keep only ruby(gtk2) req in sshmenu, move others to gnome-applet-sshmenu.

* Mon Jan 14 2008 Matthias Saou <http://freshrpms.net/> 3.15-3
- Split out the GNOME applet as gnome-applet-sshmenu while keeping the basic
  menu application as sshmenu, and have all the shared ruby files be there.

I've tested sshmenu "alone", it works fine, then the full gnome applet is in its
own "gnome-applet-sshmenu" package. The main advantage is that the basic
"sshmenu" package only requires "ruby(gtk2)", while the other requires it plus
"ruby(gconf2)" and "ruby(panelapplet2)".

I'll wait a little while in case there are further comments, but as I'm pretty
satisfied by the current solution, I'll then proceed to importing and building
the packages :-)

(In reply to comment #11)
> I've tested sshmenu "alone", it works fine, then the full gnome applet is in its
> own "gnome-applet-sshmenu" package. The main advantage is that the basic
> "sshmenu" package only requires "ruby(gtk2)", while the other requires it plus
> "ruby(gconf2)" and "ruby(panelapplet2)".

Then I think %ruby_sitelib/gnome-sshmenu.rb should be moved to
gnome-applet-sshmenu rpm as this script is not used by sshmenu and
this 'require's 'gconf2'.

Good catch!

* Thu Jan 17 2008 Matthias Saou <http://freshrpms.net/> 3.15-5
- Move gnome-sshmenu.rb to gnome-applet-sshmenu too.

Okay for me.

New Package CVS Request
=======================
Package Name: sshmenu
Short Description: Application to organize SSH connection information in a menu
Owners: thias
Branches: F-7 F-8 EL-5
InitialCC:
Cvsextras Commits: yes

cvs done.

Thanks to Marek, Xavier and Mamoru for the review, and Kevin for the CVS request!
I've imported and built it on all branches.