[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#913316: marked as done (qtbase-opensource-src: Check why we are dlOpening() libGl instead of just linking it)

Your message dated Fri, 09 Nov 2018 18:54:59 -0300
with message-id <2145575.qmuRE5Ckld@tonks>
and subject line Re: Bug#913316: qtbase-opensource-src: Check why we are dlOpening() libGl instead of just linking it
has caused the Debian Bug report #913316,
regarding qtbase-opensource-src: Check why we are dlOpening() libGl instead of just linking it
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
913316: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=913316
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message --- 
Source: qtbase-opensource-src
Version: 5.10.1+dfsg-1
Severity: normal

Check https://codereview.qt-project.org/#/c/245064/ and it's comments.

The code seems to support linking, but somehow we don't do it.

-- System Information:
Debian Release: buster/sid
  APT prefers unstable
  APT policy: (990, 'unstable'), (500, 'unstable-debug'), (500, 'testing-debug'), (500, 'buildd-unstable'), (500, 'testing'), (500, 'stable'), (1, 'experimental-debug'), (1, 'experimental')
Architecture: amd64 (x86_64)
Foreign Architectures: i386, arm64, armhf

Kernel: Linux 4.18.0-2-amd64 (SMP w/2 CPU cores)
Locale: LANG=es_AR.UTF-8, LC_CTYPE=es_AR.UTF-8 (charmap=UTF-8), LANGUAGE=es_AR:es (charmap=UTF-8)
Shell: /bin/sh linked to /bin/bash
Init: systemd (via /run/systemd/system)
LSM: AppArmor: enabled

--- End Message ---
--- Begin Message --- 
As usual you are faster than me :-)


El viernes, 9 de noviembre de 2018 18:07:37 -03 Dmitry Shachnev escribió:
> On Fri, Nov 09, 2018 at 11:22:05AM -0300, Lisandro Damián Nicanor Pérez 
Meyer wrote:
> > Check https://codereview.qt-project.org/#/c/245064/ and it's comments.
> > 
> > The code seems to support linking, but somehow we don't do it.
> 
> Interesting that we *do* link to libGL:
> 
>   $ ldd
> /usr/lib/x86_64-linux-gnu/qt5/plugins/xcbglintegrations/libqxcb-glx-integra
> tion.so | grep libGL.so libGL.so.1 => /usr/lib/x86_64-linux-gnu/libGL.so.1
> (0x00007ffade5aa000)

Right, checked it a few hours later.

> Also the code does *not* support linking.
> 
> The code first tries dlopen(NULL, RTLD_LAZY). According to dlopen(3)
> manpage, if filename is NULL, then the returned handle is for the main
> program.
> 
> This does not work when the main program is Python — python interpreter is
> not linked against libGL.so.
> 
> When the first dlopen() call does not find the needed GL symbols, it tries
> to dlopen() libGL.so, via the QLibrary wrapper. That almost always
> succeeds, the patch in linked codereview should fix the remaining problems.

Right, Allan told me they do this because it allows them to check whether the 
libGL implementation has specific features. It it doesn't they simply discard 
it. Considering the video card landscape sounds like a nice thing to do.

> I think there is nothing for us to fix in *this* bug, and it can be closed.

Agree, and so closing now.

-- 
mathematician, n.:
        Some one who believes imaginary things appear right before your i's.

Lisandro Damián Nicanor Pérez Meyer
http://perezmeyer.com.ar/
http://perezmeyer.blogspot.com/

Attachment: signature.asc
Description: This is a digitally signed message part.


--- End Message ---
Reply to: