[Bug 268069] security/clamav: 1.0.0 does no work with cld and cvd files

From: <bugzilla-noreply_at_freebsd.org>
Date: Wed, 14 Dec 2022 21:13:10 UTC
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=268069

fsbruva@yahoo.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #238736|0                           |1
        is obsolete|                            |

--- Comment #28 from fsbruva@yahoo.com ---
Created attachment 238795
  --> https://bugs.freebsd.org/bugzilla/attachment.cgi?id=238795&action=edit
clamav: patch to avoid math/tomsfastmath incompatibility

(In reply to Yasuhiro Kimura from comment #24)
It is related to the header search location used by CMake when it detects
security/openssl is installed. My previous patch was an error.

I confirmed this a few ways.

First method, I re-tried my submitted patch by itself, and it failed to fix the
error. I hadn't tested it alone - I had tested it with a few other CMake flags,
and obviously wasn't fully keeping track of which flag fixed it. Adding this
(by itself), clamav builds correctly, but fails to start:
-DCMAKE_IGNORE_PATH="/usr/local/include/tfm.h;/usr/local/lib/libtfm.a"

Second method, I reverted the clamav Makefile, then developed a series of port
patches that removed references to tfm.h and TOMSFASTMATH from clamav/bignum.h
and clamav-config.cmake.in. Resulting clamav 1.0.0 builds successfully, but
still errors out.

Third method, I removed the patches that modified the clamav source code. I
then moved the math/tomsfastmath header (tfm.h) out of /usr/local/include.
Resulting clamav 1.0.0 builds and starts successfully.

Fourth test, I re-installed the math/tomsfastmath header, and remove the
math/tomsfastmath library from /usr/local/lib (libftm.a). Resulting clamav
1.0.0 builds successfully, but fails to start.

Fifth method, I re-installed the math/tomsfastmath library, and then added the
following to the clamav Makefile:
-DOPENSSL_INCLUDE_DIR="/usr/local/include/openssl"
Resulting clamav 1.0.0 builds successfully and starts successfully.

My latest theory is the error is induced by CMake, when attempting to compile
the bundled tomsfastmath in clamav, when security/openssl and math/tomsfastmath
are both installed, because CMake adds this to the compile flags for the
detected security/openssl: 
-I/usr/local/include. 

I cannot find a way to tell CMake to exclude the specific tfm.h header file
that is confusing it.

-- 
You are receiving this mail because:
You are the assignee for the bug.