Closed Bug 1507342 Opened 5 years ago Closed 5 years ago

profileserver.py should be able to run from a fresh checkout or minimally configured build

Categories

(Firefox Build System :: General, enhancement)

enhancement
Not set
normal

Tracking

(firefox65 fixed)

RESOLVED FIXED
mozilla65
Tracking Status
firefox65 --- fixed

People

(Reporter: chmanchester, Assigned: chmanchester)

References

Details

Attachments

(1 file)

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.
Assignee: nobody → cmanchester
Pushed by cmanchester@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/5720aea08e4f
Allow profileserver.py to run from a fresh checkout. r=firefox-build-system-reviewers,gps
https://hg.mozilla.org/mozilla-central/rev/5720aea08e4f
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 65
Target Milestone: Firefox 65 → mozilla65
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: