Skip to content

Comments

Arm#4

Open
fangchangma wants to merge 2 commits intoduo3d:masterfrom
fangchangma:arm
Open

Arm#4
fangchangma wants to merge 2 commits intoduo3d:masterfrom
fangchangma:arm

Conversation

@fangchangma
Copy link

It wasn't clear before that the master branch only works on x64 machines. This pull request has replaced the two binary libraries with the ones for ARM and has been tested on Odroid-XU4.

@fangchangma
Copy link
Author

@Danserminator I should have been more explicit: the code was tested on the Ubuntu 16.04 OS image released on ODROID Wiki, with a duo3d compiled against the linux kernel headers. Therefore, the driver might not work on other OS images (e.g. the DUO DDK OS Image).

However the solution should be simple: modify Line 36-37 of CMakeLists.txt, such that your driver is linked against the compatible DUO3D libraries at compilation. For example, if your local libDUO.so and libDense3DMT.so files are located under /home/odroid/somewhere, then change the last few lines of CMakeLists.txt to

target_link_libraries(duo3d_driver /home/odroid/somewhere/libDUO.so /home/odroid/somewhere/libDense3DMT.so ${catkin_LIBRARIES} )

@danielduberg
Copy link

@fangchangma Sorry I had made an error, it did work. Thank you so much!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants