Skip to content

Commit 8a5af94

Browse files
committed
Merge #70: build: Remove -Qunused-arguments workaround for clang + ccache
98f87c5 build: Remove `-Qunused-arguments` workaround for clang + ccache (Hennadii Stepanov) Pull request description: The issue was addressed in [ccache 3.2](https://bugzilla.samba.org/show_bug.cgi?id=8118), and from a look at most major distros, it's only Debian Jessie that has a version of ccache older than that ([3.1](https://repology.org/project/ccache/versions)). Therefore I think it's acceptable to drop this workaround, and re-enable warnings for unused driver arguments (when compiling using Clang and ccache). This description quoted from bitcoin/bitcoin#18535. ACKs for top commit: sipa: ACK 98f87c5 Tree-SHA512: 1ade0de95ca6f3465b35702d3e964f4fc5c070f00ce163d8cbd4ab52362ac56e0e9d412a0c1ddfd808714bad46cd79084da16966d387a4ebc776ceaf8a5bce88
2 parents c36f1f0 + 98f87c5 commit 8a5af94

File tree

1 file changed

+0
-3
lines changed

1 file changed

+0
-3
lines changed

configure.ac

-3
Original file line numberDiff line numberDiff line change
@@ -124,9 +124,6 @@ if test "x$use_ccache" != "xno"; then
124124
fi
125125
AC_MSG_RESULT($use_ccache)
126126
fi
127-
if test "x$use_ccache" = "xyes"; then
128-
AX_CHECK_COMPILE_FLAG([-Qunused-arguments],[NOWARN_CXXFLAGS="$NOWARN_CXXFLAGS -Qunused-arguments"],,[[$CXXFLAG_WERROR]])
129-
fi
130127

131128
VERIFY_DEFINES=-DMINISKETCH_VERIFY
132129
RELEASE_DEFINES=

0 commit comments

Comments
 (0)