slipher commented Jan 19, 2025

It appears that the NaCl exception handling method is broken in general. I tried the following patch: diff --git a/src/common/System.cpp b/src/comm…

slipher commented Jan 19, 2025

Apparently it happens on MSYS MinGW also.

Gireen commented Jan 18, 2025

oblivious illegal Is that, like, accidentally snorting a bunch of cocaine? I think consumption is not illegal, and hard to achieve with text al…

sweet235 commented Jan 18, 2025

oblivious illegal Is that, like, accidentally snorting a bunch of cocaine? I suppose that, apart from murder and the likes, very few things a…

VReaperV commented Jan 18, 2025

oblivious illegal Is that, like, accidentally snorting a bunch of cocaine?

sweet235 commented Jan 18, 2025

"admin job isn't done right" sounds way to squishy It does. But if we need to contact a server owner because of abusive behavior not being proce…

Gireen commented Jan 18, 2025

But its unavoidable. What matters is where the line is drawn and to preserve as much freedom as possible. When there is a DDOS attack it should be …

sweet235 commented Jan 18, 2025

I do not like the idea of partitioning people into good and bad ones.

VReaperV commented Jan 17, 2025

Yeah, I get this result: https://users.unvanquished.net/~reaper/test2.mp4

VReaperV commented Jan 17, 2025

OK so there's a bug with listCmds that it won't show commands without a flag. Yeah, found the same issue.

VReaperV commented Jan 17, 2025

But I don't see -- anywhere with the migrated commands. This works fine on my end after a full rebuild.

slipher commented Jan 17, 2025

