Is there a situation you know of where Chef::Config['file_cache_path'] is not set? By default it is always set to platform_specific_path("/var/chef/cache"), so the "|| /tmp" is dead code.