search

aspyatkin / yarn-cookbook

Chef cookbook to install Yarn package manager
https://supermarket.chef.io/cookbooks/yarn
MIT License
5 stars 6 forks source link

Remove installation of apt-transport-https from repository recipe #5

Closed hrak closed 6 years ago

hrak commented 6 years ago

The installation of apt-transport-https is already taken care of by apt::default, and installing it from yarn::repository too triggers a resource cloning warning:

Cloning resource attributes for apt_package[apt-transport-https] from prior resource
Previous apt_package[apt-transport-https]: /var/chef/cache/cookbooks/apt/recipes/default.rb:96:in `from_file'
Current  apt_package[apt-transport-https]: /var/chef/cache/cookbooks/yarn/recipes/repository.rb:15:in `from_file' at 1 location:
    - /var/chef/cache/cookbooks/yarn/recipes/repository.rb:15:in `from_file'
   See https://docs.chef.io/deprecations_resource_cloning.html for further details.
aspyatkin commented 6 years ago

Hello, I'm a little pressed for time right now. Would you please consider submitting a pull request addressing the issue?

hrak commented 6 years ago

Sure, here you are!