Bugzilla Bug 61072

By default doomsday sets ambient light to 0 , which makes the game unplayable
as its too dark.
Even if you change the settings in panel it keeps resetting it back 0.
This patch forces ambient light level to 200 which is ideal for almost all
users and make doom look more like the real doom , than like doom3.


Reproducible: Always
Steps to Reproduce:
1.
2.
3.




Attaching new ebuild and .patch file to this bug report

Created an attachment (id=37817) [edit]
file goes in /usr/portage/games-fps/doomsday

updated ebuild with ambient light patch

Created an attachment (id=37818) [edit]
file goes in /usr/portage/games-fps/doomsday/files

the ambient-light patch file

Did you also submit this upstream?  *grin*

Created an attachment (id=37835) [edit]
doomsday-1.8.1 ebuild with ambient light patch

While i was at it , the doomsday team have just released v1.81

This is the 1.81 ebuild with the ambient light patch attached next.

Created an attachment (id=37836) [edit]
Updated ambient patch for doomsday v1.81

This patch with lock minimum ambience at 200.

If the user sets it to a higher level it will no longer reset it to 200.

Added to CVS... thanks for the patch!

I also submitted this upstream to the authors.

I just received an email from the package author, asking that I do *not*
include this patch into the ebuild.

Here is the email:

From:   Jaakko Ker

I just received an email from the package author, asking that I do *not*
include this patch into the ebuild.

Here is the email:

From:   Jaakko Keränen <jaakko.keranen@iki.fi>
To:     wolf31o2@gentoo.org
Subject:        Re: Ambient Light patch for Doomsday 1.8.1
Date:   Wed, 1 Sep 2004 16:23:41 +0300  
On Sep 1, 2004, at 1:23, Chris Gianelloni wrote:
> This patch was submitted to bug #61072 at bugs.gentoo.org and solves a
> problem with ambient lighting being too dark.

I was not aware that such a problem even existed. Anyway, what the 
patch does is unacceptable. The ambient light level is supposed to be 
zero during normal game play and is only intended for debugging because 
it breaks the Doom lighting model. Perhaps the patch submitter was 
really looking for a way to increase display brightness, like with the 
color settings (gamma, contrast, brightness) in the Control Panel 
(which have been working using SDL since Doomsday 1.8.1).

I ask that you do not include this patch in the Gentoo Doomsday package.

Regards,
--
Jaakko Keränen
http://www.iki.fi/code/


Due to this request, I am going to remove the patch, and resolve this bug as
WONTFIX.

Patch reverted...

Sorry if my patch offended the Author.

I had made this on my machine in for doomsday 1.8.0 because the brightness controls were not working with an older nvidia driver .

However in doomsday 1.8.1 this problem is fixed and the brightness slider works very well.

No problem... it was apparently a problem with 1.8.0, but he had already made
changes that would negate the need for your patch for 1.8.1... it happens.