Skip to content

Update CMakeLists.txt - fix issue missing libfzf.dll, expected in build, found in build/Release#143

Closed
lyoneel wants to merge 1 commit intonvim-telescope:mainfrom
lyoneel:patch-1
Closed

Update CMakeLists.txt - fix issue missing libfzf.dll, expected in build, found in build/Release#143
lyoneel wants to merge 1 commit intonvim-telescope:mainfrom
lyoneel:patch-1

Conversation

@lyoneel
Copy link

@lyoneel lyoneel commented Feb 10, 2025

Using the following configuration in windows using lazy.nvim and cmake:
{ 'nvim-telescope/telescope-fzf-native.nvim', build = 'cmake -S. -Bbuild -DCMAKE_BUILD_TYPE=Release && cmake --build build --config Release' }

Neovim complains about a missing dll file in the build directory. I found that the dll file is compiled and is inside build/Release folder.

This fix specify explicitly the build output directory to be ./build directory.

This was tested on win10

NOTE: I didn't realize there were other PRs addressing the issue, I just fix by myself ASAP and copy my solution here.

Speficy explicitly the build output directory, without this change libfzf.dll will be created inside ./build/Release, and neovim plugin expects this file in ./build directory.
@bruxisma
Copy link
Contributor

bruxisma commented Mar 8, 2025

Hi! I'm the original author of the CMakeLists.txt + presets file.

If you use --target install when building the project it'll place it into the correct directory. This is why there is

install(TARGETS ${PROJECT_NAME} DESTINATION ${CMAKE_BINARY_DIR})

At the very bottom of the CMakeLists.txt file. I'm unsure why the --target install was removed in the README, but it is necessary for the build step to work correctly in one go.

@clason
Copy link
Contributor

clason commented Nov 7, 2025

included in #158 (thanks!)

@clason clason closed this Nov 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants