[Bug 499137] Review Request: sipwitch - SIP telephony server for secure phone systems

bugzilla at redhat.com bugzilla at redhat.com
Sun May 10 01:47:51 UTC 2009


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=499137





--- Comment #9 from David Sugar <dyfet at gnutelephony.org>  2009-05-09 21:47:50 EDT ---
The sipwitch.py issue is partially related to swig, since it is generated by
swig.  I changed the patch to install using INSTALL_DATA, and maybe I will need
to set explicit attrib too?  I will test this a bit here...

I see now yes, "upstart" based debian/ubuntu only have /etc/init.d, where
rh/fedora symlink it to /etc/rc.d/init.d.  In the next upstream release I am
adding --with-initrddir option for configure, but for the current release, I
will use your hack...

The stop-start-daemon issue gets to the heart of some init scripting
differences between debian & rh.  I am not sure if this is best solved with a
patch, since init scripts dont change much, so that would not be too hard to
maintain, or to have an entirely separate source1 just to take along a
fedora/rh specific init script.

I have already worked on the other changes, and once I test them and resolve
the debian vs rh init script issues remaining, I will do a new update. 
Probably I will go for extending the patch for now.

Incidentally, if gcrypt is not used, sipwitch falls back to openssl libcrypto. 
This in fact should have been preferred since libeXosip2 already links in
openssl, but it seems to fail to detect libcrypto support (even for md5), and
that is strange.  Also I tested mostly with the gcrypt crypto functions, so it
is probably safer to keep using those for now.  Anyway, at some point we all
should try to  converge on nss based stuff :).

-- 
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.




More information about the Fedora-package-review mailing list