Closed Bug 1507342 Opened 2 years ago Closed 2 years ago
.py should be able to run from a fresh checkout or minimally configured build
47 bytes, text/x-phabricator-request
|Details | Review|
When running the second step of the three-staged PGO scheme we need to run profileserver.py without running all of configure (and therefore setting up all the build requirements for the task). Right now profileserver.py imports buildconfig and through get_binary_path needs to access various config keys. We should either pass everything down to if from the commandline or ensure it can do its job effectively with a minimally configured (disable-compile-environment at least) tree.
The simplest route here might be to make the second task do a source checkout so we don't have to package up all the HTML bits that the profiling run wants and then run this via a custom mach command. We probably don't need to run configure, if we pass the path to the Firefox binary in everything else it needs is in topsrcdir. We might have to double-check that the instrumented builds that clang generates aren't dynamically linked to anything from the clang toolchain. If so we'd either need to package those binaries with the build or make the toolchain available to the profiling task.
2 years ago
Assignee: nobody → cmanchester
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/5720aea08e4f Allow profileserver.py to run from a fresh checkout. r=firefox-build-system-reviewers,gps
You need to log in before you can comment on or make changes to this bug.