search

jamulussoftware / jamulus

Jamulus enables musicians to perform real-time jam sessions over the internet.
https://jamulus.io
Other
1.01k stars 224 forks source link

Update Fedora dependency list on compiling.md #3366

Open stevie2sheds opened 2 months ago

stevie2sheds commented 2 months ago

Describe the bug

The current dependency list on compiling.md for Fedora 33+ is missing qt5-qtmultimedia-devel

To Reproduce

Using Fedora 40 you are unable to compile following the existing documentation for dependencies but can if you install qt5-qtmultimedia-devel

Expected behavior NA

Screenshots NA

Operating system Fedora 40

Version of Jamulus 3.10 client

Additional context It's just a minor documentation update required that may save someone else having to seek out the missing package.

jujudusud commented 2 months ago

Hi @stevie2sheds. Isn't fedora 40 using Qt version 6 ? If so, you should compile Jamulus against Qt6.

stevie2sheds commented 2 months ago

Ah. Probably. I'm afraid I'm not a great expert, having enough knowledge to try things, but not enough to always get them right! So I was just following the instructions. With that being the case the compiling instructions are out of date and in need of review if they're instructing the wrong Qt framework. The Qt5 compile did work.

With most OS's I've used the only reason to compile is because the latest version of Jamulus isn't available in the repositories, if at all. With that being the case the compiling instructions probably need to be pointing to the latest tools rather than legacy ones. If there are too many variations then unfortunately the instructions may have to be more generic. That would be a shame because the current simple instructions do work.