OK so there's a bug with listCmds that it won't show commands without a flag. static ListCmdsCmd listCmdsRegistration("listCmds", BASE, "lists all …

VReaperV commented Jan 17, 2025

I'm certain of that. Apparently, it works properly with auto-completion, but not otherwise: Regular usage: ]/listcmds gfx 0 commands ]/listcmds sh…

slipher commented Jan 17, 2025

The descriptions for e. g. shaderexp and buildcubemaps show up as -- when using /listCmds, rather than evaluate a q3shader expression (RB_EvalExpr…

slipher commented Jan 17, 2025

This doesn't fix the issue on my end (tested with blaster and prifle, dll sgame). Really? I tested the branch again and can't reproduce the issue…

slipher commented Jan 17, 2025

Why not use the int main() no-arg variant? Then you can be sure not to get trolled by unused argument warnings.

slipher commented Jan 17, 2025

I guess the "flakiness" may be that sometimes the NaCl subfolder runs configuration and sometimes not, which gave me the impression of nondetermini…

cu-kai commented Jan 17, 2025

LGTM Might that fix Unvanquished/Unvanquished#1384 ? No, I rather suspect this is a different problem. Maybe if SDL is < 2.022 it does not set…

slipher commented Jan 17, 2025

LGTM Might that fix Unvanquished/Unvanquished#1384 ? No, I rather suspect this is a different problem. Maybe if SDL is < 2.022 it does not set …

VReaperV commented Jan 17, 2025

Tested, there's no discernible impact on loading times. The warn that tells which exact input is NaN is a bit spammy though. That might not be much…

VReaperV commented Jan 17, 2025

The descriptions for e. g. shaderexp and buildcubemaps show up as --, rather than evaluate a q3shader expression (RB_EvalExpression) and generate c…

VReaperV commented Jan 17, 2025

I suspect the spectating player switching bug comes from the fact we may play an animation from the previous models with the new model, or the oth…

VReaperV commented Jan 17, 2025

Intereseting, I'm pretty sure I hadn't seen that specific warn before, but I've been able to get it now locally as well, so I was wrong about it ea…

VReaperV commented Jan 17, 2025

This doesn't fix the issue on my end (tested with blaster and prifle, dll sgame).

DolceTriade commented Jan 17, 2025

Also remember that crash.unvanquished.net is a thing. We just need to remember to upload syms to it.

cu-kai commented Jan 17, 2025

Might that fix Unvanquished/Unvanquished#1384 ? No, I rather suspect this is a different problem.

illwieckz commented Jan 17, 2025

I suspect the spectating player switching bug comes from the fact we may play an animation from the previous models with the new model, or the othe…

slipher commented Jan 17, 2025

I just tried it and I got the message like the OP: ^3Warn: trap_R_BlendSkeleton: different number of bones 38 != 122 ^3Warn: trap_R_BlendSkeleton: …

illwieckz commented Jan 17, 2025

P.S. The messages about compilers/archs "not being used" also seem wrong. This message was previously intended to be printed before the unused de…

illwieckz commented Jan 17, 2025

Here is the fix: #1506 The cause is that PNaCl doesn't allow int main(int, char**) without parameter names in C. It is probably a matter of defau…

illwieckz commented Jan 17, 2025

I added a workaround for finding the minidump-generator and nacl-exception libraries in x86_64-nacl/lib32 when linking a i686 nexe, this workaround…

VReaperV commented Jan 17, 2025

Yes it does. It's pretty easy to reproduce if you just go on spec and scroll back and forth between different players. No, that is a different is…

illwieckz commented Jan 17, 2025

Here is the results of my tests on Linux: So, only linux-amd64 is affected.

illwieckz commented Jan 17, 2025

This is a good example of why we should not merge a complex PR like #982 without any code review. I answered there because I don't want to pollut…

illwieckz commented Jan 17, 2025

@slipher I answer here because I don't want to pollute that other thread with that topic: #1505 This is a good example of why we should not merg…

slipher commented Jan 17, 2025

Yes it does. It's pretty easy to reproduce if you just go on spec and scroll back and forth between different players.

illwieckz commented Jan 17, 2025

Interesting: Failed to build DaemonCompiler.c, relying on CMake builtin detection. -- Detected C compiler: Unknown Unknown pnacl-clang.bat -- Dete…

VReaperV commented Jan 17, 2025

This does not seem to happen anymore.

slipher commented Jan 17, 2025

Hardcoding the string instead of doing a version check could be better, since then an SDL built with old headers could still work with a newer binary.

slipher commented Jan 17, 2025

Might that fix Unvanquished/Unvanquished#1384 ?

slipher commented Jan 17, 2025

Debugging update: Interactive debugging remains somewhat usable. Tested with an old nacl-gdb. Postmortem crash analysis (which we actually use) do…

cu-kai commented Jan 17, 2025

So now I check for SDL 2.0.22 as that is where this feature was introduced. Also made sure I don't try to reference SDL in the TTY build, oops. I'v…

slipher commented Jan 17, 2025

Tried upstream Breakpad (https://chromium.googlesource.com/breakpad/breakpad) and dump_syms still crashes at the same place.

slipher commented Jan 17, 2025

A Saigo-built binary can produce a crash dump when crashing under Daemon (though we don't yet know if it's valid). However, Breakpad's dump_syms cr…

VReaperV commented Jan 17, 2025

Fixed.

slipher commented Jan 16, 2025

Fixed by #3057.

slipher commented Jan 16, 2025

I will test if minidump generation and stack tracing works in a bit. Or if anyone else wants to try it, https://wiki.unvanquished.net/wiki/Breakpad …

illwieckz commented Jan 16, 2025

The nacl_minidump.h file from libs/is exactly the same one as shipped with pnacl.

VReaperV commented Jan 16, 2025

Light styles are off again. It seems that the light factor is not being applied to light style stages: if I set r_overbrightDefaultBits 0, the dif…

illwieckz commented Jan 16, 2025

OK, done.

slipher commented Jan 16, 2025

nacl_exception.h can be removed. We don't call any of those functions directly. The nacl_exception library, however, is needed as a dependency of m…

slipher commented Jan 16, 2025

Light styles are off again. It seems that the light factor is not being applied to light style stages: if I set r_overbrightDefaultBits 0, the diff…

VReaperV commented Jan 16, 2025

This fixes the red and black dretch banners in metro-b1-2. Previously they were not displayed with the material system. Oh, nice, I saw a few pl…

VReaperV commented Jan 16, 2025

True, added that.

VReaperV commented Jan 16, 2025

The shader was doing ( colorMod & 4 ) * lightFactor, and the first part would always produce 4 if it needed to produce lightFactor, but this doesn'…

VReaperV commented Jan 16, 2025

Hmm, true, didn't notice that about mapOverBrightBits. Changed it to just be 3 bits now.

VReaperV commented Jan 16, 2025

Fixed, this was a leftover of a different implementation that did not use a look-up array.

VReaperV commented Jan 16, 2025

Done.

VReaperV commented Jan 16, 2025

Apparently this got automatically done by my IDE at some point, fixed now.

slipher commented Jan 15, 2025

There's some discrepancy here on how many bits it is allowed to be. This takes 4 bits but the comment above colorModArray describes it as 8 bits. M…

slipher commented Jan 15, 2025

The constant 4 looks suspicious... I find it produces the wrong result for some vertex lighting if r_overbrightDefaultExponent is set to 1. Station15

slipher commented Jan 15, 2025

Probably should be Cvar::CHEAT

illwieckz commented Jan 15, 2025

So, here is the repository where I implement the compilation build script of Saigo: https://github.com/DaemonEngine/DaemonSaigoNativeClientToolkit …

slipher commented Jan 15, 2025

I think the main problem is that the zero on the pitch axis as seen by the bot's command buffer can be offset for an extended period of time. The d…

sweet235 commented Jan 15, 2025

To make the bots be (sometimes) affected by chaingun jitter and trapper grab The bots were affected by the trapper grab before #3057. I will have…