Klemens Böswirth
Klemens Böswirth
In theory `precommit` should implemented. Plugins are loaded from the config https://github.com/ElektraInitiative/libelektra/blob/5f0895bc88c92c623d9e2147cdaf12f77cb4db68/src/plugins/backend/backend.c#L202-L206 and executed https://github.com/ElektraInitiative/libelektra/blob/5f0895bc88c92c623d9e2147cdaf12f77cb4db68/src/plugins/backend/backend.c#L600-L603 by the `backend` plugin as well as `libelektra-kdb` https://github.com/ElektraInitiative/libelektra/blob/5f0895bc88c92c623d9e2147cdaf12f77cb4db68/src/libs/elektra/kdb.c#L2409-L2413 Your config also seems correct at...
Are you sure you're looking at the correct invocation of `loadPluginList` / the correct invocation of the `backend` plugin? It could be that you're looking at the version used during...
Okay, in that case I also don't have any ideas... I'd have to debug it myself
> EDIT: Okay, `parseAndAddMountpoint` parses correctly. After that, 19 other mountpoints are added without precommit - which makes a total of 23 mountpoints, where only 4 have the correct definition....
I'm not quite sure, I understand what you want to say. It should work like this: You mount the file `/tmp/hosts` at `user:/hosts` with the `hosts` plugin: ```sh kdb mount...
> What's especially strange is that it still seemed to work, as the host file was successfully written to and read from when using `kdb set user:/hosts/ipv4/www.google.com 8.8.8.8` even though...
@markus2330 I think you can now start to review this, but I'll make a few more updates tomorrow (simple test fixes, etc.).
@markus2330 If you continue your review before I push the next batch of updates, please don't focus too much on dead code. I know there is a lot of it...
@markus2330 I have now updated all the TODOs. The open questions are listed in #4521. Otherwise, there are some `TODO [new_backend]` and some `FIXME [new_backend]`. The `FIXME` are more urgent,...
@markus2330 This is now ready to merge IMO. The only open thing is the "commit function" decision. I will either update it, or replace it with a "kdb phases" decision...