Add the symbol and source server scripts.

Add the symbol and source server scripts.

[M-A Ruel, 2009-07-07 13:25:12]

http://codereview.chromium.org/155136/diff/1/3
File tools/symsrc/README.google (right):

http://codereview.chromium.org/155136/diff/1/3#newcode1
Line 1: These files are related to the symbol and source server.
Please rename the file to README.chromium.

http://codereview.chromium.org/155136/diff/1/3#newcode3
Line 3: This requires some Microsoft utilities.  From a WinDbg installation with
source
WinDbg 32 bits  (I'm not sure a x64 version would work so it may be safer to
specify)

http://codereview.chromium.org/155136/diff/1/4
File tools/symsrc/img_fingerprint.py (right):

http://codereview.chromium.org/155136/diff/1/4#newcode7
Line 7: # This will retrieve an image's "fingerprint".  This is used when
retrieving
We usually use pydoc for the description? e.g.
"""This will retrieve ...
"""

http://codereview.chromium.org/155136/diff/1/5
File tools/symsrc/pdb_fingerprint_from_img.py (right):

http://codereview.chromium.org/155136/diff/1/5#newcode7
Line 7: # This will retrieve a PDBs "fingerprint" from it's corresponding
executable
We usually use pydoc for the description? e.g.
"""This will retrieve ...
"""

http://codereview.chromium.org/155136/diff/1/5#newcode46
Line 46: guid.Data4_0, guid.Data4_1, cv.Age), cv.PdbFileName.split('\x00',
1)[0])
80 cols

http://codereview.chromium.org/155136/diff/1/7
File tools/symsrc/source_index.py (right):

http://codereview.chromium.org/155136/diff/1/7#newcode7
Line 7: # Usage: <win-path-to-pdb.pdb>
We usually use pydoc for the description?

http://codereview.chromium.org/155136/diff/1/7#newcode20
Line 20: # to reside in the same directory as this python script:
Would it be fine if the executables are in PATH instead? If so, you could update
README.chromium accordingly.

http://codereview.chromium.org/155136/diff/1/7#newcode31
Line 31: import subprocess
please sort

http://codereview.chromium.org/155136/diff/1/7#newcode112
Line 112: REPO_MAP = {
Please move globals to the top

http://codereview.chromium.org/155136/diff/1/7#newcode136
Line 136: 'SVN_EXTRACT_CMD=cmd /c svn cat "%var2%%var3%@%var4%"
--non-interactive > "%svn_extract_target%"',
80 cols

[Dean McNamee, 2009-07-07 13:35:41]

Updated, you'll have to live with the 1 long line in the string literal.

[M-A Ruel, 2009-07-07 13:40:09]

lgtm

we'll need to find a way to remove the duplicate sources.