Use setuptools instead of distribute#48
Conversation
|
The recent changes to From the FAQ linked by @jamesonjlee it looks like we could simply run a UPDATE: I forgot to mention, that despite the concern around |
|
Also, the usage we have that would be a good test is to: python_pip "supervisor" do
action :upgrade
end |
|
@jamesonjlee @gondoi |
|
👍 |
|
Also ran into this issue, and successfully using @ka2n fork in our production environment. |
There was a problem hiding this comment.
Better condition is to just try and import setuptools
not_if "#{node['python']['binary']} -c 'import setuptools'"|
Hello @ka2n, Thank you for the pull request. Before we can merge it, we need a contributor license agreement (CLA) on file for you. More information can be found on the following wiki page: http://wiki.opscode.com/display/chef/How+to+Contribute Thank you, |
|
Hello @jtimberman, Thank you. |
|
@ka2n there are still a few places where you're basing the guard condition on the presence of easy_install, rather than the way @coderanger suggested. Could you please update the remaining ones? Thanks! 😄 |
|
@sethvargo I did not realize this. I'll fix it as soon as possible. Thanks! |
|
Thanky, if you can't get to fixing it tonight I can just take care of it and do the release magics :-) |
|
the remaining places is this? remote_file "#{Chef::Config[:file_cache_path]}/ez_setup.py" do
source node['python']['setuptools_script_url']
mode "0644"
not_if { ::File.exists?(ez_binary) }
end@coderanger Thanks you. I just woke up 😮 I'll fix this, but could you tell me above? |
|
Yep, same thing of you can use |
|
@coderanger, @sethvargo |
|
👍 |
Use setuptools instead of distribute
It seems that setuptools is updated and distribute 0.6.x is deprecated now.