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