Time  Nick      Message
01:55 MTDiscord <paradust> sfan5: I asked Claude to look at the screenshot commit and determine whether it will work. The result might interest you: https://luanti.dustlabs.io/screenshots.png
07:29 sfan5     I am not surprised and the explanation makes sense (I remember seeing a section about swap behavior on the OpenGL wiki). this means the code in the legacy driver is and was always broken.
07:30 sfan5     however I only see glReadBuffer(GL_BACK) being invalid with an FBO bound in the docs for GLES3, not GL3/4. might be a GLES specific limitation that you can not read from the back buffer if an fbo is bound (why?)
10:30 MTDiscord <paradust> sfan5: I believe its using https://registry.khronos.org/OpenGL/specs/gl/glspec30.pdf, page 269, and the table on page 258.
10:30 MTDiscord <paradust> https://cdn.discordapp.com/attachments/747163566800633906/1549366756919873618/Screenshot_From_2026-09-15_06-27-18.png?ex=6aaa6fbc&is=6aa91e3c&hm=db351e5c4da1776693a5cc858c96fea37fe2dc3f5e496310d0c96c21dde60df8&
10:30 MTDiscord <paradust> (can you see screenshots?)
10:30 MTDiscord <paradust> i mean, can you see discord image pastes
10:33 sfan5     I can
10:33 sfan5     my source is https://docs.gl/gl3/glReadBuffer
10:38 MTDiscord <paradust> the reference page appears to be incomplete
10:38 sfan5     looks like that's the case
11:02 MTDiscord <paradust> the suggested changes are quite small, I will make a PR
11:14 sfan5     you can get rid of the "preferBackbuffer" thing entirely. my intent was that a caller may want to potentially make a screenshot of a render target and not the back buffer
11:14 sfan5     but this won't actually work this way
11:22 MTDiscord <paradust> I'll add that to the PR
12:22 sfan5     any reviewers for #14900?
12:22 ShadowBot https://github.com/luanti-org/luanti/issues/14900 -- Mods should be able to influence the fixed map lighting
12:37 pgimeno   reviewers of what? (also, pet peeve of mine about ShadowBot: why doesn't it distinguish PRs from issues in the URL? all it would need to do is read the Location: header)
12:38 pgimeno   you mean #16697?
12:38 ShadowBot https://github.com/luanti-org/luanti/issues/16697 -- [no squash] Make static world lighting (light curve & AO) configurable via API by sfan5
12:42 sfan5     paste fail
12:42 sfan5     yes
12:42 sfan5     any coredevs specifically
17:29 MTDiscord <andrey2470t> 17523 is ready for re-review