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

notice: use "wd deploy --check" instead of "check='yes' wd deploy" since gem version 0.5.2 #3

Closed
rick opened this issue Dec 24, 2010 · 1 comment

Comments

@rick
Copy link
Member

rick commented Dec 24, 2010

tl;dr: use "wd deploy --check" instead of "check='yes' wd deploy" since gem version 0.5.2

This isn't an open issue so much as a notice about a behavior change that may be surprising to some users.

Basically if you've been doing automated deploys with:

check='yes' wd deploy --to=: ...

As of this commit (which means, as of gem version 0.5.2, released back on September 6, 2010):
158845c

You will want to instead do:

wd deploy --check --to=: ...

Otherwise staleness checking will not run.

Note that all the internal wd machinery is a set of 4 rake tasks which use environment variables as controls, and then which call the WhiskeyDisk ruby libraries. The 'wd' command-line binary is just an args processor which sets environment variables and calls the wd rake tasks.

Setting 'check="yes"' on the command-line is sufficient to turn on staleness checking when running the whiskey_disk rake tasks. Setting that variable and running the 'wd' command (with or without --check specified) will not, however, pass along the environment setting for "check" to the rake tasks, as the wd binary sets 'check' in the environment based on the presence of absence of the --check flag -- so whatever value is specified in the enclosing environment will always be overwritten.

I'm inclined to think this is the desired behavior, it's just a bit surprising for folks who upgraded through the 0.5.2 release. Hence the "issue".

@rick
Copy link
Member Author

rick commented Mar 8, 2011

Closing, as this is getting pretty old as a "heads up" notice.

This issue was closed.
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

1 participant