-
dpranke@chromium.org authored
Until an issue is uploaded to Rietveld, we don't know the official email address to use for an owners check. There are three ways to fix this: we could attempt to log in to rietveld prior to doing the check and extract the address to use, or we could use ~/.last_codereview_email_address, or we can use the email address we can determine from the checkout. All three options have flaws; the first is particularly awkward since there doesn't seem to be a good way to fetch the email without posting an issue. The second is flawed if we use different addresses for different repos, and the third is flawed if the checkout's email address is different from the rietveld address, or if it is anonymous. However, since this is only being used for owners checks (in this case), anonymous checkouts probably don't matter, and hopefully the cases where the email addresses differ are rare. R=maruel@chromium.org BUG=118388, 150049 Review URL: https://chromiumcodereview.appspot.com/12377023 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@186259 0039d316-1c4b-4281-b951-d872f2087c98
f6ddfa4b