Print gsutil error message when we enconter as 403 or 404

download_from_google_storage.py

Index: download_from_google_storage.py
diff --git a/download_from_google_storage.py b/download_from_google_storage.py
index 38517efeb123f6a7d5d353b22b5a7c8e7d5f1710..b3987904989a7b80d4a20e55b16873e4d91676de 100755
--- a/download_from_google_storage.py
+++ b/download_from_google_storage.py
@@ -86,13 +86,13 @@ def check_bucket_permissions(bucket, gsutil):
   base_url = 'gs://%s' % bucket
 
   code, _, ls_err = gsutil.check_call('ls', base_url)
+  if code != 0:
+    print >> sys.stderr, ls_err

[Isaac (away), 2013/09/19 17:57:59]

Optional nit: consider wrapping this so it's clear where the text is coming
from, perhaps:

'gsutil: "%s"' % ls_err

Though gsutil already does a pretty good just of informative output, for
example:
GSResponseError: status=403, code=RequestTimeTooSkewed, reason=Forbidden

[Ryan Tseng, 2013/09/19 18:05:51]

Yeah I think gsutil already does a pretty good job, no need to add any more
colons

$ download_from_google_storage . -b chsihd-dhef -dr                             
                                                                                
      
GSResponseError: status=404, code=NoSuchBucket, reason=Not Found.

gs://chsihd-dhef not found.

vs.

mudkip(er) ~/Documents/depot_tools$ download_from_google_storage . -b
chsihd-dhef -dr
gsutil: GSResponseError: status=404, code=NoSuchBucket, reason=Not Found.

gs://chsihd-dhef not found.

   if code == 403:
     print >> sys.stderr, 'Got error 403 while authenticating to %s.' % base_url
     print >> sys.stderr, 'Try running "download_from_google_storage --config".'
   elif code == 404:
     print >> sys.stderr, '%s not found.' % base_url
-  elif code != 0:
-    print >> sys.stderr, ls_err
   return (base_url, code)