Skip to content

Conversation

@HertzDevil
Copy link

@HertzDevil HertzDevil commented Nov 12, 2025

This is required on MSVC to copy the correct DLL to the output directory.

This assumes the DLL is the official sqlite-dll-win-x64-*.zip download.

This is required on MSVC to copy the correct DLL to the output directory
Copy link
Member

@bcardiff bcardiff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds good. Should we look into adding a CI for windows? And install instructions? Not sure how you get the official dll is msvc.

@HertzDevil
Copy link
Author

The official repository has a Makefile.msc NMAKE file, which is pretty much MSVC-exclusive, so it is hard to imagine them using something like this instead to build the official binaries.

The DLL itself does not depend on any C or C++ runtime library.

@ysbaddaden
Copy link

For the record: I tried building the specs on Windows in GitHub Actions with crystal-lang/install-crystal to install Crystal and choco install sqlite to install the library (it installs the .DLL and the .EXE) but crystal spec fails with:

Cannot locate the .lib files for the following libraries: sqlite3

There's indeed no .lib file installed.

@HertzDevil
Copy link
Author

I don't know about Choco, but from the official download you have to do something like lib /def:sqlite3.def inside an MSVC developer prompt to get the import library. There is probably a MinGW-w64 equivalent too but it is unnecessary with MSYS2

@ysbaddaden
Copy link

Thanks. It's packaging the original zip file, so I need to figure out how to start the developer console, run the command, and check if the folder is found 😓

@HertzDevil
Copy link
Author

MinGW: dlltool.exe -d sqlite3.def -D sqlite3.dll -l sqlite3-dynamic.lib
LLVM: llvm-dlltool.exe -d sqlite3.def -D sqlite3.dll -l sqlite3-dynamic.lib

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