774780 - Support reading a repo manifest.xml in symbolstore.py for source info

Status: RESOLVED FIXED

(Toolkit :: Crash Reporting, defect)

Description

[(not currently active) Ted Mielczarek]

B2G uses the Google 'repo' tool to pull in a forest of git repositories. repo has an XML manifest format that describes all the repositories in use, and it can dump out a version of the manifest that contains the changesets that each repo was checked out at. Since we will probably wind up building B2G builds from tarballs instead of cloning a zillion repos, we should support reading this manifest in symbolstore.py so we can add VCS info to our symbol files without having actual repos present.


repo command to dump current repository info:
repo manifest -r -o current.xml

Comment 1

[Dietrich Ayala (:dietrich)]

is this required for b2g crash reporting? who should this be assigned to?

Comment 2

[David Bolter [:davidb] (NeedInfo me for attention)]

I think this is a very strong 'nice to have', but bug 774284 has the edge on priority. Ted will you be taking this one too?

Comment 3

[(not currently active) Ted Mielczarek]

This is required for making B2G crash reports have source links in the stack traces.

I don't have time to work on this right now, but I'd be happy to mentor someone. It'd just be some Python hacking in symbolstore.py.

Comment 4

[(not currently active) Ted Mielczarek]

Attachment: refactor VCS file mangling in symbolstore.py

This patch just refactors how the data structures underlying GetVCSFilename work. The next patch in this series gets this info from the repo manifest, so it's easier to implement if we do this first.

Comment 5

[(not currently active) Ted Mielczarek]

Attachment: Support reading a repo manifest.xml in symbolstore.py for source info

This patch adds a --repo-manifest argument to symbolstore.py, and uses that info to populate VCS info. I'll hook this up to the B2G buildsymbols in a separate patch to gonk-misc.

Comment 6

[(not currently active) Ted Mielczarek]

https://hg.mozilla.org/integration/mozilla-inbound/rev/69b34a83ee61
https://hg.mozilla.org/integration/mozilla-inbound/rev/caca2267f82b

Comment 7

[Ryan VanderMeulen (PTO, back 6-April)]

https://hg.mozilla.org/mozilla-central/rev/69b34a83ee61
https://hg.mozilla.org/mozilla-central/rev/caca2267f82b