[pacman-dev] [PATCH 0/1] Simple config parser added to scripts library

lolilolicon lolilolicon at gmail.com
Fri Sep 30 07:37:47 EDT 2011


We're aware of many duplicates in our scripts, among which are things like

  eval $(awk '/DBPath/ {print $1$2$3}' @sysconfdir@/pacman.conf)

which is obviously utterly hackish and potentially dangerous, yet (shockingly)
quite a common practice in our scripts.  Trying to fix all these here and
there is a waste of time.

As suggested by Dave, I took the liberty to add a config parser to the scripts
library.  Currently there're only two functions conf_key_val and conf_key_set,
which are modified from pacman-key's get_from().  The usage is a bit different,
as described in the commit message.

If we agree on this change, feel free to port our scripts to use this.

lolilolicon (1):
  scripts/library: add config_parser.sh

 scripts/library/README           |    3 ++
 scripts/library/config_parser.sh |   56 ++++++++++++++++++++++++++++++++++++++
 2 files changed, 59 insertions(+), 0 deletions(-)
 create mode 100644 scripts/library/config_parser.sh

-- 
1.7.6.4



More information about the pacman-dev mailing list