Use trailing slash with include directory

cmusphinx / pocketsphinx

A small speech recognizer

Other

3.87k stars 713 forks source link

Use trailing slash with include directory #332

Closed samford closed 1 year ago

samford commented 1 year ago

CMakeLists.txt references the include directory without a trailing slash, which can cause related files to be installed to include/include (e.g., we experienced this in Homebrew while updating the cmu-pocketsphinx formula to 5.0.0). Using include/ ensures the files will be installed to include, as expected (related documentation here: https://cmake.org/cmake/help/latest/command/install.html#directory).

dhdaines commented 1 year ago

Yikes! you're right. How did I miss that...

samford commented 1 year ago

Thanks for the quick turnaround! I've updated the related Homebrew PR to use the commit patch from this repository for the time being and we'll be able to remove the patch from the formula on the next release 👍