[...]

>> > 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

Reply via email to