Hello LibreOffice gerrit bot,
I'd like you to reexamine a change. Please visit
https://gerrit.libreoffice.org/4904
to look at the new patch set (#2).
Change subject: gbuild: update included dep-targets as side effect of targets
......................................................................
gbuild: update included dep-targets as side effect of targets
Make builds in 2 phases: first it ensures that all targets that are
included by the makefiles (i.e. the .d files) are up to date, then
it restarts and builds everything else.
It would be possible to update the .d files in the same build by simply
having dependencies from the dep-targets to the corresponding real targets,
but that has undesirable effects (8b5a984d45005d3df1c89eae897d6e04612625d8).
For a successful build, the .d files can actually be updated as a side
effect of other rules in such a way that they will be up-to-date on the
next make invocation, thus the restart can be avoided.
This optimization will not always work in case of build failures.
Change-Id: I753039ed33f5a84104f35990769eddc83e1d0d7e
---
M solenv/gbuild/AllLangResTarget.mk
M solenv/gbuild/LinkTarget.mk
M solenv/gbuild/UnoApiTarget.mk
3 files changed, 39 insertions(+), 15 deletions(-)
git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/04/4904/2
--
To view, visit https://gerrit.libreoffice.org/4904
To unsubscribe, visit https://gerrit.libreoffice.org/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I753039ed33f5a84104f35990769eddc83e1d0d7e
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Michael Stahl <mstahl@redhat.com>
Gerrit-Reviewer: LibreOffice gerrit bot <gerrit@libreoffice.org>
Context
- [PATCH] gbuild: update included dep-targets as side effect of target... · Michael Stahl (via Code Review)
Privacy Policy |
Impressum (Legal Info) |
Copyright information: Unless otherwise specified, all text and images
on this website are licensed under the
Creative Commons Attribution-Share Alike 3.0 License.
This does not include the source code of LibreOffice, which is
licensed under the Mozilla Public License (
MPLv2).
"LibreOffice" and "The Document Foundation" are
registered trademarks of their corresponding registered owners or are
in actual use as trademarks in one or more countries. Their respective
logos and icons are also subject to international copyright laws. Use
thereof is explained in our
trademark policy.