[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PULL v2 6/9] configure: add GDBUS_CODEGEN
From: |
Peter Maydell |
Subject: |
Re: [PULL v2 6/9] configure: add GDBUS_CODEGEN |
Date: |
Fri, 10 Jan 2020 11:18:32 +0000 |
On Tue, 7 Jan 2020 at 16:40, Marc-André Lureau
<address@hidden> wrote:
>
> Hi
>
> On Tue, Jan 7, 2020 at 8:14 PM Peter Maydell <address@hidden> wrote:
> > I've just noticed that this seems to cause problems for
> > the NetBSD VM, because pkg-config reports a value for
> > gdbus_codegen which isn't an installed binary:
> >
> > localhost$ pkg-config --variable=gdbus_codegen gio-2.0
> > /usr/pkg/bin/gdbus-codegen
> > localhost$ ls /usr/pkg/bin/gdbus-codegen
> > ls: /usr/pkg/bin/gdbus-codegen: No such file or directory
> >
> > I think we need to install the 'gdbus-codegen' package in
> > our netbsd VM image.
> >
> > Maybe configure should check that it's actually got
> > an executable file before trying to use it?
>
> Sure, that should be enough:
>
> + if [ ! -x "$gdbus_codegen" ]; then
> + gdbus_codegen=
> + fi
>
> I can send the patch.
Hi -- did you send this patch? I can't find it in my mail archive.
thanks
-- PMM
- [PULL v2 0/9] Add dbus-vmstate, Marc-André Lureau, 2020/01/06
- [PULL v2 1/9] vmstate: add qom interface to get id, Marc-André Lureau, 2020/01/06
- [PULL v2 2/9] vmstate: replace DeviceState with VMStateIf, Marc-André Lureau, 2020/01/06
- [PULL v2 3/9] docs: start a document to describe D-Bus usage, Marc-André Lureau, 2020/01/06
- [PULL v2 4/9] util: add dbus helper unit, Marc-André Lureau, 2020/01/06
- [PULL v2 5/9] Add dbus-vmstate object, Marc-André Lureau, 2020/01/06
- [PULL v2 6/9] configure: add GDBUS_CODEGEN, Marc-André Lureau, 2020/01/06
- [PULL v2 7/9] dockerfiles: add dbus-daemon to some of latest distributions, Marc-André Lureau, 2020/01/06
- [PULL v2 8/9] tests: add migration-helpers unit, Marc-André Lureau, 2020/01/06
- [PULL v2 9/9] tests: add dbus-vmstate-test, Marc-André Lureau, 2020/01/06
- Re: [PULL v2 0/9] Add dbus-vmstate, Peter Maydell, 2020/01/07