libelektra

libelektra copied to clipboard

quickdump: don't close stdout

2

elektraQuickdumpSet always(at the end or on failure) closed `file`, which makes sense, but `file` can be `stdout` in which case it should not be closed. Should be merge after #4469...

hannes99

CMake Release Parameters.

2

In #3530 @robaerd noted that `scripts/dev/configure-debian` is not suitable for our release automation. We should decide on some sane CMake parameters to be used for release builds. If there are...

mpranj

document all ways how to disable mmap

2

- [ ] for whole Elektra (with disabling PLUGINS) - [ ] for whole Elektra at run-time by unmounting the global plugin (does this work? Maybe we create a new...

markus2330

Test cases may fail if config files already exist

9

For example: - testshell_markdown_ccode and testshell_markdown_tcl fail if ~/.config/config.tcl is a non-TCL file - testshell_markdown_tutorial_validation fails if ~/.config/tutorial.dump contains user/tests/tutorial/spec/should_not_be_here - testshell_markdown_toml fails if ~/.config/test.toml exists Furthermore, the situation, that...

markus2330

ni: metadata: namespace

4

Is it on purpose that ni files now contain `meta:` in front of every metakey? E.g. ```sh kdb meta-set /sw/app/current/#0/server/port type short cat `kdb file spec:/sw/app/current/#0` ``` now yields: ```ini...

markus2330

mountpoint-info broken

7

``` Version: 0.9.3 Mountpoint: user:/sw/elektra/dq/#0 File: dq.ini config: sed: -e Ausdruck #1, Zeichen 36: Unbekannte Option für »s« sed: -e Ausdruck #1, Zeichen 36: Unbekannte Option für »s« sed: -e...

markus2330

reformatting failed due to src/plugins/specload/specload/remove.quickdump

6

https://build.libelektra.org/blue/organizations/jenkins/libelektra/detail/PR-3678/4/pipeline ``` 113/215 Test #131: testscr_check_formatting ......................***Failed 0.09 sec ELEKTRA CHECK FORMATTING ———————————————————————————————————————————————————————————— Warning — Reformatting command `reformat-c` failed ClangFormat: Version Info: 0.9.4 Major Version: Please install clang-format 11 ————————————————————————————————————————————————————————————...

markus2330

ksAppendKey locks name, ksPop does not unlock name

20

It makes sense that ksAppendKey() locks the key name, since it determines the position of the key in the keyset. However, I did not find an API function that clears...

mpranj

implement semantics of meta:/ keys

3

- [ ] no meta-data can be contained (as meta-meta data lives in spec:/) - [ ] no binary values are allowed (as they require meta-data) - [ ] value...

markus2330

Have a way to say that all errors in a function should be emitted as a warning

6

We should have a way to say that all errors in a function should be emitted as a warning. This was originally proposed by @kodebach and @markus2330 in this discussion:...

atmaxinger