Bug#863118: devscripts needs proper configuration file parsing

[Osamu Aoki]

control: tags -1 patch

> > > This is a hack, not a proper handling of the configuration.
> > 
> > fwiw, the maintainers have been in violent agreement with this for years,
> > but it needs someone with tuits to replace the current situation with a
> > solution that's usable from all the languages included in devscripts and
> > maintains the existing semantics (it's not just simple key=value pairs,
> > users are actively relying on being able to use shell within the file in
> > some cases, and that at least needs considering).

OK, here is my try to do the following.  Ugly, yes.  But it seems to
catch most errors in friendly way.

 * "egrep" test to catch space after "="
 * "set -e" to detect command not found; 
   * space before "="
   * "false" in line
 * "bash -n" test
 * Report the file name if a bug is found as much.

If no objection, I will apply this patch to git.  

Osamu

-------------- next part --------------
A non-text attachment was scrubbed...
Name: config.diff
Type: text/x-diff
Size: 1618 bytes
Desc: not available
URL: <http://lists.alioth.debian.org/pipermail/devscripts-devel/attachments/20170603/10a43b57/attachment.diff>