search

hydrogen-music / hydrogen

The advanced drum machine for Linux, macOS, and Windows
http://www.hydrogen-music.org
GNU General Public License v2.0
1.03k stars 172 forks source link

debug: add logs to debug #1820 #1832

Closed theGreatWhiteShark closed 1 year ago

theGreatWhiteShark commented 1 year ago

it seems the DiskWriterDriver is getting killed in the macOS pipeline before the song in TransportTests::testSampleConsistency was fully exported. I added some debug logs that might help determining why this is the case

cme commented 1 year ago

Would it be helpful to have access to a mac vm so you can run stuff like this at will rather than having to use Appveyor? I can sort you with one if it'd help.

theGreatWhiteShark commented 1 year ago

Would it be helpful to have access to a mac vm so you can run stuff like this at will rather than having to use Appveyor? I can sort you with one if it'd help.

Well, just for #1820 the current workflow is fine. At least as long as it feels like a bug in the macOS pipeline/unit tests rather than an actual Hydrogen bug. The priority is pretty low and I just tweak some log messages now and then in case the build fails.

But in general this might be a good idea as I just have Linux device and an old Windows workstation at hand. I would ping you when I'm in the need of it (or it turns out there are indeed problems with the AudioEngine on macOS).

TBH I'm kinda saturated with new technologies right now after looking into Flatpak and AppImage for the past weeks. I need to do some developing I intended to do for almost a year first :laughing: . Also, I do not want to stall the 1.2.2 patch release (which will be the test balloon for a Hydrogen.AppImage release generate in our build pipeline).