Created attachment 182055 [details]
Fix clang 5 detection for valgrind
I'm doing preliminary tests with clang 5.0.0 in the projects/clang500-import branch, and it turned out that devel/valgrind and devel/valgrind-devel fail to detect the version. This is because of a case statement with:
Here's a patch that adds clang-5.* for now (upstream might want to think about extending this to any correct numerical version). It also drops the USE_GCC from the Makefiles, which is apparently something added by Baptiste to support FreeBSD 8, so it is no longer necessary.
Approved, please commit
A commit references this bug:
Date: Wed Jul 5 17:20:57 UTC 2017
New revision: 445087
Detect clang 5.0.0 correctly, and drop USE_GCC.
Approved by: bdrewery (maintainer)
USE_GCC hunk is missing from ports r445087, so the commit message is slightly confusing.
(In reply to Jan Beich from comment #3)
> USE_GCC hunk is missing from ports r445087, so the commit message is
> slightly confusing.
Ah yeah sorry, that was already fixed in r440063 by bdrewery, some time ago. I had that in my tree for ages, so I was sort of convinced it still needed to be done. :)