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 FIND_PACKAGE feature #197

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

Add FIND_PACKAGE feature #197

wants to merge 1 commit into from

Conversation

piponazo
Copy link

@piponazo piponazo commented Feb 3, 2011

I usually use the FIND_PACKAGE cmake feature with my personal projects instead of pkg-config tool. In order to use libfreenect library in my personal code I've added this feature to my personal branch. If you are interested in this contribution, I would love that you include this in the main branch :).

I hope I have done nothing wrong, this is the first time I use git.

  means of cmake.
- Changed CMakeLists.txt in order to handle config.cmake.in file.
@qdot
Copy link
Member

qdot commented Feb 15, 2011

Nice work! However, the commit needs a sign off (not something you did wrong with git, it's just part of our project policy). Check out the contributing code document at

http://openkinect.org/wiki/Contributing_Code

And let me know if you have any questions about getting this going.

@piponazo
Copy link
Author

Thanks for your help qdot. After taking a look to the wiki page I've got this error message when I try to run the first command indicated in the "Forgetting to sign off after committing" section:

~$ git checkout -b signoff $(git merge-head origin/master)
git: 'merge-head' is not a git command. See 'git --help'.

I'm using Ubuntu 10.04, and the version of GIT that I've installed is 1.7.0.4.

@Goddard
Copy link

Goddard commented Mar 22, 2012

Is it libfreenect? because I built from source and using FIND_PACKAGE is giving me an error.

@piedar piedar force-pushed the master branch 2 times, most recently from e241863 to 4e489af Compare September 1, 2014 21:07
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.

None yet

3 participants