[pacman-dev] [PATCH] libmakepkg/lint_pkgbuild: check for invalid variables even if they're empty

[Allan McRae]

On 13/6/18 11:17 am, Eli Schwartz wrote:
> Checking the length of the variable to be non-zero before considering it
> an error is inconsistent; license=() and depends='' and `declare arch`
> should be considered just as wrong.
> 
> In fact the current check detects depends='' as non-zero and returns an
> error, but happily considers the others to be perfectly okay.
> 
> A more reliable check is to simply see if the name has been declared
> (whether it is set or not), and then enforce that it's been declared to
> the right type.
> 
> As an added benefit, avoiding the creation of proxy-evaled variables to
> count the number of indexes results in simpler code.
> 
> Signed-off-by: Eli Schwartz <eschwartz at archlinux.org>

Looks good.   I'm 99% sure this was only supposed to be delayed being
committed until pacman-5.1.1 was done...

Allan