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

Adds server autostart functionality. #157

Merged
merged 3 commits into from
Jul 14, 2016
Merged

Adds server autostart functionality. #157

merged 3 commits into from
Jul 14, 2016

Conversation

DuFF14
Copy link
Member

@DuFF14 DuFF14 commented Apr 29, 2016

Note that at this time there is a bug preventing this from working unless an environment variable is changed:

For the environment variable OSVR_SERVER_ROOT, remove the trailing slash in the directory

@DuFF14
Copy link
Member Author

DuFF14 commented May 10, 2016

@JeroMiya any issues with merging this before the bug is fixed?

@JeroMiya
Copy link
Contributor

The only issue is that auto-start won't work properly if users run the installer more than once (or I think the latest installer since it adds a leading slash), however once the installer is fixed, the user just needs to run the latest installer to fix existing apps, so it's probably ok to merge this.

@DuFF14
Copy link
Member Author

DuFF14 commented May 24, 2016

@JeroMiya I've held off on merging this to make sure that it will work with Android. Keep me updated on any Android progress as it pertains to running Unity apps as I know you've been doing some work there.

@cobbchaseRazer
Copy link

@DuFF14 There is currently a larger discussion ongoing about the future of the Android platform. Please hold off on merging more auto-start functionality until we make progress on the discussion. @JeroMiya @leemichaelRazer

OSVR/OSVR-Core#433

@DuFF14
Copy link
Member Author

DuFF14 commented Jul 14, 2016

Restricted this to Standalone Windows platform only. @JeroMiya let me know when the OSVR_SERVER_ROOT fix is in place and then I'll merge. Thanks.

@DuFF14 DuFF14 merged commit 9e6d077 into master Jul 14, 2016
@DuFF14 DuFF14 deleted the serverAutostart branch July 14, 2016 21:20
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.

4 participants