1359852 - Show why .xpt files were rebuilt with REPORT_BUILD

Status: RESOLVED FIXED

(Firefox Build System :: General, enhancement)

Description

[Michael Shal [:mshal]]

Building with REBUILD_CHECK=1 is helpful to show why things are rebuilt, but it isn't used in the .xpt rule.

Comment 1

[Michael Shal [:mshal]]

Attachment: Bug 1359852 - Use REPORT_BUILD for .xpt files;

By using $(REPORT_BUILD) instead of just echoing the filename, there is
no change during a regular build without REBUILD_CHECK, but specifying
REBUILD_CHECK in the environment will show which files triggered an .xpt
to rebuild. This is helpful in debugging why these files may be built
unnecessarily.

Review commit: https://reviewboard.mozilla.org/r/133988/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/133988/

Comment 2

[Chris Manchester (limited bugmail, email directly)]

Comment on attachment 8862001 [details]
Bug 1359852 - Use REPORT_BUILD for .xpt files;

https://reviewboard.mozilla.org/r/133988/#review136996

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/960c7d09adb6
Use REPORT_BUILD for .xpt files; r=chmanchester

Comment 4

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/960c7d09adb6