Bug 4021 - Shader order bug
Status: RESOLVED INVALID
Alias: None
Product: ioquake3
Classification: Unclassified
Component: Video
Version: GIT MASTER
Hardware: PC Windows XP
: P3 normal
Assignee: Tim Angus
QA Contact: ioquake3 bugzilla mailing list
URL:
Depends on:
Blocks:
 
Reported: 2009-03-25 15:54 EDT by Eugene C.
Modified: 2009-10-25 17:09:33 EDT
1 user (show)

See Also:


Attachments
solution (578 bytes, patch)
2009-03-25 15:54 EDT, Eugene C.

Description Eugene C. 2009-03-25 15:54:55 EDT
Created attachment 2022 [details]
solution

Shader hash chains should be stored in reverse order to allow more recent (custom) shaders to be used instead of default shaders with the same name.
Comment 1 Zachary J. Slater 2009-03-25 17:15:36 EDT
This might be a result of revision 1129 http://svn.icculus.org/quake3/trunk/code/renderer/tr_shader.c?r1=1124&r2=1129 but I can't be sure why that behavior was changed. Timbo, can you?
Comment 2 Zachary J. Slater 2009-03-25 17:24:57 EDT
changing to Tim....
Comment 3 Thilo Schulz 2009-10-25 16:18:04 EDT
Not a bug, this is standard quake3 behaviour and was never changed in ioquake3 in the first place.
If you want custom shaders with the same name to replace the default ones, just make sure the corresponding pak files get loaded before pak[0-9].pk3
If we were to change this now, god knows how many quake3 installations would be broken due to some mappers being careless and having shaders with default names in their maps.
Comment 4 Eugene C. 2009-10-25 17:09:33 EDT
(In reply to comment #3)
> Not a bug, this is standard quake3 behaviour and was never changed in ioquake3
> in the first place.
> If you want custom shaders with the same name to replace the default ones, just
> make sure the corresponding pak files get loaded before pak[0-9].pk3
> If we were to change this now, god knows how many quake3 installations would be
> broken due to some mappers being careless and having shaders with default names
> in their maps.

If you mean ioquake3 as standard quake3 then yes, but if you mean 1.32[b,c] then definitely not - just put custom console shader and check it