Bug 143662 – xscreensaver-5.00 badly parse a float resource

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Shogun 2006-08-12 05:47:08 UTC]

When I try to execute glmatrix, it returns this:

glmatrix: speed must be a float, not 1.0.

so I realized that it badly parses a float option.
I saw the code and I found the problem a fixed it with a patch I attach to this report.
I've already mailed the author of xscreensaver about this...
Bye guys!
 Shogun

[Comment 1 Shogun 2006-08-12 05:47:32 UTC]

Created attachment 94045 [details, diff]
Patch to fix the problem

[Comment 2 Krzysztof Pawlik (RETIRED) 2006-09-03 02:52:50 UTC]

Fixed in 5.00-r1, thanks.

[Comment 3 Samuli Suominen (RETIRED) 2007-06-13 18:53:36 UTC]

jwz, we are also applying this for 5.02 and I'm not sure have you been informed about it yet. review & comment would be appericiated.

(trying to keep xscreensaver much upstream as possible)

[Comment 4 Samuli Suominen (RETIRED) 2007-06-13 18:55:32 UTC]

err, missed the part "I've already mailed the author of xscreensaver about this..."

[Comment 5 Jamie Zawinski 2007-06-13 19:01:25 UTC]

That patch is definitely not right.

I vaguely recall seeing this report and fixing it elsewhere, but I can't remember what I did.  Does it still happen in 5.02 without that patch?

[Comment 6 Samuli Suominen (RETIRED) 2007-06-13 20:40:48 UTC]

(In reply to comment #5)
> That patch is definitely not right.

Figured. I've removed it.

> 
> I vaguely recall seeing this report and fixing it elsewhere, but I can't
> remember what I did.  Does it still happen in 5.02 without that patch?
> 

Couldn't reproduce it with 5.02.

Thanks for your time. Again.