search

adamdruppe / arsd

This is a collection of modules that I've released over the years. Most of them stand alone, or have just one or two dependencies in here, so you don't have to download this whole repo.
http://arsd-official.dpldocs.info/arsd.html
530 stars 128 forks source link

sqlite.d does not specify libs in package metadata #181

Closed michael-fadely closed 5 years ago

michael-fadely commented 5 years ago

This is a problem when generating Visual D solutions. It won't add sqlite3.lib to the list of dependencies the generated project and causes the build to fail.

adamdruppe commented 5 years ago

With dmd, it just works thanks to pragma(lib)... thinking VD might be using the dub, so I added it to dub.json. See if that works better.

On Thu, Jan 10, 2019 at 07:07:40AM -0800, Michael Fadely wrote:

This is a problem when generating Visual D solutions. It won't add sqlite3.lib to the list of dependencies the generated project and causes the build to fail.

michael-fadely commented 5 years ago

Sorry I haven't had a chance to test it out. I hope to do that today.

I believe the reason it doesn't work with the generated Visual D project is because it gets places in the .dub folder. The project file's directory is then typically used as the working directory/default lib search path, so the pragma fails to load the lib file (which in my case is in the package root dir).

michael-fadely commented 5 years ago

Yep, it now adds the lib file to the list of required libraries. It looks like it's still going to end up searching the project's working directory by default, though (.dub), so this probably ends up being moot. Would probably be easier for me to just add the lib to my PATH rather than wrangle dub into cooperating.

adamdruppe commented 5 years ago

Yeah, maybe some linker flag could do it too with dub's lflags-windows config thing but I don't really know.