[libvirt] [PATCHv2] maint: avoid false positives on unmarked diagnostics

Stefan Berger stefanb at linux.vnet.ibm.com
Tue May 1 14:46:05 UTC 2012


On 05/01/2012 10:44 AM, Eric Blake wrote:
> Otherwise, a string such as _("Don't use \"" VAR "\".") would
> complain about unmarked diagnostics.
>
> * cfg.mk (sc_libvirt_unmarked_diagnostics): Handle \" in message.
> ---
>
> v2: fix regex to skip all \ escape sequences, and put \+ in right place.
>
>   cfg.mk |    2 +-
>   1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/cfg.mk b/cfg.mk
> index fb4df2f..9dab3c3 100644
> --- a/cfg.mk
> +++ b/cfg.mk
> @@ -603,7 +603,7 @@ sc_libvirt_unmarked_diagnostics:
>   	  $(_sc_search_regexp)
>   	@{ grep     -nE '\<$(func_re) *\(.*;$$' $$($(VC_LIST_EXCEPT));   \
>   	   grep -A1 -nE '\<$(func_re) *\(.*,$$' $$($(VC_LIST_EXCEPT)); } \
> -	   | sed 's/_("[^"][^"]*"//;s/[	 ]"%s"//'			\
> +	   | sed 's/_("\([^\"]\|\\.\)\+"//;s/[	 ]"%s"//'		\
>   	   | grep '[	 ]"'&&						\
>   	  { echo '$(ME): found unmarked diagnostic(s)' 1>&2;		\
>   	    exit 1; } || :

ACK




More information about the libvir-list mailing list