TLDR: We’ve enabled build error reporting for C/C++ on Linux on Phabricator.
Hello all, Besides the linting and static analysis that is run on patches at submission, we’ve added the reporting of build errors for C/C++ as Unit Test failures. For now, the build is run on a linux64 machine targeting a linux64 environment, so the potential errors that may occur will strictly come from this environment. In order to have a better picture on how these errors are reported, please see the attached screenshot. https://i.ibb.co/2S3pwY5/first.png <https://i.ibb.co/2S3pwY5/first.png> Also please note that each time a patch is updated, the Unit Tests results for the previous version of the patch will be replaced by the results of the newly submitted one. Example of an error being reported: https://phabricator.services.mozilla.com/D47181 As we are still experimenting with this aspect of Phabricator, don’t hesitate if you have any feedback. In parallel, we will also send an email through Taskcluster to make sure that the error is notified to the author. https://i.ibb.co/ckTtKzG/midle.png With this change, we are aiming at decreasing the turnaround time while reducing the number of backouts (which require human intervention). We have recently also improved TreeHerder integration. For every patch submitted for review (since a few months ago), a new try job is automatically created <https://wiki.mozilla.org/ReleaseEngineering/TryServer#Attach_new_jobs_from_the_review> . Now, the reviewbot jobs also clearly reference the review: https://i.ibb.co/hLBLqZ5/last.png Many Thanks, Andi, on the behalf of the team _______________________________________________ dev-platform mailing list dev-platform@lists.mozilla.org https://lists.mozilla.org/listinfo/dev-platform
