-
Notifications
You must be signed in to change notification settings - Fork 1
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
fix permissions to run on binary distrib (linux/osx) #26
Comments
This also goes for griffonw script |
Binary distribution does not contain any griffonw, so I suppose you mean source distrib. And using source one, I think you should not call bundled ./griffonw, but /bin/griffon from the required griffon-1.1.0 package that should be already installed. |
Ok. chmod 755 in the source distro then. |
FYI griffonw allows you to run run Griffon apps without having a Griffon distribution installed in your system before hand. This is a feature we copied from Gradle, see http://griffon.codehaus.org/guide/latest/guide/theCommandLine.html#theGriffonWrapper for reference. |
thanks, I didn't know that until Jan mentioned it. I'll update the readme to reflect that. Any comment on the issue itself? Should premissions be on already on the zipped dist or is up to the user to do it after the installation? |
According to line 328 https://github.com/griffon/griffon/blob/master/subprojects/griffon-scripts/src/main/groovy/Package.groovy we already set permissions on non *.bat files. Is this not working for your package? |
No Clean Git checkout:
should be
|
Note that I package the dist on windows...not sure if it's relevant... |
I think we have two problems here:
I think this issue is about the second item, is it not? |
yes, this was about the second item. Jan just added the info about griffonw also lacking permissions, as both are pretty similar |
OK, so packing on Windows is still an issue as the Gant script does not apply executable permissions to files found under 'bin' ? |
No description provided.
The text was updated successfully, but these errors were encountered: