Re: UPDATE: audio/hydrogen

[...]

>> > Yes, I already have a (working but incomplete) diff for 1.0.0-beta1.
>> > It will need some more work on the midi part to support the new
>> > midi feedback feature.
>>
>> ack. Ports-wise this update looks good, except for a hidden dep on
>> cppunit, used to build src/tests/tests.
>>
>> --8<--
>> russell /usr/ports/pobj/hydrogen-0.9.7/build-amd64$ src/tests/tests
>> QCoreApplication::applicationDirPath: Please instantiate the QApplication object first
>> assertion "__instance" failed: file "/usr/ports/pobj/hydrogen-0.9.7/hydrogen-0.9.7/src/core/include/hydrogen/Preferences.h", line 238, function "get_instance"
>> Abort trap
>> -->8--
>>
>> So ok jca@ ports-wise if you add devel/cppunit to BUILD_DEPENDS (not
>> TEST_DEPENDS). Good luck if you want to fix tests. :)
>
> Instead of adding the dependency on cppunit, we can also disable building the
> tests by adding -DWANT_CPPUNIT=OFF to CONFIGURE_ARGS (diff below).
> (I plan to enable the tests in the next update, they do work in 1.0.0-beta)
>
> ok?

yep, ok jca@

--
jca | PGP : 0x1524E7EE / 5135 92C1 AD36 5293 2BDF DDCC 0DFA 74AE 1524 E7EE