Comment 1 Vladimir Druzenko 2025-01-26 23:09:58 UTC

Check if this patch is correct:
audio/libopenshot-audio/files/patch-JuceLibraryCode_modules_juce__audio__devices_native_juce__JackAudio__linux.cpp

Comment 2 Vladimir Druzenko 2025-02-03 18:54:14 UTC

ping

Created attachment 257283 [details]
patch for multimedia/openshot, audio/libopenshot-audio and multimedia/libopenshot

Since they are mutually related, I want them to be committed in one go.

audio/libopenshot-audio
JUCE version has been increased from 5 to 7.
MIDI seems to be unnecessary, so I made it unavailable in the patch.
It seems to be related to when alsa-seq-server(audio/alsa-seq-server) is running.
Since the singleton instance is destroyed by a different thread than the one that created it, it is detected on the libopenshot side.
The method of dealing with it is unknown :)
Dexed(audio/dexed)  may have the same issue, but it might just be a problem with how libopenshot uses threads.

multimedia/openshot
Since the conditions for acquiring video with the libx264 codec could not be found, it was rewritten to libx264rgb, but the problem seems to have been fixed, so the patch has been abolished.
Files related to setuptools are rewritten in a flat-layout style by the patch.
This suppresses repeated warning logs.
It seems that the copying of files in the configure phase has also been reduced.

Comment 4 Vladimir Druzenko 2025-02-06 16:57:35 UTC

I have 2 patches more:

files/patch-JuceLibraryCode_modules_juce__audio__devices_audio__io_juce__AudioIODeviceType.cpp:
--- JuceLibraryCode/modules/juce_audio_devices/audio_io/juce_AudioIODeviceType.cpp.orig 2024-12-21 22:21:33 UTC
+++ JuceLibraryCode/modules/juce_audio_devices/audio_io/juce_AudioIODeviceType.cpp
@@ -100,7 +100,7 @@ void AudioIODeviceType::callDeviceChangeListeners()
  AudioIODeviceType* AudioIODeviceType::createAudioIODeviceType_JACK()         { return nullptr; }
 #endif

-#if JUCE_LINUX && JUCE_BELA
+#if (JUCE_LINUX || JUCE_BSD) && JUCE_BELA
  AudioIODeviceType* AudioIODeviceType::createAudioIODeviceType_Bela()         { return new BelaAudioIODeviceType(); }
 #else
  AudioIODeviceType* AudioIODeviceType::createAudioIODeviceType_Bela()         { return nullptr; }
[root@phobos /usr/ports/audio/libopenshot-audio]# la files/patch-JuceLibraryCode_modules_juce__audio__devices_juce__audio__devices.cpp
1045666 -rw-r--r--  1 root wheel - 498 25 янв.  15:34:56 2025 files/patch-JuceLibraryCode_modules_juce__audio__devices_juce__audio__devices.cpp


files/patch-JuceLibraryCode_modules_juce__audio__devices_juce__audio__devices.cpp:
--- JuceLibraryCode/modules/juce_audio_devices/juce_audio_devices.cpp.orig      2024-12-21 22:21:33 UTC
+++ JuceLibraryCode/modules/juce_audio_devices/juce_audio_devices.cpp
@@ -177,7 +177,7 @@
   #include "native/juce_JackAudio_linux.cpp"
  #endif

- #if (JUCE_LINUX && JUCE_BELA)
+ #if ((JUCE_LINUX || JUCE_BSD) && JUCE_BELA)
   /* Got an include error here? If so, you've either not got the bela headers
      installed, or you've not got your paths set up correctly to find its header
      files.

Are you sure they are not needed?

Comment 5 Vladimir Druzenko 2025-02-06 17:05:09 UTC

AFAIU. patch files/patch-JuceLibraryCode_modules_juce__core_native_juce__SharedCode__posix.h not needed:
+    #if JUCE_BSD
+    using cpu_set_t = cpuset_t;
+    #endif
Build without it fine.

Comment 6 Vladimir Druzenko 2025-02-06 17:10:28 UTC

-        auto cpuData = call ("lscpu");
+        auto cpuData = String {};

Maybe: auto cpuData = call ("sysctl -n hw.model"); ?