config: add support for file-based configuration #122

dnkl commented 2022年04月12日 13:38:16 +02:00

This adds the boiler plate code necessary to handle file-based configurations:

• Lookup fuzzel.ini in XDG_CONFIG_HOME or XDG_CONFIG_DIRS
• Recognize valid option names
• Command line arguments override both defaults, and options set in fuzzel.ini
• Minor updates to existing API, only what was necessary to get things to compile

(削除) NOTE: does not yet work - values aren't (yet) loaded from the conf file. See TODO below. (削除ここまで)

This should now be ready for testing. See the bundled fuzzel.ini for available options.

TODO:

• Clean up APIs. Most objects should just take a pointer to a conf struct, instead of all their options as individual arguments
• Actually parse option values
• Documentation
• Changelog
• Add --config=PATH command line option

Closes #3

This adds the boiler plate code necessary to handle file-based configurations: * Lookup `fuzzel.ini` in `XDG_CONFIG_HOME` or `XDG_CONFIG_DIRS` * Recognize valid option names * Command line arguments override both defaults, and options set in `fuzzel.ini` * Minor updates to existing API, only what was necessary to get things to compile ~~**NOTE**: does not yet work - values aren't (yet) loaded from the conf file. See TODO below.~~ This should now be ready for testing. See the bundled `fuzzel.ini` for available options. **TODO**: * [x] Clean up APIs. Most objects should just take a pointer to a conf struct, instead of all their options as individual arguments * [x] Actually parse option **values** * [x] Documentation * [x] Changelog * [x] Add `--config=PATH` command line option Closes #3

markstos commented 2022年08月09日 16:06:43 +02:00

This is a great feature, thanks.

The one part of my wrapper script that doesn't translate is: --terminal "$TERMINAL -e".

This could be addressed by setting terminal to default to "$TERMINAL -e if $TERMINAL is set".

This is a great feature, thanks. The one part of my wrapper script that doesn't translate is: `--terminal "$TERMINAL -e"`. This could be addressed by setting terminal to default to "`$TERMINAL -e` if $TERMINAL is set".

dnkl commented 2022年08月09日 18:38:57 +02:00

This could be addressed by setting terminal to default to "$TERMINAL -e if $TERMINAL is set".

Like this: #156 ?

> This could be addressed by setting terminal to default to "$TERMINAL -e if $TERMINAL is set". Like this: https://codeberg.org/dnkl/fuzzel/pulls/156 ?