Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add portable makefile definitions #65

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Add portable makefile definitions #65

wants to merge 1 commit into from

Conversation

suihkulokki
Copy link

This set of changes adds support for building OpenNI2 on most
Linux based architectures without tinkering.

This has also been submitted to debian:

https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=801123

Signed-off-by: Riku Voipio riku.voipio@linaro.org

This set of changes adds support for building OpenNI2 on most
Linux based architectures without tinkering.

Signed-off-by: Riku Voipio <riku.voipio@linaro.org>
@beeedy
Copy link

beeedy commented Apr 11, 2017

With minimal changes to this, I was able to compile/run on an Nvidia TX1

@fulkast
Copy link

fulkast commented May 13, 2017

@beeedy How did you do it?

@beeedy
Copy link

beeedy commented May 16, 2017

@fulkast I will go through my changes and compile a list to see exactly what I had to change in order to get this to run. I plan to get to this within the next day, and will get back to you then.

The only change I can recall at this very moment was to change the setting that makes compilation error out on unused variables. I will verify this and any other changes that were made.

@suihkulokki
Copy link
Author

suihkulokki commented May 17, 2017

Notice that if you have Debian or Ubuntu, this fix is already included in the openni2 packages. Just install with "apt install openni2-utils libopenni2-dev"

@beeedy
Copy link

beeedy commented May 17, 2017

Is there a more active codebase for OpenNI2 somewhere?

@suihkulokki
Copy link
Author

@beeedy The repository is at https://anonscm.debian.org/cgit/pkg-multimedia/openni2.git . But the changes you see there are mostly just packaging related. The actual codebase changes are in the set of patches at https://anonscm.debian.org/cgit/pkg-multimedia/openni2.git/tree/debian/patches

@beeedy
Copy link

beeedy commented May 18, 2017

@suihkulokki Verified that "apt install openni2-utils libopenni2-dev" works on a vanilla TX1 board. Thank you immensely

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.

3 participants