Skip to content

Enable deprecation warnings#2036

Merged
nomis52 merged 1 commit intoOpenLightingProject:masterfrom
DaAwesomeP:DaAwesomeP-enable-deprecation-errors
Mar 22, 2026
Merged

Enable deprecation warnings#2036
nomis52 merged 1 commit intoOpenLightingProject:masterfrom
DaAwesomeP:DaAwesomeP-enable-deprecation-errors

Conversation

@DaAwesomeP
Copy link
Copy Markdown
Member

It seems that we missed a lot of the deprecation warnings because they were turned off. Now that we have settled on minimum compiler support in #2005, it seems appropriate to turn these on and not ignore them. If we run into something specific that is deprecated in modern GCC/Clang but has no replacement for the minimum compiler version we support, we can handle those case-by-case.

The build for this PR will intentionally fail until many of @aroffringa's PRs are merged. Once those are merged I will rebase and we can see if we missed anything.

@DaAwesomeP DaAwesomeP force-pushed the DaAwesomeP-enable-deprecation-errors branch from 7683c15 to ef7e1af Compare March 22, 2026 16:46
@aroffringa
Copy link
Copy Markdown
Contributor

Good idea. I see in your changes the warnings were disabled in CI, but not on a normal build (those related to a normal build were all -Wno-error options). If I do a local build with mine and your changes combined I don't see warnings.

I can't approve, but the changes look fine to me once the warning fixes are merged.

@DaAwesomeP
Copy link
Copy Markdown
Member Author

I see in your changes the warnings were disabled in CI, but not on a normal build (those related to a normal build were all -Wno-error options).

The GNU_PLUS_PLUS_11_DEPRECATIONS part was enabling -Wno-error=deprecated-declarations for local builds as well. If I try build now locally it doesn't get very far and runs into std::binary_function and std::auto_ptr.

If I do a local build with mine and your changes combined I don't see warnings.

Yay! That means you found everything.

@nomis52 nomis52 merged commit 4e82005 into OpenLightingProject:master Mar 22, 2026
8 of 22 checks passed
@aroffringa
Copy link
Copy Markdown
Contributor

@nomis52 be aware that by merging this, a standard

./configure
make

in the source dir fails to compile now, so the fixes #1890 and #1889 have to be merged to repair such a master build...

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.

3 participants