wip: fix MSVC CI by adding opus_types shim#9
Conversation
|
Fixes included in this update:\n- tests: add missing 'tests/test_oac_demo_wav.c' (fixes AutoTools make error)\n- ci: add Clang ASAN/UBSAN workflow to catch memory/UB issues\n- fix: remove unused WAV variable in 'src/oac_demo.c'\n\nI've pushed these commits to 'wip/fix-msvc-ci' (updated branch). Please re-run CI (or it will trigger automatically). If Windows/MSVC failures persist, I can investigate the DNN data header syntax errors next. |
|
Temporary workaround: disable building DNN/DRED/OSCE on MSVC (cl) to avoid numerous compile errors in generated DNN sources. This is a short-term mitigation so Windows MSVC CI can turn green while we investigate making the generated dnn/* sources MSVC-friendly (fix typedefs, constant initializers, or generator). Will follow up with generator fixes and/or more targeted shims. CI updated and branch pushed: wip/fix-msvc-ci |
…art1) — MSVC-friendly outputs
…nerated DNN symbols
…designated-initializer regex
…oaci_lpcnet_plc_*)
|
Is there anything in there that still applies after PR1 is merged? |
2 participants
This is a WIP/draft patch that adds a compatibility shim
include/opus_types.hand registers it in CMake/Makefile to address Windows MSVC CI failures where generated DNN headers expectopus_types.h. Please re-run CI to see if the errors are resolved; I'll follow up with more MSVC-specific fixes if needed.