facebookresearch / vrs

VRS is a file format optimized to record & playback streams of sensor data, such as images, audio samples, and any other discrete sensors (IMU, temperature, etc), stored in per-device streams of timestamped records.
https://facebookresearch.github.io/vrs/
Apache License 2.0
308 stars 53 forks source link

Undefined Reference to aio_write and aio_return When Building with Clang #162

Open jianxiapyh opened 1 week ago

jianxiapyh commented 1 week ago

Instructions To Reproduce the 🐛 Bug:

  1. Clone the repository
  2. Install dependencies
  3. Generate build files using CMake with Ninja: cmake -S . -B build/ -G Ninja
  4. Build the project using Ninja ninja -C build/

Full logs or other relevant observations:

undefined reference to `aio_return' 
undefined reference to `aio_write' 

in building sample_playback_app, sample_recording_app, test_vrslib, test_vrslib_file_tests, vrs_utils

System Ubuntu 20.04 Clang 10.0.0 Ninja 1.10.0

My Fix I manually edited the CMakeLists.txt files to add the rt library to the target_link_libraries command for the affected targets to make it compile

jianxiapyh commented 1 week ago

After building and running ctest, the following test fails: test 12: StringsHelpersTester.readUInt64

SeaOtocinclus commented 1 day ago

Should be fixed by https://github.com/facebookresearch/vrs/commit/1755b5d833c10fb1e26718f9564e6bf9d86223a5