[Bug 234612] Review Request: Ice - The Internet Communications Engine (Object middleware)

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: Ice - The Internet Communications Engine (Object middleware)


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=234612





------- Additional Comments From mtasaka at ioa.s.u-tokyo.ac.jp  2007-08-23 14:23 EST -------
For -4:

* Redundant Requires
  - For -java-devel subpackage, "Requires: ice = %{version}-%{release}"
    is redundant because -java-devel requires -java subpackage
    and -java subpackage requires ice main package

* scriptlets
--------------------------------------------------------
getent group iceuser > /dev/null || groupadd -r iceuser
--------------------------------------------------------
  - Ah.. "groupadd" must be written in full path
    (%_sbindir/groupadd) otherwise this will cause problem
    when normal user do "su -c 'yum install'" or so.
    Please write groupadd, useradd and so on (i.e.
    commands which are not user normal users' path) in
    full path.
    I also asked Fedora-packaing to change wiki. 

* Directory ownership issue
  - Well, still for python subpackage this is not fixed.
    For example:
--------------------------------------------------------
[tasaka1 at localhost ~]$ LANG=C rpm -qf
/usr/lib/python2.5/site-packages/Ice/Glacier2/__init__.py
ice-python-3.2.1-4.fc8
[tasaka1 at localhost ~]$ LANG=C rpm -qf /usr/lib/python2.5/site-packages/Ice/Glacier2/
file /usr/lib/python2.5/site-packages/Ice/Glacier2 is not owned by any package
---------------------------------------------------------
    Please check the line
---------------------------------------------------------
%{python_sitelib}/Ice/*/*.py*
---------------------------------------------------------

    Note: When we write
----------------------------------------------------------
%files
foo/
----------------------------------------------------------
    (here foo/ is a directory), this contains the directory foo
    itself and all files/directories/etc under foo/, where
    when we write
-----------------------------------------------------------
%files
%dir foo/
------------------------------------------------------------
    this contains only foo/ itself.

    So, for example
------------------------------------------------------------
%files ruby
%defattr(644,root,root,755)
%dir %{ruby_sitearch}/Ice
%{ruby_sitearch}/Ice/*
------------------------------------------------------------
    is the same as
------------------------------------------------------------
%files ruby
%defattr(644,root,root,755)
%{ruby_sitearch}/Ice/
------------------------------------------------------------
    (it is my habit to add slash at last when the entry is
     a directory)

? /var/lib/icegrid
  - Well actually I don't know how to use ice at all, however
    anyway I tried:
-------------------------------------------------------------
[root at localhost ~]# service icegridregistry start
Starting icegridregistry: runuser: warning: cannot change directory to
/var/lib/icegrid: No such file or directory
/usr/bin/icegridregistry: failure occurred in daemon
                                                           [FAILED]
-------------------------------------------------------------
    While I don't think the failure is the issue (perhaps some
    configuration is needed beforehand), does this mean that
    %_localstatedir/lib/icegrid must be owned by some package?


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.