Request for feedback

[vrischmann]

Hello everyone,

I'd like to get feedback from users of zig-sqlite, if anyone happens to look at issues and sees this.

I recently worked on improving the build script in the use-upstream branch.
The main change is that we leverage the zig package manager to fetch the sqlite amalgamation instead of vendoring the source code in this repository.

I'm looking for people that can test their programs with this branch and report back if everything works fine; for people not using the loadable extension feature everything should be transparent.

If you are using loadable extensions however, there is a breaking change: you need to use the sqliteext module in your build script instead of sqlite.
To summarize:

• loadable extensions => sqliteext
• everything else => sqlite

I'll wait a couple of weeks before merging the branch.

[lun-4]

is there any way this could be backported to the zig-0.13.0 branch? I pin my projects to stable zig versions nowadays

[vrischmann]

is there any way this could be backported to the zig-0.13.0 branch? I pin my projects to stable zig versions nowadays

It's probably doable, but I didn't really want to introduce a stable branch until zig 0.14.0, and given zig 0.14.0 is intended to be released soon-ish, I don't think I want to do this work.

However if anyone wants to work on this I'll happily review a PR.

[malcolmstill]

This looks to work fine for me with malcolmstill/clerk#43, just had to remove a exe.linkLibrary(sqlite.artifact("sqlite")); line from my build.zig.

[vrischmann]

This looks to work fine for me with malcolmstill/clerk#43, just had to remove a exe.linkLibrary(sqlite.artifact("sqlite")); line from my build.zig.

Nice, thank you for testing !

[vrischmann]

I'm kind of surprised that the artifact doesn't exist anymore, not sure exactly what causes it to be available in the first place so I'll look into it.

[vrischmann]

Nevermind, I found the issue, I removed the call to installArtifact for the library and forgot to add it back.

(but yeah, the linkLibrary call is not necessary, addImport is enough).