search

sous-chefs / apt

Development repository for the apt cookbook
https://supermarket.chef.io/cookbooks/apt
Apache License 2.0
202 stars 266 forks source link

Add a warning on the README that https://github.com/chef/chef/pull/4782 supersedes this cookbook's `apt_repository` LWRP #204

Closed wk8 closed 7 years ago

jeremymv2 commented 8 years ago

@thommay Thoughts on users who are consuming this cookbook and using chef-client that has the apt_repository Resource built-in? When that is the case, they receive an exception: superclass mismatch for class AptRepository

grimm26 commented 8 years ago

It only affects me because it broke zap https://github.com/nvwls/zap/issues/33

poliva83 commented 8 years ago

I was hitting superclass mismatch for class AptRepository because my rake task was triggering a chef-client run via Chef:Client object. I switched to use a Chef::Application::Client object instead and I no longer have this issue (now I see a warning instead during compile phase).

tas50 commented 8 years ago

@jeremymv2 were you seeing that mismatch in the 3.X release or on master?

tas50 commented 8 years ago

If someone wants to open a PR with a small note that explains how this related to the built in resource I'd be more than happy to merge that in.

tas50 commented 7 years ago

We've removed the apt_update and apt_repository resources from this cookbook as they were merged into core chef. I'm going to close this issue out now. If you think it's still valid against a new chef-client release please open it up against the chef/chef repo so we can work on it where the code now resides.