Bug while running smallpt example notebook with lite

[anutosh491]

The smallpt example notebook was merged through this pr #299

I asked @kr-2003 to try it out too and it worked absolutely fine #299 (comment)

But this fails while we run it through our link https://compiler-research.org/xeus-cpp/lab/index.html

This is happening because in cell 4 we have this include (obviously required for any SDL utility)

#include <SDL2/SDL.h>

Now if you look at the stuff present in the sysroot path I see both SDL and SDL2

anutosh491@Anutoshs-MacBook-Air include % ls
.....
......
SDL        .....
SDL2 

So locally I see contents out of SDL2 in xcpp.data !!!!
So locally trying this out is absolutely fine.

But through our link, I don't see the file being present at all (which is very strange)

[anutosh491]

We can probably use

#include <SDL/SDL.h>

Getting the same result, but is there a reason why SDL overlaps/overwrites SDL2 (I see SDL2 headers being a part of xcpp.data locally so not sure if this should happen)

@DerThorsten what do you think ?

[anutosh491]

Through the link

Building master as it is without any changes locally

[anutosh491]

Arghhh okay just realized that we are building our deployment through ubuntu

xeus-cpp/.github/workflows/deploy-github-page.yml

Lines 25 to 26 in 98585e8

	- name: Github-page
	os: ubuntu-24.04

and there was an ubuntu build recently I suppose https://prefix.dev/channels/emscripten-forge-dev/packages/emscripten_emscripten-wasm32

Now @kr-2003 and I realize that this one fails to provide the SDL2 directory !

Hence running stuff locally on my macos (this one fetches this build py313hb76f48a_6 (6) ) and this works perfectly.

As a quick fix, we can try using a previous linux build for this or shift our deployment to macos if the first one doesn't work (technically I can change the notebook to use SDL.h from SDL rather than SDL2 if that's the easiest)

cc @SylvainCorlay

[SylvainCorlay]

(technically I can change the notebook to use SDL.h from SDL rather than SDL2 if that's the easiest)

That seems like a fine solution while you figure out why SDL2 is missing.

[vgvassilev]

Arghhh okay just realized that we are building our deployment through ubuntu

xeus-cpp/.github/workflows/deploy-github-page.yml

Lines 25 to 26 in 98585e8

• name: Github-page
  os: ubuntu-24.04
  and there was an ubuntu build recently I suppose https://prefix.dev/channels/emscripten-forge-dev/packages/emscripten_emscripten-wasm32

Now @kr-2003 and I realize that this one fails to provide the SDL2 directory !

Hence running stuff locally on my macos (this one fetches this build py313hb76f48a_6 (6) ) and this works perfectly.

As a quick fix, we can try using a previous linux build for this or shift our deployment to macos if the first one doesn't work (technically I can change the notebook to use SDL.h from SDL rather than SDL2 if that's the easiest)

cc @SylvainCorlay

You can probably use the has_include macro to detect which header file is available.

[anutosh491]

This is basically a weird emscripten issue of some sorts

Made a note of this on emscripten's github issues emscripten-core/emscripten#24394 (comment) if someone can help us figure this out !

[mcbarton]

I can confirm that it is not a Linux runner only issue. CppInterOps deployment already uses a MacOS runner, and the example doesn't work there either. I'm not entirely sure why the smallpt example was picked up by CppInterOps deployment, since we don't actually add the smallpt.ipynb example directly as can be seen here https://github.com/compiler-research/CppInterOp/blob/039b3fd6382c122c9ce6bb3542ee58096c058c76/.github/workflows/deploy-pages.yml#L196