| Message ID | 7f5dd2ac9f3504e2699f23e69bc3d8051b729832.1568925097.git.crobinso@redhat.com |
|---|---|
| State | New |
| Headers | show |
| Series | vhost-user-gpu: Drop trailing json comma | expand |
On Fri, Sep 20, 2019 at 12:34 AM Cole Robinson <crobinso@redhat.com> 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 > > Signed-off-by: Cole Robinson <crobinso@redhat.com> oh json.. Reviewed-by: Marc-André Lureau <marcandre.lureau@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" > } > -- > 2.23.0 > >
Cole Robinson <crobinso@redhat.com> 于2019年9月20日周五 上午4:34写道： > 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 > > Signed-off-by: Cole Robinson <crobinso@redhat.com> > Reviewed-by: Li Qiang <liq3ea@gmail.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" > } > -- > 2.23.0 > > >
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 <crobinso@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é <philmd@redhat.com>
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" }
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 Signed-off-by: Cole Robinson <crobinso@redhat.com> --- contrib/vhost-user-gpu/50-qemu-gpu.json.in | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)