Skip to content

Remove the VS solution as it is outdated. #166

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

Merged
merged 1 commit into from
Apr 1, 2015

Conversation

larshg
Copy link
Contributor

@larshg larshg commented Feb 12, 2015

Updated README for now.

I might look into a more detailed explanation for dependencies and the like.

@floe
Copy link
Contributor

floe commented Feb 17, 2015

Can you currently use CMake on Windows to generate a VS solution?

@larshg
Copy link
Contributor Author

larshg commented Feb 17, 2015

You can yes - but it doesn't find all the dependencies and mine look like this:
master

This should change with #68 merged. As for now, you can insert all the paths/libs manually.

But if I use the so called VS solution that already exist - this is what I have:
mastervssolution

Which doesn't eveny build the libfreenect2 project and hence, the protonect project won't work at all. And all the paths in the solution is hardcoded and doens't match mine etc.

@floe
Copy link
Contributor

floe commented Apr 1, 2015

As nobody has touched the VS solution since, it's even more outdated by now, so let's get rid of it. Merging.

floe added a commit that referenced this pull request Apr 1, 2015
Remove the VS solution as it is outdated.
@floe floe merged commit 2794f3c into OpenKinect:master Apr 1, 2015
@larshg larshg deleted the VSSolutionRemove branch April 8, 2015 11:22
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