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

"Deploy directory 'dist' is empty" but it isn't #17

Open
MoOx opened this issue Oct 29, 2015 · 4 comments
Open

"Deploy directory 'dist' is empty" but it isn't #17

MoOx opened this issue Oct 29, 2015 · 4 comments

Comments

@MoOx
Copy link
Contributor

MoOx commented Oct 29, 2015

I just grabbed the latest version of your script and I got this issue.
It guess the script need to be at the same level of dist?

~/Sync/Development/statinamic/demo master* ⇡./scripts/deploy.sh 
Deploy directory 'dist' is empty. Aborting. If you're sure you want to deploy an empty tree, use the --allow-empty / -e flag.

~/Sync/Development/statinamic/demo master* ⇡
❯❯ ls dist 
collection.json       docs                  index.html            index.json            statinamic-client.css statinamic-client.js
@MoOx
Copy link
Contributor Author

MoOx commented Oct 29, 2015

I opened the script to try to fix that and saw ls --almost-all

~/Sync/Development/statinamic/demo master* ⇡
❯❯ ls --almost-all dist
ls: illegal option -- -

What am I doing wrong? (OS X 10.11.1)

@MoOx
Copy link
Contributor Author

MoOx commented Oct 29, 2015

Not sure why this option is not working for me. I changed the syntax to ls -A and it is working now. Maybe you should do the same in your script?

@X1011
Copy link
Owner

X1011 commented Oct 29, 2015

ok, it seems the ls on OS X is from BSD and doesn't support long options. so i should:

  • revert to using the short form of the option, -A
  • make sure errors in command substitutions / conditionals are propagated properly
  • see if i can have my CI run tests on OS X or BSD, too

@MoOx
Copy link
Contributor Author

MoOx commented Oct 29, 2015

great plan!

X1011 added a commit that referenced this issue Nov 6, 2015
`ls` on those systems only recognizes short form flags
fixes main problem of #17
X1011 added a commit that referenced this issue Dec 15, 2015
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