There are a few steps in the README which seem incorrect or incomplete. Fixing these would make it easier for people to contribute.
This line seems vague -- I couldn't confidently figure out what it was telling me to do. I'm a Linux newbie and I am here to represent all Linux newbies. What command should I actually run here?
When I ran upload.py it said that the configure file had been changed -- a different tool version number was listed. I didn't want that as part of my diff set so I reverted that file and then tried again.
Most importantly, the instructions for running upload.py are missing a vital piece of information. It asks you for your password but apparently it will not accept your actual e-mail password. I would never have figured that out if somebody hadn't told me. The instructions I was given for this step are:
By the way, although you do use your Google user name with upload.py,
you can't use your Google password with it. That would give upload.py
too much power, and upload.py is written to refuse to accept that.
Instead, you need to generate a fresh password for use with upload.py
and Breakpad's patch review site only.
To get the password for upload.py to use, you'll need to:
1) go to your Google account settings
2) click "Security" at the upper left
3) click "Manage Access" in the "Connected applications and sites"
section at the bottom
4) enter your Google password
5) Under "Application-specific passwords", generate a new password for
the upload.py script to use (I call it "Rietveld", since that's the
name of the patch review site's software, but the label can be
anything that helps you remember; "Breakpad patch review" wouldn't be
bad). That will show you a freshly generated password, for use with
the Breakpad patch review site only.
Then, when you run upload.py, enter your Google user name and the
fresh password.
I'm happy to play the part of naive user to make sure the instructions are complete and accurate.