poker2d crash (SIGSEGV) at startup

[Johan Euphrosine]

This is already fixed in upstream svn, not released in debian yet

[Johan Euphrosine (proppy)]

This is the upstream patch that fix this issue.

[Cesare Tirabassi (norsetto)]

Thanks Johan,

unfortunately your patch do not apply correctly. I have therefore prepared a new patch based on your input, which includes as well the outstanding fix/overrides for the lintian/linda warnings.

One of the warnings concern the inclusion in your tarball and afterwards installation of the ttf-freefont fonts. Since these are pretty standard fonts, is there any particular reason why you add them in the tarball instead of just having a dependancy on them?
For the time being I fixed this just including a soft link and a dependancy but obviously I need your input on this.

I'm still in discussion with the Debian maintainer about how to fix the debconf issues, which, for the time being, means the configuration of the server has to be completed manually (editing /etc/).

[Johan Euphrosine (proppy)]

Hi, do you know why the patch didn't apply successfully, since I used cdbs-edit-patch I thought (wrongly) that I couldn't really shoot me in the foot.

Thanks for fixing the lintian/linda warnings, I will notify these fixes to the upstream, by forwarding your mail to <email address hidden>, I suggest you do the same with the font & debconf issue as the debian maintainer has pretty paranoid spam filter.

Thanks for the time you dedicate to help packaging our software for Ubuntu.

[Cesare Tirabassi (norsetto)]

Hi,

it was not the cdbs patch that didn't apply, but your debdiff; there was a problem with the files to which it applied (were you working in a chroot?).

In any case, with your patch applied, I'm still getting a SIGSEGV error:

Permission deniedPermission denied/usr/lib/python2.5/site-packages/pokerclient2d/gamewindow.py:150: Warning: g_object_get_qdata: assertion `G_IS_OBJECT (object)' failed
  return self.glade.signal_autoconnect(instance)
Fatal Python error: (pygame parachute) Segmentation Fault
Aborted (core dumped)

and I'm having a problem with pokerserver too:

poll reactor already installed
Traceback (most recent call last):
  File "/usr/sbin/pokerserver", line 35, in <module>
    run()
  File "/usr/lib/python2.5/site-packages/pokernetwork/pokerserver.py", line 109, in run
    app.startApplication(application, None)
  File "/usr/lib/python2.5/site-packages/twisted/application/app.py", line 400, in startApplication
    service.IService(application).startService()
  File "/usr/lib/python2.5/site-packages/twisted/application/service.py", line 233, in startService
    service.startService()
  File "/usr/lib/python2.5/site-packages/twisted/application/internet.py", line 73, in startService
    self._port = self._getPort()
  File "/usr/lib/python2.5/site-packages/twisted/application/internet.py", line 86, in _getPort
    return getattr(reactor, 'listen'+self.method)(*self.args, **self.kwargs)
  File "/usr/lib/python2.5/site-packages/twisted/internet/posixbase.py", line 467, in listenTCP
    p.startListening()
  File "/usr/lib/python2.5/site-packages/twisted/internet/tcp.py", line 733, in startListening
    raise CannotListenError, (self.interface, self.port, le)
twisted.internet.error.CannotListenError: Couldn't listen on any:19380: (98, 'Address already in use').

For this latter, perhaps something wrong with the configuration file?

Any idea what is causing this?

I've already passed my fixes to Loic (even though I'm still not completely happy about one); he is considering them for the next update.

PS. Sorry for the confusion, I thought YOU were upstream :-)

[Cesare Tirabassi (norsetto)]

At least some good news from Debian:

From: Loic Dachary
To: Cesare Tirabassi
Subject: Re: Poker2d: patch from upstream
Date: Wed, 05 Sep 2007 23:50:58 +0200

Hi,

I've tested the patch and it works.

Thanks !

[Daniel Holbach (dholbach)]

Does the patch apply now?

[Johan Euphrosine (proppy)]

It looks that the provided patch didn't correct the bug,
I'm working on a new one.
Thanks for your patience.

[Johan Euphrosine (proppy)]

Does someone manage to generate .crash file usable with apport-retrace ?

[Johan Euphrosine (proppy)]

Does someone manage to generate .crash file usable with apport-retrace ?
Thanks in advance

[Johan Euphrosine (proppy)]

Please add the following patch to your debdiffs,
This should correct the SIGSEGV at startup

[Cesare Tirabassi (norsetto)]

Thanks.
I've added it to my update: this is indeed solving the SIGSEGV related above.

[Daniel Holbach (dholbach)]

Can we get the changes upstream? They look fairly big. Other than that I'm happy to upload it.

[Cesare Tirabassi (norsetto)]

I understand Loic Dachary is both the DD and upstream, so he has got all the fixes.

[Johan Euphrosine (proppy)]

Changes commited upstream.

[Scott Kitterman (kitterman)]

First install attempt:

Selecting previously deselected package poker-web.
(Reading database ... 142988 files and directories currently installed.)
Unpacking poker-web (from poker-web_1.1.1-1ubuntu2_all.deb) ...
.: 15: Can't open /usr/share/dbconfig-common/dpkg/config.mysql
dpkg: error processing poker-web_1.1.1-1ubuntu2_all.deb (--install):
 subprocess pre-installation script returned error exit status 2
/var/lib/dpkg/tmp.ci/postrm: line 28: /usr/share/dbconfig-common/dpkg/postrm.mysql: No such file or directory
dpkg: error while cleaning up:
 subprocess post-removal script returned error exit status 1

Second install attempt:

$ sudo dpkg -i poker-web_1.1.1-1ubuntu2_all.deb
(Reading database ...
dpkg: serious warning: files list file for package `poker-web' missing, assuming package has no files currently installed.
143409 files and directories currently installed.)
Preparing to replace poker-web 1.1.1-1ubuntu2 (using poker-web_1.1.1-1ubuntu2_all.deb) ...
.: 15: Can't open /usr/share/dbconfig-common/dpkg/config.mysql
dpkg: error processing poker-web_1.1.1-1ubuntu2_all.deb (--install):
 subprocess pre-installation script returned error exit status 2
