840793 - Add a gecko commit optional command line argument to b2gperf

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia, defect)

Description

[Dave Hunt [:davehunt] [he/him] ⌚BST]

When we switch to using the pvtbuilds we will lose the option to extract the gecko commit from system/sources.xml and there appears to be no suitable replacement. As we still need the gecko commit for submitting results to DataZilla, we should have a command line option for this.

The sources.xml provided alongside the pvtbuilds (see bug 837784) contains the commit, so we should be able to extract this in our CI and pass it on the command line.

The following command has proven successful in obtaining the gecko commit:

$ sed -n 's/.*<project name=".*" path="gecko" remote="hgmozillaorg" revision="\(.*\)"\/>.*/\1/p' sources.xml

Comment 1

[Dave Hunt [:davehunt] [he/him] ⌚BST]

Attachment: Reintroduce sources command line option. v1.0

The latest pvtbuilds have the gecko/platform revisions in the sources.xml again. Rather than taking the revisions on the command line, I've reintroduced the --sources command line option to retrieve them from there.

Comment 2

[Dave Hunt [:davehunt] [he/him] ⌚BST]

Comment on attachment 719743 [details] [diff] [review]
Reintroduce sources command line option. v1.0

Use github pull request in favour of attached patch: https://github.com/mozilla/b2gperf/pull/3

Comment 3

[Dave Hunt [:davehunt] [he/him] ⌚BST]

Landed in: https://github.com/mozilla/b2gperf/commit/d7e79f763e2841c7794c17e05408275f21d23235