MFT: add macro for conversion digits to raw#2176
Conversation
davidrohr
left a comment
davidrohr
left a comment
There was a problem hiding this comment.
Looks good in principle.
After the CMake migration yesterday, all macros should be added explicitly to the tests in the CMake. Could you please have a look at macros/CMakeLists.txt
|
@bovulpes let me know if you need help editing the new CMakeLists.txt |
|
Thanks @aphecetche I just duplicated the ITS entries, I hope it works. |
|
@bovulpes It should. But note that you can also check locally whether it's working or not using either : ALIBUILD_O2_TESTS=1 aliBuild --defaults o2 build O2 or aliBuild --defaults o2 build O2 |
|
bash: ctest: command not found ?? |
|
If you don't have cmake installed locally (i.e. just have the one installed by aliBuild), then for the second option above you also need to load CMake/latest from alienv. |
|
OK thanks: 100% tests passed, 0 tests failed out of 16 |
* MFT: add macro for conversion digits to raw * MFT: add macro for decoding MFT raw data * MFT: add new macros to CMakeLists
* MFT: add macro for conversion digits to raw * MFT: add macro for decoding MFT raw data * MFT: add new macros to CMakeLists
* MFT: add macro for conversion digits to raw * MFT: add macro for decoding MFT raw data * MFT: add new macros to CMakeLists
3 participants
This is a clone of the corresponding macro for the ITS, adapted to the MFT read-out.