/var/lib/dpkg/tmp.ci/postrm: line 28: /usr/share/dbconfig-common/dpkg/postrm.mysql: No such file or directory
dpkg: error while cleaning up:
 subprocess post-removal script returned error exit status 1
Errors were encountered while processing:
 poker-web_1.1.1-1ubuntu2_all.deb

Please test on a system that does not have mysql installed, update the fix, and resubscribe UUS.

[Cesare Tirabassi (norsetto)]

There are several problems with this package at the moment, last but not least that the package is not configured properly by the maintainer's scripts.
The above patch doesn't claim to solve all the outstanding issues, but at least solves some (at least the fact that poker2d crash at startup, which is what this bug is about).
We can either keep the unusable version we have now, or have a working software albeit with some specific installation issues.
My personal suggestion is:

- assess if the above patch is acceptable in that it solves the reported problem.
- open anyhow a second bug report about the installation issue re. mySQL which will be worked in due time.

[Daniel Holbach (dholbach)]

Does it depends on mysql?

[Johan Euphrosine (proppy)]

I pasted Scott K. comment on a separate bug report as Cesare Tirabassi suggested:
https://bugs.launchpad.net/ubuntu/+source/poker-network/+bug/138836

Can someone "assess if the above patch is acceptable in that it solves the reported problem":
poker2d crash (SIGSEGV) at startup ?

[Cesare Tirabassi (norsetto)]

> Does it depends on mysql?

There are 3 binary packages: python-poker-network, python-poker2d and poker-web

python-poker-network is the poker server and also includes the client libraries.
It depends on mysql-client and suggests mysql-server.

python-poker2d is the GTK poker client; it depends indirectly on mysql since it depends on python-poker-network.

poker-web is the Web interface to a poker-network server: it only suggests mysql-client

[Johan Euphrosine (proppy)]

Following debdiff should solve both #137573 and #138836.

[Daniel Holbach (dholbach)]

Why are you overriding all the lintian and linda warnings?

[Cesare Tirabassi (norsetto)]

You are correct, all these will be in the next issue of the Debian package, so there is no need to do anything right now. We will simply sync from Debian next release.
Please unsubscribe u-u-s.

[Johan Euphrosine (proppy)]

Withdrawed Cesare Tirabassi changes from the patch, has he suggested.

[Johan Euphrosine (proppy)]

Use patch -p4 for the patch to apply correctly

[Johan Euphrosine (proppy)]

poker-network (1.1.1-1ubuntu2) gutsy; urgency=low

  * Fix poker-web installation issue (LP: #138836):
    - Add dbconfig-common (preinst required) as Pre-Depends

  * Fix SIGSEGV on start-up for poker2d (LP: #137573):
    - Add simple-patchsys.mk to debian/rules
    - Add 00-fix-double-initdisplay.patch patch in debian/patches
    - Add 01-fix-glade-autoconnect.patch in debian/patches

 -- Johan Euphrosine <email address hidden> Wed, 12 Sep 2007 18:01:32 +0200