Thomas Steiner

Results 420 comments of Thomas Steiner
trafficstars

@filipe-freire I'm not opposed, but am unsure where you'd add this, the caveat from https://github.com/sqlite/sqlite-wasm/issues/53#issuecomment-1827405950 still applies 🫣. To the thin wrapper code part of this repo (if so, please...

I don't know how merge-ready #54 is. Do people here generally approve of a not-perfect-but-good-enough merger? I think for the documentation, we should keep it vanilla. Those wanting to use...

@sgbeal I managed to get a transient database to work on Node. I had to patch one `self` to be `globalThis` and it ran, see the [changes](https://github.com/sqlite/sqlite-wasm/pull/48/files) (in [`sqlite3-node.mjs`](https://github.com/sqlite/sqlite-wasm/pull/48/files#diff-6c5604a4337107d1a1c56a4dab26374c13791df2f610b2a146be0853e8261a39) specifically)...

> Perhaps there's a way to tell Emscripten to use node's native i/o support? @sgbeal There's the [NODEFS](https://emscripten.org/docs/api_reference/Filesystem-API.html#nodefs) that should work. Have you looked at this? > It uses node’s...

According to the docs, it can be activated via the `-lnodefs.js` flag.

> The addition of `sqlite3InitModuleNode` to `index.mjs` in itself would be helpful for creating transient databases in node. I agree, but my fear is that people won't read the small...

I have just created a separate entry point for Node: `node.mjs`. This should allow interested parties to play with transient databases in Node, and at the same time work in...

> I made a small demo repo in case anyone is interested. It's using a fork of this branch where `"node": "./node.mjs",` is before `"import": "./index.mjs",` as mentioned above. >...

> But maybe it would be better if `node.mjs` default exported a function like `index.mjs` does to keep the same interface? Changed to this.

> This looks ready to go, anything that's being waited on? It's ready to go if you _don't_ need to persist your database, which makes me believe it's not ready...