Skip to content

RELEASE build broken on windows with SUPPORT_JBM_TRACEFILE

Basic info

  • Commit SHA: acdbe6
  • Platform: Windows

Bug description

The preprocessor macro SUPPORT_JBM_TRACEFILE currently breaks the RELEASE build of Visual Studio. The reason is that this switch introduces a dependency on jbm_file_writer.h into lib_dec. The file is part of lib_util. If I remember correctly, only the applications should depend on lib_util while the libraries should not - this is more of a design principle question, though. However, when I look into lib_dec->properties->C/C++->Additional Include Directories, there is lib_util listed for both DEBUG and RELEASE configs, so I don't understand why this is not building.

Side note: I tried moving the macro definition under ifndef RELEASE in options.h, but then found that the RELEASE configuration does not actually inject the RELEASE preprocessor macro. This is probably also something to fix. When I manually added #define RELEASE before that, the build does fail with even more errors...

Ways to reproduce

Open project in Visual Studio, switch to RELEASE configuration and build.