Re: [PATCH 4/4] cfg.mk: Always enclose syntax-check error messages in quotes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Apr 11, 2016 at 17:32:46 +0200, Andrea Bolognani wrote:
> To prevent the error messages in cfg.mk from triggering the very
> same rules they're supposed to explain, we split the message in
> the middle of a symbol name.
> 
> In some cases we end up with something like
> 
>   'I am a me'ssage
> 
> However, using a little more punctuation and ensuring the message
> is properly enclosed in quotes is nicer.
> ---
> A more proper fix is arguably to exclude cfg.mk from these specific
> checks... It's not like we're going to use asprintf() in a Makefile
> anyway.

I've actually thought it's done this way ...

>  cfg.mk | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/cfg.mk b/cfg.mk
> index cd3b515..15e7c40 100644
> --- a/cfg.mk
> +++ b/cfg.mk
> @@ -379,7 +379,7 @@ sc_prohibit_strtol:
>  # But for plain %s, virAsprintf is overkill compared to strdup.
>  sc_prohibit_asprintf:
>  	@prohibit='\<v?a[s]printf\>'					\
> -	halt='use virAsprintf, not as'printf				\
> +	halt='use virAsprintf, not as''printf'				\

... until I saw this stuff. Yuck.

ACK though since it works.

Attachment: signature.asc
Description: Digital signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]