insilmaril / vym

View Your Mind
https://sourceforge.net/projects/vym/
Other
139 stars 33 forks source link

QSocketNotifier: Can only be used with threads started with QThread #92

Closed plutuplutu closed 10 months ago

plutuplutu commented 10 months ago

I'm getting this with build release and devel on up to date solus gnome and wayland: $ vym QSocketNotifier: Can only be used with threads started with QThread Trappe pour point d'arrêt et de trace (core dumped) and only core dump with Xorg. I found this post https://forum.qt.io/topic/29587/solved-qsocketnotifier-can-only-be-used-with-threads-started-with-qthread/8 but have no idea how to use it.

insilmaril commented 10 months ago

Hm, the core dump could be interesting, also building vym with debug flags, see also CMakeLists.txt:

# Switch on Debug mode with one of below:
# cmake -DCMAKE_BUILD_TYPE=Debug .
# set(CMAKE_BUILD_TYPE Debug)

But I somehow suspect this is related to the way Qt itself interacts with gnome and wayland. I have never seen this on 3 platforms (Win, Mac, Linux) so far... So if it crashes in Qt or xorg there is nothing I can do.

plutuplutu commented 10 months ago

If I uncomment # cmake -DCMAKE_BUILD_TYPE=Debug . I get $ cmake . CMake Error at CMakeLists.txt:27: Parse error. Expected "(", got unquoted argument with text "-DCMAKE_BUILD_TYPE=Debug".

When I uncomment the other line I don't see any changes even running vym -d. There's just a couple attention messages about the use of deprecated things during make.

insilmaril commented 10 months ago

It's two different ways to generate the debug build: First is the command line when running cmake:

cmake -DCMAKE_BUILD_TYPE=Debug .

The 2nd is to really remove comment and enable debug from within CMakeLists.txt

Either way maybe CMakeCache.txt may need to be removed, too.

insilmaril commented 10 months ago

Slightly modified documentation in 234d16bb921f08c24c8fb81c2f31f5e7fc7f90ff

joebonrichie commented 10 months ago

This was a packaging issue in Solus that made it's way to the stable repo. I believe this is resolved in our unstable repo now. Sorry for the noise!

insilmaril commented 10 months ago

Ah, thanks for the feedback!