Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Crash in unittest on db.execute("DROP TABLE test"), Windows x86, sqlite 3.10.0 #22

Open
MrSmith33 opened this issue Jan 10, 2016 · 3 comments
Labels
Unsure Needs confirmation or more info

Comments

@MrSmith33
Copy link

Crash occurs on these lines:
https://github.com/biozic/d2sqlite3/blob/master/source/d2sqlite3.d#L934
https://github.com/biozic/d2sqlite3/blob/master/source/d2sqlite3.d#L940

Crashes with and without assertThrown
Happens with master and 0.8.1.

@biozic
Copy link
Collaborator

biozic commented Jan 11, 2016

I managed to have access to a Windows setup. I could reproduce the crash using the precompiled sqlite3 DLL from the sqlite website. But then I built it from the amalgamation source, and there were no problems. Maybe you could check that yourself as well.

DMD 2.069.2 (-m32mscoff)

@MrSmith33
Copy link
Author

I compiled it myself
"C:\Program Files (x86)\Microsoft Visual Studio 12.0\VC\bin\vcvars32.bat" && cl /O2 /Isqlite3.h /D_USRDLL /D_WINDLL sqlite3.c /LD /Fesqlite3.dll /link
then
coffimplib.exe sqlite3.lib -f
and used resulting .lib and .dll files.

But it works with 64bit version (used static lib here)
32bit version without -m32mscoff

Can you give your sc.ini for -m32mscoff?

@biozic
Copy link
Collaborator

biozic commented Jan 12, 2016

I guess the sc.ini file is the default one, because I just downloaded DMD from the website and compiled. Actually, I linked my home-built sqlite3 statically, not as a DLL. If it's ok with static libs but not with DLLs, I'm afraid it's beyond my knowledge of the Windows toolchain...

@biozic biozic added Enhancement Unsure Needs confirmation or more info and removed Enhancement labels Sep 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Unsure Needs confirmation or more info
Projects
None yet
Development

No branches or pull requests

2 participants