Closed Bug 1720846 Opened 3 years ago Closed 3 years ago

Make the profiler buildable outside of `unified-build` environment

Categories

(Core :: Gecko Profiler, task, P2)

task

Tracking

()

RESOLVED FIXED
92 Branch
Tracking Status
firefox92 --- fixed

People

(Reporter: florian, Assigned: mozbugz)

References

Details

Attachments

(2 files)

tools/profiler and mozglue/baseprofiler should pass when running ./mach static-analysis check-syntax on them, and then we can add these folders to the list at https://searchfox.org/mozilla-central/source/build/non-unified-compat

Currently there are errors in tools/profiler/tests/gtest/LulTestInfrastructure.cpp, tools/profiler/core/platform.cpp (incomplete JS::RecordAllocationInfo type), mozglue/baseprofiler/core/ProfilerBacktrace.cpp and mozglue/baseprofiler/core/ProfilerBuffer.cpp.

Severity: -- → N/A
Priority: -- → P2
Depends on: 1720853

I'm getting build failures because of added files while working on bug 1716959, so I will work on this one now.

Assignee: nobody → gsquelart
Blocks: 1716959
Pushed by gsquelart@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/46ecc01a4a63 Fix non-unified build by adding missing #includes - r=florian https://hg.mozilla.org/integration/autoland/rev/ee08d481f68e Add mozglue/baseprofiler and tools/profiler to non-unified-compat - r=florian
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 92 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: