Bug#853801: useless with this bug

[Adam Borowski]

On Wed, May 17, 2017 at 11:11:49PM +0200, Stephen Kitt wrote:
> On Wed, 17 May 2017 23:03:19 +0200, Adam Borowski  wrote:
> > Uhm, then how possibly could it work without crashing?  Unless you already
> > have a config made with an earlier version, and don't change it.
> > 
> > Like the submitter, it reliably crashes for me every time I left-click to
> > bring the setup.
> 
> I don’t doubt you’re seeing crashes, I was just reacting to the claim that I
> didn’t even run the program. (I didn’t specifically verify #752125 though,
> that was committed a long time ago by Jordan.)

Hmm, is there something special with your setup?  I've just tried the
following:
* amd64 as in archive
* amd64 rebuilt with clang++
* i386 (although in a chroot with amd64 X)
* ssh -X to armhf (again, amd64 X)

The only way I found for left click to not crash is to not have any
joysticks connected, which brings an error message -- obviously avoiding
the code in question.


Meow!
-- 
Don't be racist.  White, amber or black, all beers should be judged based
solely on their merits.  Heck, even if occasionally a cider applies for a
beer's job, why not?
On the other hand, corpo lager is not a race.

Bug#853801: useless with this bug

[Stephen Kitt]

On Wed, 17 May 2017 23:03:19 +0200, Adam Borowski  wrote:
> On Wed, May 17, 2017 at 10:53:50PM +0200, Stephen Kitt wrote:
> > On Mon, 15 May 2017 06:22:59 +0200, Adam Borowski 
> > wrote:  
> > > The patch doesn't apply, need to revert that clang "fix" first.  Looks
> > > like someone only build-tested that, without even trying to run the
> > > program...  
> > 
> > That would be me, and I have qjoypad running as I write this.  
> 
> Uhm, then how possibly could it work without crashing?  Unless you already
> have a config made with an earlier version, and don't change it.
> 
> Like the submitter, it reliably crashes for me every time I left-click to
> bring the setup.

I don’t doubt you’re seeing crashes, I was just reacting to the claim that I
didn’t even run the program. (I didn’t specifically verify #752125 though,
that was committed a long time ago by Jordan.)

Regards,

Stephen


pgpPEop6uuYOi.pgp
Description: OpenPGP digital signature

Bug#853801: useless with this bug

[Adam Borowski]

On Wed, May 17, 2017 at 10:53:50PM +0200, Stephen Kitt wrote:
> On Mon, 15 May 2017 06:22:59 +0200, Adam Borowski  wrote:
> > The patch doesn't apply, need to revert that clang "fix" first.  Looks like
> > someone only build-tested that, without even trying to run the program...
> 
> That would be me, and I have qjoypad running as I write this.

Uhm, then how possibly could it work without crashing?  Unless you already
have a config made with an earlier version, and don't change it.

Like the submitter, it reliably crashes for me every time I left-click to
bring the setup.


-- 
Don't be racist.  White, amber or black, all beers should be judged based
solely on their merits.  Heck, even if occasionally a cider applies for a
beer's job, why not?
On the other hand, corpo lager is not a race.

Bug#853801: useless with this bug

[Stephen Kitt]

On Mon, 15 May 2017 06:22:59 +0200, Adam Borowski  wrote:
> The patch doesn't apply, need to revert that clang "fix" first.  Looks like
> someone only build-tested that, without even trying to run the program...

That would be me, and I have qjoypad running as I write this.

Regards,

Stephen


pgpEZXFPcwDTS.pgp
Description: OpenPGP digital signature

Bug#853801: useless with this bug

[Adam Borowski]

Control: severity -1 grave

Well, since left click is the only documented way to set up any mapping
(doing it manually requires 1. seeing a sample config, 2. having the table
of key codes), the package is effectively useless as shipped.

The patch doesn't apply, need to revert that clang "fix" first.  Looks like
someone only build-tested that, without even trying to run the program...


Meow!
-- 
Don't be racist.  White, amber or black, all beers should be judged based
solely on their merits.  Heck, even if occasionally a cider applies for a
beer's job, why not?
On the other hand, corpo lager is not a race.