Add include directories for fplll to dependent CMake targets

keeganryan / flatter

Fast lattice reduction

GNU Lesser General Public License v3.0

170 stars 18 forks source link

Add include directories for fplll to dependent CMake targets #4

Closed jacobkahn closed 1 year ago

jacobkahn commented 1 year ago

See title. The include path for<fplll/fplll.h> was never passed to relevant targets — only the library was linked.

Tested via

git clone https://github.com/fplll/fplll
cd fplll
./autogen.sh
./configure --prefix=$HOME/usr
make -j80
make install

git clone https://github.com/jacobkahn/flatter --branch fix_fplll_find
cmake -S . -B build -DCMAKE_PREFIX_PATH=$HOME/usr
cmake --build build --parallel

now succeeds

keeganryan commented 1 year ago

Looks good, thank you for the fix!