[Bug 497441] Review Request: mumble - Voice chat application

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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


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





--- Comment #62 from Mamoru Tasaka <mtasaka@xxxxxxxxxxxxxxxxxxx>  2009-05-16 14:10:56 EDT ---
Created an attachment (id=344291)
 --> (https://bugzilla.redhat.com/attachment.cgi?id=344291)
Patch to compile on dist-f10-updates-candidate

For -10:

* BR
  - BR: qt3-sqlite should not be needed. This uses Qt4, not
    Qt3.

* About mumble-server user directory
  - Well, -d %{buildroot}%{_datadir}/%{name}/ is definitely wrong,
    this directory should be used only when building this package.
    I would suggest -d %{_localstatedir}/lib/mumble-server/

* Requires
  - Needed Requires(pre) or so are still missing for murmur subpackage:
   
https://fedoraproject.org/wiki/Packaging/SysVInitScript#Initscripts_in_spec_file_scriptlets

* Build failure
  - The attached patch will make this package compile on
    F-11/F-10 (F-12 not checked)
    http://koji.fedoraproject.org/koji/taskinfo?taskID=1358131
    http://koji.fedoraproject.org/koji/taskinfo?taskID=1358127

* Devel files in main package
----------------------------------------------------------------------
ln -s %{_libdir}/libmumble.so.%{version} %{buildroot}%{_libdir}/libmumble.so
----------------------------------------------------------------------
  - I guess %_libdir/libmumble.so is not needed.
    Note that rpmlint warns about this:
----------------------------------------------------------------------
mumble.i586: W: devel-file-in-non-devel-package /usr/lib/libmumble.so
----------------------------------------------------------------------

* Service enabled by default
----------------------------------------------------------------------
murmur.i586: W: service-default-enabled /etc/rc.d/init.d/murmur
----------------------------------------------------------------------
  - Installing murmur package enables "murmur" service by default
    on level 345:
----------------------------------------------------------------------
# chkconfig --list murmur
murmur   0:off 1:off 2:off 3:on 4:on 5:on 6:off
----------------------------------------------------------------------
    These should be disabled by default.
    In murmur init script, # Default-Start: item should have no
    level.

* Init script fix
----------------------------------------------------------------------
diff -uN mumble-1.1.8-9.fc9.src/murmur.init
mumble-1.1.8-10.fc10.src/murmur.init
--- mumble-1.1.8-9.fc9.src/murmur.init  2009-04-30 01:03:18.000000000 +0900
+++ mumble-1.1.8-10.fc10.src/murmur.init        2009-05-15 08:32:48.000000000
+0900
@@ -52,7 +52,7 @@
        RETVAL=$?
        echo
        if [ $RETVAL -eq 0 ]; then
-               /bin/rm -f "$lockfile" "$pidfile" 2> /dev/null && success ||
failure
+               /bin/rm -f "$lockfile" "$pidfile" 2> /dev/null || failure
        fi
 }

----------------------------------------------------------------------
  - I guess " || failure" can also be removed.

* undefined non-weak symbols
----------------------------------------------------------------------
mumble.i586: W: undefined-non-weak-symbol /usr/lib/libmumble.so.1.1.8
glAttachShader
mumble.i586: W: undefined-non-weak-symbol /usr/lib/libmumble.so.1.1.8 glEnable
mumble.i586: W: undefined-non-weak-symbol /usr/lib/libmumble.so.1.1.8
glTexCoordPointer
....
(and so many)
----------------------------------------------------------------------
  - libmumble.so has undefined non-weak symbols. As
    this package does not provide -devel subpackage, currently this
    can be ignored, however it is preferable that this issue
    is fixed.
    ! You can also check this issue by
       $ ldd -r /usr/lib/libmumble.so.1 > /dev/null
    This can be fixed by making libmumble.so linked against libGL.so.

* Permission
----------------------------------------------------------------------
install -pD -m0664 man/murmurd.1 %{buildroot}%{_mandir}/man1/
install -pD -m0664 man/mumble* %{buildroot}%{_mandir}/man1/
----------------------------------------------------------------------
  - These should be 0644 permission

(In reply to comment #60)
> > * Installation directory
> > ------------------------------------------------------------------
> > install -pD -m0755 release/murmurd %{buildroot}%{_sbindir}/murmurd
> > ln -s ../sbin/murmurd %{buildroot}%{_bindir}/%{name}-server
> > ------------------------------------------------------------------
> >   - Why is the same command has to be in two different directories
> >     in the path with different names?
> Less confusion. After years of mumble nobody told me murmur - everyone says
> mumble-server.
  - Well, What I wonder is that where murmurd is under /usr/sbin,
    mumble-server is under /usr/bin. Why these directories are different?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]