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

fix permissions to run on binary distrib (linux/osx) #26

Open
jmlucjav opened this issue Feb 25, 2013 · 11 comments
Open

fix permissions to run on binary distrib (linux/osx) #26

jmlucjav opened this issue Feb 25, 2013 · 11 comments

Comments

@jmlucjav
Copy link
Owner

No description provided.

@janhoy
Copy link

janhoy commented Feb 26, 2013

This also goes for griffonw script

@jmlucjav
Copy link
Owner Author

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.

@janhoy
Copy link

janhoy commented Feb 26, 2013

Ok. chmod 755 in the source distro then.
griffonw seems to work equally well as manually installing griffon first, so it's a faster solution.

@aalmiray
Copy link

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.

@jmlucjav
Copy link
Owner Author

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?

@aalmiray
Copy link

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?

@janhoy
Copy link

janhoy commented Feb 27, 2013

No

Clean Git checkout:

-rw-r--r--   1 janhoy  staff  5050 Feb 27 14:34 griffonw

should be

-rwxr-xr-x   1 janhoy  staff  5050 Feb 27 14:34 griffonw

@jmlucjav
Copy link
Owner Author

Note that I package the dist on windows...not sure if it's relevant...

@aalmiray
Copy link

I think we have two problems here:

  1. griffon distro does not assign executable permissions to griffonw. Filed and Fixed http://jira.codehaus.org/browse/GRIFFON-610
  2. zip package does not assign executable permissions to bin/vifun

I think this issue is about the second item, is it not?

@jmlucjav
Copy link
Owner Author

yes, this was about the second item. Jan just added the info about griffonw also lacking permissions, as both are pretty similar

@aalmiray
Copy link

OK, so packing on Windows is still an issue as the Gant script does not apply executable permissions to files found under 'bin' ?

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

3 participants