Bug#863118: devscripts needs proper configuration file parsing
James McCoy
jamessan at debian.org
Sat Jun 3 17:45:36 UTC 2017
On Sat, Jun 03, 2017 at 11:37:15PM +0900, Osamu Aoki wrote:
> 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.
The concept looks like a step in the right direction. However, uscan.pl
isn't the only script that does this type of config parsing.
Also, I would wait until after Stretch is released.
Cheers,
--
James
GPG Key: 4096R/91BF BF4D 6956 BD5D F7B7 2D23 DFE6 91AE 331B A3DB
More information about the devscripts-devel
mailing list