Klemens Böswirth

Results 411 comments of Klemens Böswirth

Is it possible that our Alpine Linux builds are somehow broken? There clearly are images that are built and pushed to Docker Hub that have the new 0.9.9 version (e.g....

@markus2330 Why don't we just call `fsync()` in `resolver`? I assume it is because, there could be further modifications in `postcommit`. But (at least in `new-backend`) it would be pretty...

@markus2330 making it the resolvers obligation to sync isn't really off topic. The resolver is already required (without one nothing works), so by moving the sync there, it couldn't be...

It's somewhat done yes. But the setup in `new-backend` would actually fit better, if `kdbCommit` and `kdbError` are merged with `kdbSet`, since they are just phases of the `set` operation...

> Yes, we need to decide: either phases or separate functions. "separate functions" would also mean separate exports for `get-prestorage`, `get-storage`, `get-poststorage` etc. instead of a single `get`. AFAIK you...

> the two possibilities are what you later describe with "VS". To be clear, if we go with separate functions that would apply to e.g. `poststorage` too. I extended the...

Also discussed in this comment chain: https://github.com/ElektraInitiative/libelektra/pull/4484#discussion_r979453510 IMO we should undo the separately exported `commit` function and also remove the separately exported `error` function and merge both of them back...

Actually it is kind of urgent, so I know what to document in #4484, see also https://github.com/ElektraInitiative/libelektra/pull/4484#discussion_r984908094 (whole chain). I will of course document the current implementation, _but_ I'd like...

@markus2330 Can you please at least try to check what is already there, before delegating work to other people. `doc/decisions/commit_function.md` was already part of #4187. You even mentioned it in...

Should #3693 also move from the generic global plugin mechanism to specialised "hooks" for each of the currently existing global plugins? I think would make the implementation easier. For users...