search

audinux / fedora-spec

Spec files for fedora packages
https://audinux.github.io
GNU General Public License v3.0
24 stars 7 forks source link

Make the JUCE package useful #14

Closed jpcima closed 2 years ago

jpcima commented 2 years ago

The JUCE package (current 7.0.1) can be simplified by using the cmake build+install procedure, that exists since JUCE 7.

Not only it's simpler, but it installs a hierarchy that is actually useful to a developer. One can create a project and type find_package(JUCE REQUIRED) in cmake to start right away. Current package does not provide a cmake find module.

@ycollet

ycollet commented 2 years ago

OK, I see. I will try to install the cmake package in a JUCE-devel package. I will check what has been done in the corresponding debian package too.

ycollet commented 2 years ago

The debian package git repo: https://salsa.debian.org/multimedia-team/juce/-/tree/master

ycollet commented 2 years ago

I updated juce to 7.0.2. I will build next version (or realease a new version of the package) where I will build everything with cmake and install the required files for cmake.

ycollet commented 2 years ago

Just pushed a new version. Can you check if it works for you ? I will try to enhqnce a little bit the spec latter: there is a JUCE-7.0.2 directory in /usr?bin with 2 executables. I will try to move this directory into /usr>lib64/libexec

jpcima commented 2 years ago

Nice, thank you. I will try this in a moment. One remark: the package description is outdated. The newer one has mention of LV2 support, and removed the mention of live-coding which is an abandoned feature.

jpcima commented 2 years ago

This creates a weird folder hierarchy cp -ra docs/doxygen/doc/ %{buildroot}/%{_datadir}/JUCE-%{version}/doc/

It gives: /usr/share/JUCE-7.0.2/doc/doc/

jpcima commented 2 years ago

Can you check if it works for you ?

Woking fine, when tried with a small example program. juce_hello.tar.gz

ycollet commented 2 years ago

Thanks a lot. I will fix the doc generation and probably put this doc into another package.

jpcima commented 2 years ago

You may want to consider this patch as well. It fixes the Juce's native font system that selects Italic by default. It's a bug which I reported upstream just now. You might have seen this italics problem happen before, and the "juce_hello" above also is affected. (example https://user-images.githubusercontent.com/1494920/183043746-0844b5dd-a3b5-4d3d-b1cb-6bdf780db60c.png)

JUCE-7.0.2-linux-font-default.patch.gz

ycollet commented 2 years ago

Thanks for the patch. I will add it