Skip to content
This repository has been archived by the owner on May 14, 2021. It is now read-only.

Correct bug in example relating to git usage #90

Closed
wants to merge 1 commit into from
Closed

Correct bug in example relating to git usage #90

wants to merge 1 commit into from

Conversation

JCotton1123
Copy link

The git resource uses the name as the default location to checkout the code.

The git resource uses the name as the default location to checkout the code.
@coderanger
Copy link
Member

No, this is correct if you use application_git. It does some fancy magic for the shortcut.

@coderanger coderanger closed this Jan 18, 2017
@JCotton1123
Copy link
Author

Interesting. Seems like the magic is broken in some way b/c I followed the example to the t and it resulted in an error with the below resource def (notice the destination):

    git("git@github.com:turnstylecycle/themedream.git") do
      action [:sync]
      retries 0
      retry_delay 2
      default_guard_interpreter :default
      destination "git@github.com:turnstylecycle/themedream.git"
      enable_checkout true
      revision "HEAD"
      remote "origin"
      checkout_branch "deploy"
      declared_type :git
      cookbook_name "turnstyle-appserver"
      recipe_name "_themedream"
    end

@coderanger
Copy link
Member

Add depends 'application_git' to your cookbook metadata. You would see the resource name in there be application_git if it was enabled (though it wouldn't error so you wouldn't see it :).

@JCotton1123
Copy link
Author

Thanks. It might be worth adding this as a note to README.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants