[PATCH] Warn when whitespace is missing before macro body

[Panu Matilainen]

On 01/21/2013 08:43 PM, Alexey Tourbin wrote:

Hi!

> This will now issue a warning when macro definition is possibly
> incorrect or ambigous, such as the one found in FC18 lvm2.spec:
>
> %define util-linux_version 2.22.1
> warning: Macro %util needs whitespace before body

Hmm. I like the idea, but the message is obscure (as of course is the 
existing behavior). I'd be ready to bet the first reaction of packagers 
seeing that would be filing "rpm macro parser has gone crazy" bugs 
instead of fixing their macro names :)

I think it'd be better just to sanity check the name at define/undefine 
time instead, eg change the COPYNAME() macro into a function which 
validates the whole name as it goes and errors out on invalid 
characters, or something to that effect.

P.S. In general, rpm-maint at lists.rpm.org is the more appropriate forum 
for patch submissions, this is more of a user-oriented list.

	- Panu -