| 1 May 2026 |
Sergei Zimmerman (xokdvium) | I thought it was an unmerged patchset? | 11:24:35 |
aloisw | Not sure if it's the exact same API, but this looks like the entry point to the thing: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/tree/io_uring/register.c?h=v7.0.3#n1014 | 11:26:15 |
Sergei Zimmerman (xokdvium) | Oh great | 11:32:28 |
| llakala set a profile picture. | 17:12:38 |
| 2 May 2026 |
Sergei Zimmerman (xokdvium) | Noticed one thing here:
https://git.lix.systems/lix-project/lix/src/commit/cd573beb0a60d6bf0e341bcd4d2c464ed97c1a8a/lix/libstore/sqlite.cc#L186-L190
The reasoning there doesn't seem entirely correct since per sqlite docs NUL bytes anywhere in the string are UB?
If a non-negative fourth parameter is provided to sqlite3_bind_text() or sqlite3_bind_text16() or sqlite3_bind_text64() then that parameter must be the byte offset where the NUL terminator would occur assuming the string were NUL terminated. If any NUL characters occur at byte offsets less than the value of the fourth parameter then the resulting string value will contain embedded NULs. The result of expressions involving strings with embedded NULs is undefined.
| 01:14:56 |
aloisw | That's not my interpretation of "result of expressions involving strings with embedded NULs is undefined". To me it seems that storing or retrieving these strings is completely fine, but if you put it into a SQL expression the behaviour is unspecified in C standards language. | 05:10:20 |
zoë (@blokyk) | there's a docs page dedicated to this:
[...] The use of NUL within strings can lead to surprising behaviors:
-
The length() SQL function only counts characters up to and excluding the first NUL.
-
The quote() SQL function only shows characters up to and excluding the first NUL.
-
The .dump command in the CLI omits the first NUL character and all subsequent text in the SQL output that it generates. In fact, the CLI omits everything past the first NUL character in all contexts.
The use of NUL characters in SQL text strings is not recommended.
| 07:25:16 |
aloisw |
This seems like a bug. But it is how SQLite works.
Lol, that's quite a formulation for "we didn't put the effort into making it work in the CLI". | 07:29:19 |
| Benjamin Hartmann joined the room. | 14:06:27 |
| 3 May 2026 |
| Daniel joined the room. | 09:53:44 |
| isabel changed their profile picture. | 10:39:01 |
| 16 May 2024 |
| zrsk joined the room. | 13:54:49 |
samrose | In reply to @lunaphied:lunaphied.me I think there were a few CLs on the Gerrit but nothing being actively worked The other thing that I could do if it helps is test things and try to find bugs. I did do some C++ work in the past, but may lack the time to do it justice here at least for about 30 days or so | 15:55:29 |
Qyriad | we are not in any rush 🙂 | 17:20:53 |
samrose | Would it help to also test out the existing Lix code and try to find issues/bugs etc? | 17:23:21 |
Qyriad | absolutely | 17:23:41 |
samrose |
- how do people feel about the existing test suite that comes along with nix source code or Lix?
| 17:23:48 |
Qyriad | it's pitiful | 17:24:10 |
samrose | heh | 17:24:16 |
raitobezarius | expanding it is cool | 17:24:23 |
raitobezarius | writing new tests for builtins which are not tested | 17:24:30 |
raitobezarius | new test behaviors, etc. | 17:24:32 |
Qyriad | we have three flavors of test:
gtest (offer only available in libexpr and libutil) bash script virtual machine
the vast, vast majority of testing is in the "bash script" flavor and it is a mess | 17:25:01 |
samrose | I was just going to ask on the "functional" tests: do we still like using bash there? | 17:25:56 |
samrose | the last time that I worked on a major nix related cli project that used bash, or bats for testing, over time it became rather kind of hard to maintain | 17:26:42 |
samrose | I am not usually a big python fan, but in that project we heard from some in the Rust community that they actually use Python to test CLI and seem to have success there. | 17:27:59 |
Qyriad | no gods please kill bash testing. the problem is that it's kind of really difficult to migrate an entire test suite and be sure that you actually migrated the test suite correctly and won't lose coverage accidentally in the process, which makes any kind of migration a bit nerve wracking | 17:28:36 |
samrose | yes it's a rather large undertaking | 17:28:52 |
samrose | could be done in chunks | 17:28:59 |
Qyriad | at the very least though, it would be wonderful to have an option for new tests to not just be a mess of bash | 17:29:17 |