On 9/19/19 10:33 PM, Cole Robinson wrote:
> Trailing comma is not valid json:
>
> $ cat contrib/vhost-user-gpu/50-qemu-gpu.json.in | jq
> parse error: Expected another key-value pair at line 5, column 1
Shouldn't we add this as a test?
> Signed-off-by: Cole Robinson <crobi...@redhat.com>
> ---
> contrib/vhost-user-gpu/50-qemu-gpu.json.in | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/contrib/vhost-user-gpu/50-qemu-gpu.json.in
> b/contrib/vhost-user-gpu/50-qemu-gpu.json.in
> index 658b545864..f5edd097f8 100644
> --- a/contrib/vhost-user-gpu/50-qemu-gpu.json.in
> +++ b/contrib/vhost-user-gpu/50-qemu-gpu.json.in
> @@ -1,5 +1,5 @@
> {
> "description": "QEMU vhost-user-gpu",
> "type": "gpu",
> - "binary": "@libexecdir@/vhost-user-gpu",
> + "binary": "@libexecdir@/vhost-user-gpu"
> }
>
Reviewed-by: Philippe Mathieu-Daudé <phi...@redhat.com>
