1399394 - Support code coverage builds on Mac

Status: NEW

(Testing :: Code Coverage, task)

Comment 1

[Marco Castelluccio [:marco]]

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.

Comment 2

[Marco Castelluccio [:marco]]

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.

Comment 3

[Marco Castelluccio [:marco]]

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.

Comment 4

[Marco Castelluccio [:marco]]

(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.

Comment 5

[Marco Castelluccio [:marco]]

After we fix bug 1495683, we should start to schedule the build (at least weekly) so we notice regressions earlier.

Comment 6

[Marco Castelluccio [:marco]]

Attachment: Run Mac coverage builds on mozilla-central pushes, without running its tests

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/0d0fab4a52f7
Run Mac coverage builds on mozilla-central pushes, without running its tests. r=jmaher

Comment 8

[Stefan Hindli [:stefan_hindli]]

https://hg.mozilla.org/mozilla-central/rev/0d0fab4a52f7

Comment 9

[BugBot [:suhaib / :marco/ :calixte]]

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

Comment 10

[BugBot [:suhaib / :marco/ :calixte]]

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

Comment 11

[Marco Castelluccio [:marco]]

There's still work to do here.

Comment 12

[BugBot [:suhaib / :marco/ :calixte]]

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

Comment 13

[Marco Castelluccio [:marco]]

This is still something we should do at some point.

Comment 14

[BugBot [:suhaib / :marco/ :calixte]]

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

Comment 15

[BugBot [:suhaib / :marco/ :calixte]]

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

Comment 16

[BugBot [:suhaib / :marco/ :calixte]]

The leave-open keyword is there and there is no activity for 6 months.
:marco, maybe it's time to close this bug?
For more information, please visit auto_nag documentation.