Hi Joshua, On Wed, 07 Feb 2018 15:19:12 -0600 Joshua Watt <[email protected]> wrote:
> On Wed, 2018-02-07 at 21:59 +0100, Tomas Novotny wrote: > > Hi Joshua, > > > > On Tue, 6 Feb 2018 16:01:19 -0600 > > Joshua Watt <[email protected]> wrote: > > > > > Fix up support for using Icecream to do distributed builds, which > > > appears to have been broken for some time. > > > > we were using icecream a year ago and we set ICECC_CARET_WORKAROUND > > env > > variable to 0 (in set_icecc_env() in icecc.bbclass). The default is 1 > > and it > > causes a local rebuild when any warning is emitted. > > > > I don't have exact numbers, but the extreme case was opencv. There > > was so > > many warnings that distributed build on ~10 threads was slower than > > local > > build on 4 threads. > > Ok, that make sense. I'll add a patch to default to > ICECC_CARET_WORKAROUND to my patch series. Thanks for the heads up! thanks a lot. Please note one smaller consequence. The warning message is retained, but the second line of warning containing "^" pointing to the exact position is not printed. I guess that this is ok for OE. Tomas > > I know that it is some kind of configuration, but I would set the > > default to > > 0. Or do it at least configurable. > > > > > In addition, Icecream support can now be enabled in the SDK. When > > > enabled, the SDK install process will check if the host supports > > > icecc > > > and if so will construct a proper environment tarball and use it > > > when > > > compiling. > > > > Thanks for work on that, I'm keen to use it in our environment. > > > > Tomas > > > > > Most of the changes should be straightforward. icecc-create-env has > > > been almost completely re-written but the changes were broken up > > > into > > > hopefully logical chunks. > > > > > > The following changes since commit > > > 385944254d0ef88bd6450a221a54cbcb40020b42: > > > > > > tcmode-default.inc: drop preferred version of gzip-native (2018- > > > 01-31 17:01:20 +0000) > > > > > > are available in the Git repository at: > > > > > > git://push.yoctoproject.org/poky-contrib jpew/icecream > > > > > > Joshua Watt (14): > > > icecc.bbclass: Move to shared work directory > > > icecc.bbclass: Fix STAGING_BINDIR_TOOLCHAIN usage > > > icecc.bbclass: Skip canadian-cross compiles > > > icecc-create-env: Reformat > > > icecc-create-env: Add /bin/true to environment > > > icecc-create-env: Use program interpreter for deps > > > icecc-create-env: Symlink alternate names > > > icecc-create-env: Archive directory > > > icecc-create-env: Fix executable rpaths > > > icecc-create-env: Add support for nativesdk > > > icecc-create-env: Allow multiple tool aliases > > > toolchain-shar-extract: Add post-relocate scripts > > > icecc-toolchain: Add SDK icecream setup > > > icecc.bbclass: Add IceCream support to SDK > > > > > > meta/classes/icecc.bbclass | 37 ++- > > > meta/classes/toolchain-scripts.bbclass | 15 + > > > meta/files/toolchain-shar-extract.sh | 8 + > > > meta/recipes-core/meta/meta-environment.bb | 2 + > > > .../icecc-create-env/icecc-create-env | 328 > > > ++++++++++++++------- > > > ...e-env-native_0.1.bb => icecc-create-env_0.1.bb} | 4 +- > > > .../icecc-toolchain/icecc-toolchain/icecc-env.sh | 37 +++ > > > .../icecc-toolchain/icecc-toolchain/icecc-setup.sh | 43 +++ > > > .../nativesdk-icecc-toolchain_0.1.bb | 38 +++ > > > 9 files changed, 390 insertions(+), 122 deletions(-) > > > rename meta/recipes-devtools/icecc-create-env/{icecc-create-env- > > > native_0.1.bb => icecc-create-env_0.1.bb} (98%) > > > create mode 100644 meta/recipes-devtools/icecc-toolchain/icecc- > > > toolchain/icecc-env.sh > > > create mode 100644 meta/recipes-devtools/icecc-toolchain/icecc- > > > toolchain/icecc-setup.sh > > > create mode 100644 meta/recipes-devtools/icecc- > > > toolchain/nativesdk-icecc-toolchain_0.1.bb > > > > -- _______________________________________________ Openembedded-core mailing list [email protected] http://lists.openembedded.org/mailman/listinfo/openembedded-core
