Introducing Miroil

greyback

Hey folks,
I can make this slightly easier with a few handy env vars.

sudo ln -s /usr/local/lib/qt5/plugins/platforms/libqpa-mirserver.so /usr/lib/x86_64-linux-gnu/qt5/plugins/platforms/

Allows Qt to find the platform.

You can avoid this soft link by setting this env var instead:

export QT_QPA_PLATFORM_PLUGIN_PATH=/usr/local/lib/qt5/plugins/platforms

@alan_g said in Introducing Miroil:

So Let's make that easier to find too:
sudo ln -s /usr/local/lib/qt5/qml/Unity/ /usr/lib/x86_64-linux-gnu/qt5/qml/

Similarly, you can avoid the soft link again and point Qt directly to where those QML plugins live with:

export QML2_IMPORT_PATH=/usr/local/lib/qt5/qml

Hope this helps
-G

Aury88

during the QtMir build with make -j 6 all test I also had the fatal error described by @jonius :

[ 87%] Building CXX object tests/mirserver/ScreensModel/CMakeFiles/ScreensModelTest.dir/screensmodel_test.cpp.o
In file included from /home/aurelio/qtmir/src/modules/Unity/Application/mirsurface.h:35:0,
                 from /home/aurelio/qtmir/tests/modules/SurfaceManager/surface_manager_test.cpp:22:
/home/aurelio/qtmir/src/modules/Unity/Application/mirbuffersgtexture.h:20:30: fatal error: miroil/mirbuffer.h: Not existing file or directory 
compilation terminated.
tests/modules/SurfaceManager/CMakeFiles/surfacemanager_test.dir/build.make:62:  instruction set  for the target "tests/modules/SurfaceManager/CMakeFiles/surfacemanager_test.dir/surface_manager_test.cpp.o" failed
make[2]: *** [tests/modules/SurfaceManager/CMakeFiles/surfacemanager_test.dir/surface_manager_test.cpp.o] Error 1
CMakeFiles/Makefile2:2146:  instruction set  for the target "tests/modules/SurfaceManager/CMakeFiles/surfacemanager_test.dir/all" failed
make[1]: *** [tests/modules/SurfaceManager/CMakeFiles/surfacemanager_test.dir/all] Error 2
make[1]: *** waiting incomplete process....

and after that

[ 95%] Built target applicationmanager_test
Makefile:138: instruction set  for the target  "all" failed
make: *** [all] Error 2

alan_g

@jonius can you PR your fix? It looks to tidy up the cmake code too.

alan_g

@aury88 I'm not sure when I'll be at a computer where I can fix this. In the meantime you could google enough git magic to cherry-pick @jonius's fix.

UPDATE: I've pushed a fix for the build error. There's still the test failure noted above.

@Aury88 You can get the fix with git pull

jonius

@alan_g does that mean no need for a PR from my side?

Aury88

@jonius i tried again after the git pull but obtain the same error

alan_g

@aury88 said in Introducing Miroil:

@jonius i tried again after the git pull but obtain the same error

Sorry, you'll need a git fetch first.

alan_g

@jonius said in Introducing Miroil:

@alan_g does that mean no need for a PR from my side?

Well, I've fixed the compile error, but you modernized the code as well. I think that's still worth a PR.

Aury88

@alan_g
Oh! so I totally misunderstood that command...reading the documentation seemed to me git pull already encompass a git fetch and a git merge. also I tried the git pull after the make command and that confirmed the local branch was up-to-date. ok, i will try again after the fetch. thank you.

alan_g

@jonius said in Introducing Miroil:

@jonius next issue is a failing test (ApplicationManager), which fails here.

I had a look over the weekend. The test is failing in the branch I based this off, and I didn't see an obvious cause for it.

I probably need to find a better base branch for this work. I really find all these QtMir branches a mystery. Maybe the best one is one of those in the gitlab repo?