[Qemu-devel] qemu is missing monitor

[Qemu-devel] qemu is missing monitor

[Jiri Slaby]

[-- Attachment #1: Type: text/plain, Size: 328 bytes --]

Hi guys,

qemu in opensuse latest (factory) does not have monitor. Despite
libvte-devel is installed, it is not detected by qemu. The problem is
that configure looks for vte-2.90, but we have vte-2.91 in factory. The
attached patch fixes that for factory, but it's dirty, of course.

Any better idea?

thanks,
-- 
js
suse labs


[-- Attachment #2: vte-2.91-for-factory.patch --]
[-- Type: application/mbox, Size: 685 bytes --]

Re: [Qemu-devel] qemu is missing monitor

[Peter Maydell]

On 9 October 2014 14:37, Jiri Slaby <jslaby@suse.cz> wrote:
> Hi guys,
>
> qemu in opensuse latest (factory) does not have monitor. Despite
> libvte-devel is installed, it is not detected by qemu. The problem is
> that configure looks for vte-2.90, but we have vte-2.91 in factory. The
> attached patch fixes that for factory, but it's dirty, of course.

Why the heck have the GTK folk made point releases end up
needing their own pkg-config library name??

-- PMM

Re: [Qemu-devel] qemu is missing monitor

[Gerd Hoffmann]

On Do, 2014-10-09 at 17:05 +0100, Peter Maydell wrote:
> On 9 October 2014 14:37, Jiri Slaby <jslaby@suse.cz> wrote:
> > Hi guys,
> >
> > qemu in opensuse latest (factory) does not have monitor. Despite
> > libvte-devel is installed, it is not detected by qemu. The problem is
> > that configure looks for vte-2.90, but we have vte-2.91 in factory. The
> > attached patch fixes that for factory, but it's dirty, of course.
> 
> Why the heck have the GTK folk made point releases end up
> needing their own pkg-config library name??

Good question.  They should not do that in the first place unless they
have a ABI change (i.e. gtk2 -> gtk3 switch was a good reason).  The
point release doesn't look like that though ...

I'd suggest to try sort that with vte upstream before trying to hack
around this in qemu (and if we have to we'll need to check for both 2.90
and 2.91 because you don't know what version the user has installed).

cheers,
  Gerd

Re: [Qemu-devel] qemu is missing monitor

[Jiri Slaby]

On 10/10/2014, 09:38 AM, Gerd Hoffmann wrote:
> On Do, 2014-10-09 at 17:05 +0100, Peter Maydell wrote:
>> On 9 October 2014 14:37, Jiri Slaby <jslaby@suse.cz> wrote:
>>> Hi guys,
>>>
>>> qemu in opensuse latest (factory) does not have monitor. Despite
>>> libvte-devel is installed, it is not detected by qemu. The problem is
>>> that configure looks for vte-2.90, but we have vte-2.91 in factory. The
>>> attached patch fixes that for factory, but it's dirty, of course.
>>
>> Why the heck have the GTK folk made point releases end up
>> needing their own pkg-config library name??
> 
> Good question.  They should not do that in the first place unless they
> have a ABI change (i.e. gtk2 -> gtk3 switch was a good reason).  The
> point release doesn't look like that though ...

Note, that qemu is terribly broken with 2.91. The monitor window is over
30 000 pixels in height. So there is perhaps some weird API/ABI change.
Built against 2.90, it works fine.

> I'd suggest to try sort that with vte upstream before trying to hack
> around this in qemu (and if we have to we'll need to check for both 2.90
> and 2.91 because you don't know what version the user has installed).

Ok, could you take care of that and sort that out?

-- 
js
suse labs

Re: [Qemu-devel] qemu is missing monitor

[Jiri Slaby]

On 10/09/2014, 06:05 PM, Peter Maydell wrote:
> On 9 October 2014 14:37, Jiri Slaby <jslaby@suse.cz> wrote:
>> Hi guys,
>>
>> qemu in opensuse latest (factory) does not have monitor. Despite
>> libvte-devel is installed, it is not detected by qemu. The problem is
>> that configure looks for vte-2.90, but we have vte-2.91 in factory. The
>> attached patch fixes that for factory, but it's dirty, of course.
> 
> Why the heck have the GTK folk made point releases end up
> needing their own pkg-config library name??

Don't ask me, /me wonders too. I am thinking of one of
* --enable-vte=2.91: we would have to do rpm -q vte-devel in .spec, but
that should work
* for ver in `seq 0 9`; do
    if pkg-config --exists vte-2.9$ver; then
      vteversion="vte-2.9$ver"
    fi
  done
* yell at the author of the versioning to retract and rethink!

thanks,
-- 
js
suse labs