configglue team mailing list archive

[Ricardo Kirkner <ricardo.kirkner@xxxxxxxxxxxxx>]

The proposal to merge lp:~ricardokirkner/configglue/parse-default-dict-value into lp:configglue has been updated.

Description changed to:

Overview
========

Fix bug in parsing extra sections

Details
=======

While parsing the configuration, configglue looks for a special case in dicts and list of dicts, where the value itself is not a dict, but the names of sections that define the dicts.
There was a bug in that code where the default value was being considered as the section name, and thus making configglue fail.

The proposed solution is not the most elegant one, but it solves the problem at hand with a little amount of extra code (though, through the use of the ast module).

For more details, see:
https://code.launchpad.net/~ricardokirkner/configglue/parse-default-dict-value/+merge/65840
-- 
https://code.launchpad.net/~ricardokirkner/configglue/parse-default-dict-value/+merge/65840
Your team Configglue developers is requested to review the proposed merge of lp:~ricardokirkner/configglue/parse-default-dict-value into lp:configglue.