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

Deployment scripts directory is mandatory #35

Closed
sophpie opened this issue Jun 19, 2014 · 4 comments
Closed

Deployment scripts directory is mandatory #35

sophpie opened this issue Jun 19, 2014 · 4 comments
Assignees
Labels

Comments

@sophpie
Copy link

sophpie commented Jun 19, 2014

Hi there,

If you don't provide any deployment scripts directory within deploiement.properties, first packaging will succeed but will throw a exception and then deployment will fail.

@slaff slaff added the bug label Jun 20, 2014
@slaff slaff self-assigned this Jun 20, 2014
@slaff
Copy link
Contributor

slaff commented Jun 23, 2014

Sophie, can you attach to this ticket sample deployment.properties and deployment.xml files where the error shows up?

@sophpie
Copy link
Author

sophpie commented Jun 23, 2014

These files can be packaged because you have a scriptdir data, but package
won't be deployable.
delete scriptdir and you won't be able to package at all.

On Mon, Jun 23, 2014 at 9:59 AM, slaff notifications@github.com wrote:

Sophie, can you attach to this ticket sample deployment.properties and
deployment.xml files where the error shows up?


Reply to this email directly or view it on GitHub
#35 (comment)
.

Sophie BeaupuisPS Consultant
Zend Technologies

105 rue Anatole France

92 300 Levallois-Perret

m.: +336 75 51 34 43
sophie.b@zend.com

slaff pushed a commit that referenced this issue Jun 23, 2014
…ployment.properties file and scriptsdir is present in deployment.xml.

Fix for issue #35.
@slaff
Copy link
Contributor

slaff commented Jun 23, 2014

The file phar from this commit 6af3b6a has the change in it.

@mkherlakian
Copy link

@sophpie please confirm that this fixes the problem you've been having so we can close the issue. Thanks!

@boedah boedah mentioned this issue Apr 14, 2015
@slaff slaff closed this as completed Apr 24, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants