Skip to content

Commit

Permalink
Set -Wno-deprecated-declarations for the OSS build (#446)
Browse files Browse the repository at this point in the history
Summary:
Building mcrouter currently generates a high volume of deprecation warnings from generated thrift code and from OpenSSL-related deprecations in upstream folly. We already try to set `-Wno-deprecated`, but in modern GCC and clang `-Wno-deprecated-declarations` is needed to silence these warnings instead.

Pull Request resolved: #446

Reviewed By: djvaporize

Differential Revision: D59706029

Pulled By: stuclar

fbshipit-source-id: b2751c59f53a58d5f765e958bdac3229e3a81dc2
  • Loading branch information
mszabo-wikia authored and facebook-github-bot committed Jul 25, 2024
1 parent 34eb62e commit 805351f
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion mcrouter/configure.ac
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ LT_INIT

CXXFLAGS="-fno-strict-aliasing -std=c++20 $CXXFLAGS"
CXXFLAGS="-W -Wall -Wextra -Wno-unused-parameter $CXXFLAGS"
CXXFLAGS=" -Wno-missing-field-initializers -Wno-deprecated $CXXFLAGS"
CXXFLAGS=" -Wno-missing-field-initializers -Wno-deprecated-declarations $CXXFLAGS"
CXXFLAGS="-DLIBMC_FBTRACE_DISABLE -DDISABLE_COMPRESSION $CXXFLAGS"

CFLAGS="-DLIBMC_FBTRACE_DISABLE -DDISABLE_COMPRESSION $CFLAGS"
Expand Down

0 comments on commit 805351f

Please sign in to comment.