qtrpi icon indicating copy to clipboard operation
qtrpi copied to clipboard

[WIP] Qt 5.12.9

Open MoJo2600 opened this issue 3 years ago • 5 comments

I'm trying to compile welle.io for raspberry pi and stumbled upon this great project. But I want to compile it with 5.12 LTS. I adjusted your code to work, but right now I'm only focusing on Raspbery Pi 3, only full init and disregarded backward compatibility right now, hence WIP.

My current status is:

  • Upgraded Crosscompiler to lenaro 7.5 (I have to upgrade the download script)
  • Upgraded raspi image to buster
  • Qt 5.12.9 builds successfully
  • Qt modules build successfully

Still working on it

MoJo2600 avatar Jan 23 '21 10:01 MoJo2600

Hello Christian,

Wow thanks a lot for this awesome contribution! I've skimmed through your changes and it looks very promising. Ping me again when you have stabilized your changes and I'll gladly review and test your PR.

We might have some back and forth about a few fixes but I definitely want to merge your work :+1: !

synapticvoid avatar Jan 24 '21 15:01 synapticvoid

Yes, it is very crude right now, as I just figured out how everything is working. Have to do some cleanup later. I managed to build my intended application welle.io and it is running on my raspberry so the build is definitely working.

MoJo2600 avatar Jan 24 '21 19:01 MoJo2600

Okay, i did some cleanup and removed the requirements for welle.io. I tested it with rpi3 and 5.12.9. I will try to compile the old QT versions tomorrow. But I think we can start to talk about the fixes you mentioned.

I'd like to hear your opinion about docker builds, but i moved this to an issue so maybe I can add this after the merge.

MoJo2600 avatar Jan 24 '21 20:01 MoJo2600

Alright, I've reviewed the change set. If you could do some minor tweaks, it would be perfect:

  • Add the executable permission to docker/build.sh
  • Update the README.md with the commands to build the Docker image
  • Remove the -j flags on the make commands. It was my mistake, but it's actually useless, because omitting it will not limit the number of jobs

After that, I'll merge your PR and I'll take the time to clean up the repo to refer to the Docker build as the main one. Again, thanks for sharing your work :)

synapticvoid avatar Feb 02 '21 16:02 synapticvoid

I will update the README tomorrow

MoJo2600 avatar Feb 02 '21 21:02 MoJo2600