Open Bug 1399394 Opened 3 years ago Updated 23 days ago

Support code coverage builds on Mac

Categories

(Testing :: Code Coverage, task)

task
Not set
normal

Tracking

(Not tracked)

ASSIGNED

People

(Reporter: marco, Assigned: marco)

References

(Depends on 2 open bugs, Blocks 1 open bug)

Details

(Keywords: leave-open)

Attachments

(1 file)

No description provided.
Assignee: nobody → jyuen1
Depends on: 1409747
From bug 1381069 comment 8:
> Update: With the fix in Bug 1409747 (thanks :marco) applied locally to
> latest inbound, my osx ccov build finishes now - but when I startup Firefox
> it errors out with:
> 
> Roberts-MacBook-Pro-1903:mozilla-unified rwood$ ./mach run
>  0:00.69
> /Users/rwood/mozilla-unified/obj-x86_64-apple-darwin16.7.0/dist/NightlyDebug.
> app/Contents/MacOS/firefox -no-remote -foreground -profile
> /Users/rwood/mozilla-unified/obj-x86_64-apple-darwin16.7.0/tmp/scratch_user
> Assertion failure: Storage<T>::initialized(), at
> /Users/rwood/mozilla-unified/obj-x86_64-apple-darwin16.7.0/dist/include/
> mozilla/ThreadLocal.h:225

I'll see if I can reproduce the same crash tomorrow or Friday.
I've managed to get a working build using the following mozconfig:
> ac_add_options --disable-install-strip
> ac_add_options --disable-jemalloc
> ac_add_options --disable-elf-hack
> ac_add_options --enable-debug
> ac_add_options --enable-debug-symbols
> ac_add_options --disable-sandbox
> ac_add_options --enable-coverage
> 
> export CFLAGS="--coverage"
> export CXXFLAGS="--coverage"
> export LDFLAGS="--coverage"

Revision: f7c7cb48610c.
Clang version: Apple LLVM version 9.0.0 (clang-900.0.38).
OS X version: 10.13.
Blocks: 1429455
Assignee: jyuen1 → nobody
Mentor: mcastelluccio
Assignee: nobody → mcastelluccio
Mentor: mcastelluccio
Status: NEW → ASSIGNED
Depends on: 1465735
Depends on: 1465737
Depends on: 1465738
Depends on: 1465739
Depends on: 1465740
I've tested a build with mochitest-a11y (one of the fastest suites), after applying all patches from the current blocking bugs.
The JavaScript coverage looks sane, the C++ one is still missing a lot of files. I'll try downloading the gcda files from the test machines to figure out what's wrong with the C++ coverage.
Depends on: 1467304
(In reply to Marco Castelluccio [:marco] from comment #3)
> I've tested a build with mochitest-a11y (one of the fastest suites), after
> applying all patches from the current blocking bugs.
> The JavaScript coverage looks sane, the C++ one is still missing a lot of
> files. I'll try downloading the gcda files from the test machines to figure
> out what's wrong with the C++ coverage.

Now that all the basic blocking bugs are fixed, I re-tested and both JS and C++ coverage looked fine.
Depends on: 1472729
Depends on: 1473531
No longer depends on: 1472729
Depends on: 1439880
See Also: → 1473541
Depends on: 1490247
Depends on: 1495683
After we fix bug 1495683, we should start to schedule the build (at least weekly) so we notice regressions earlier.
Depends on: 1504188
Depends on: 1504250
Keywords: leave-open
Attachment #9022242 - Flags: review?(jmaher) → review+
Depends on: 1504632
Depends on: 1504680
Pushed by mcastelluccio@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/0d0fab4a52f7
Run Mac coverage builds on mozilla-central pushes, without running its tests. r=jmaher

The leave-open keyword is there and there is no activity for 6 months.
:marco, maybe it's time to close this bug?

Flags: needinfo?(mcastelluccio)
Flags: needinfo?(mcastelluccio)
Type: defect → task
No longer blocks: code-coverage
Blocks: code-coverage
No longer blocks: 1429455

The leave-open keyword is there and there is no activity for 6 months.
:marco, maybe it's time to close this bug?

Flags: needinfo?(mcastelluccio)

There's still work to do here.

Flags: needinfo?(mcastelluccio)
You need to log in before you can comment on or make changes to this bug.