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

No app should install anything to root #41

Open
danielb2 opened this issue Feb 5, 2015 · 6 comments
Open

No app should install anything to root #41

danielb2 opened this issue Feb 5, 2015 · 6 comments

Comments

@danielb2
Copy link

danielb2 commented Feb 5, 2015

/boardtextures/ and /sounds/ is violating this. Any reason to do this?

@pzorin
Copy link
Owner

pzorin commented Feb 6, 2015

Works for me on Ubuntu 14.10. On which platform do you see this behaviour?

@danielb2
Copy link
Author

danielb2 commented Feb 6, 2015

OSX installed it to root

@pzorin
Copy link
Owner

pzorin commented Feb 6, 2015

I suspected that, the variables QGO_INSTALL_PATH and QGO_INSTALL_BIN_PATH in qgo/src.pro are not set for mac os x. You can try setting them to whatever is appropriate. I can't be of much help since I do not use mac os and don't know its conventions.

@danielb2
Copy link
Author

danielb2 commented Feb 6, 2015

Follows the same conventions as FreeBSD and most UNIX. It should be installed in /usr/local

Does that help?

@pzorin
Copy link
Owner

pzorin commented Feb 7, 2015

You can try replacing
unix_:!macx-_ {
in src.pro by
macx-* {
Then everything will be installed to /usr/share. It seems however that qt applications are supposed to be deployed as something called "bundles" on mac os, and I am not really interested in learning how that works.

@danielb2
Copy link
Author

danielb2 commented Feb 7, 2015

When the app is compiled, the result is qgo.app. It appears to be the whole self-contained app already. I deleted the other folders which had no noticeable impact, but I couldn't connect to a server and play so I couldn't tell for sure. Regardless, this is probably the "bundle" it's referring to.

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

No branches or pull requests

2 participants