Skip to content

chore: hide noisy build log warnings#1175

Open
mcoliver wants to merge 2 commits intoAcademySoftwareFoundation:mainfrom
mcoliver:build-warning-supression
Open

chore: hide noisy build log warnings#1175
mcoliver wants to merge 2 commits intoAcademySoftwareFoundation:mainfrom
mcoliver:build-warning-supression

Conversation

@mcoliver
Copy link
Contributor

@mcoliver mcoliver commented Mar 3, 2026

Local build logs are incredibly noisy and often surpass 100,000 lines. Hiding these two specific warnings cleans up over 10% of the log lines. Pyside6 builds alone produce over 12,000 lines of these warnings

/Users/moliver/dev/_openrv/OpenRV/_build/_deps/rv_deps_pyside6-src/build/.venv/build/pyside6/PySide6/QtCore/PySide6/QtCore/qtimer_wrapper.cpp:1198:18: warning: cast from 'PyObject *(*)(PyObject *)' (aka '_object *(*)(_object *)') to 'PyCFunction' (aka '_object *(*)(_object *, _object *)') converts to incompatible function type [-Wcast-function-type-mismatch]
 1198 |     {"interval", reinterpret_cast<PyCFunction>(Sbk_QTimerFunc_interval), METH_NOARGS, nullptr},
      |                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 

Adding

…no-cast-function-type-mismatch (GCC/Clang)
-Wno-deprecated-declarations (GCC/Clang)
/wd4996 (MSVC equivalent for deprecated declarations)

Linked issues

Summarize your change.

Describe the reason for the change.

Describe what you have tested and on which operating system.

Add a list of changes, and note any that might need special attention during the review.

If possible, provide screenshots.

mcoliver added 2 commits March 3, 2026 10:01
…no-cast-function-type-mismatch (GCC/Clang)

-Wno-deprecated-declarations (GCC/Clang)
/wd4996 (MSVC equivalent for deprecated declarations)
Signed-off-by: Michael Oliver <mcoliver@gmail.com>
…ve 10% (12,000) log lines.

Signed-off-by: Michael Oliver <mcoliver@gmail.com>
@mcoliver mcoliver changed the title clean up 90% of build log warnings which are quite noisy by adding -W… chore: hide noisy build log warnings Mar 3, 2026
@mcoliver mcoliver marked this pull request as ready for review March 3, 2026 19:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant