Skip to content

JENKINS-23430 - option to make grailsw executable#11

Open
beccagaspard wants to merge 1 commit intojenkinsci:masterfrom
beccagaspard:JENKINS-23430
Open

JENKINS-23430 - option to make grailsw executable#11
beccagaspard wants to merge 1 commit intojenkinsci:masterfrom
beccagaspard:JENKINS-23430

Conversation

@beccagaspard
Copy link
Copy Markdown

Provide option to make grailsw executable when using grails wrapper.

@cloudbees-pull-request-builder
Copy link
Copy Markdown

plugins » grails-plugin #17 SUCCESS
This pull request looks good

@beccagaspard
Copy link
Copy Markdown
Author

@beccagaspard beccagaspard changed the title JENKINS-23430 JENKINS-23430 - option to make grailsw executable Mar 3, 2015
@kjarnet
Copy link
Copy Markdown

kjarnet commented Aug 13, 2015

As a workaround, one can add a buildstep before executing the build that does this:
Add build step -> Execut shell command -> chmod a+x ${WORKSPACE}/grailsw

@mbidewell
Copy link
Copy Markdown

That is a workaround as long as the build box is Unix-based. The proper fix should really be made.

@robertoschwald
Copy link
Copy Markdown

robertoschwald commented Oct 18, 2017

The PR causes a NPE if installing plugin version on jenkins with existing projects, since makeExecutable Boolean is null in this case and unboxing to boolean in if (makeExecutable){} happens.

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.

5 participants