build, ci: Add file-based logging for individual packages #19952

pull hebasto wants to merge 2 commits into bitcoin:master from hebasto:200913-log changing 4 files +17 −8
  1. hebasto commented at 1:53 pm on September 13, 2020: member

    This PR adds file-based logging for individual packages in depends. To use this feature one should provide LOG=1.

    A log file is printed out automatically in case of a build error. After successful build log files are being moved along with package archives:

     0$ make -C depends HOST=x86_64-w64-mingw32 LOG=1
 1$ find ./depends/built/x86_64-w64-mingw32 -name '*.log' | sort
 2./depends/built/x86_64-w64-mingw32/bdb/bdb-4.8.30-5100a099801.log
 3./depends/built/x86_64-w64-mingw32/boost/boost-1_71_0-313f82dc7de.log
 4./depends/built/x86_64-w64-mingw32/libevent/libevent-2.1.12-stable-3fa27048d5e.log
 5./depends/built/x86_64-w64-mingw32/libnatpmp/libnatpmp-4536032ae32268a45c073a4d5e91bbab4534773a-9db4850dd32.log
 6./depends/built/x86_64-w64-mingw32/miniupnpc/miniupnpc-2.2.2-75d9a1807e0.log
 7./depends/built/x86_64-w64-mingw32/native_b2/native_b2-1_71_0-3bf253c19bf.log
 8./depends/built/x86_64-w64-mingw32/qrencode/qrencode-3.4.4-dfac87af599.log
 9./depends/built/x86_64-w64-mingw32/qt/qt-5.15.2-9304e03d3ac.log
10./depends/built/x86_64-w64-mingw32/sqlite/sqlite-3320100-455acafa7be.log
11./depends/built/x86_64-w64-mingw32/zeromq/zeromq-4.3.1-5ff627ec84a.log

    An example of CI tasks with package build errors – https://cirrus-ci.com/task/5275741788045312

    Closes #16368.

  2. fanquake added the label Build system on Sep 13, 2020
  3. fanquake added the label Utils/log/libs on Sep 13, 2020
  4. practicalswift commented at 2:06 pm on September 13, 2020: contributor
    Concept ACK
  5. hebasto commented at 3:29 pm on September 13, 2020: member
    cc @dongcarl
  6. DrahtBot commented at 1:35 pm on September 19, 2020: member

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    Conflicts

    Reviewers, this pull request conflicts with the following ones:

    • #24285 (build, refactor: Drop useless call Make function by hebasto)
    • #24279 (build: Make $(package)_*_env available to all $(package)_*_cmds by hebasto)
    • #23611 (build: add LTO option to depends (no Qt) by fanquake)
    • #23571 (build: Propagate user-defined tools to native packages by hebasto)
    • #22811 (build: Fix depends build system when working with subtargets by hebasto)
    • #22126 (build: Disable make builtin rules. by dgoncharov)

    If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

  7. MarcoFalke commented at 10:39 am on October 2, 2020: member
    Concept ACK. Does the ci print the log on failure?
  8. hebasto force-pushed on Oct 2, 2020
  9. hebasto commented at 8:36 pm on October 2, 2020: member

    Updated 95b5dfa4da8c876c7c1e9e0bbc8ae9ccb97c9370 -> 68459dd31ad6dced394ae6d5d4bd07fe3fd8f510 (pr19952.01 -> pr19952.02):

    Concept ACK. Does the ci print the log on failure?

    An example of failure logging in CI – https://travis-ci.org/github/hebasto/bitcoin/builds/732364252 with the following patch:

     0--- a/depends/patches/qt/freetype_back_compat.patch
 1+++ b/depends/patches/qt/freetype_back_compat.patch
 2@@ -22,7 +22,7 @@ index 3f543755..8ecc1c8c 100644
 3      }
 4  #if defined(FT_FONT_FORMATS_H)
 5 -    const char *fmt = FT_Get_Font_Format(face);
 6-+    const char *fmt = FT_Get_X11_Font_Format(face);
 7++    const char *fmt = FT_Get_X11_Font_Format(face)
 8      if (fmt && qstrncmp(fmt, "CFF", 4) == 0) {
 9          FT_Bool no_stem_darkening = true;
10          FT_Error err = FT_Property_Get(qt_getFreetype(), "cff", "no-stem-darkening", &no_stem_darkening);
  10. hebasto force-pushed on Nov 8, 2020
  11. hebasto commented at 9:34 pm on November 8, 2020: member
    Rebased 68459dd31ad6dced394ae6d5d4bd07fe3fd8f510 -> 4b676b1757a397ea39e6fabff42c3b30c4489630 (pr19952.02 -> pr19952.03).
  12. in depends/README.md:125 in cc80061a1d outdated 
    120@@ -121,6 +121,8 @@ build script logic) are searched for among the host system packages using
121 <dd>(EXPERTS ONLY) When cross-compiling for macOS, use Clang found in the
122 system's <code>$PATH</code> rather than the default prebuilt release of Clang
123 from llvm.org. Clang 8 or later is required.</dd>
124+<dt>LOG</dt>
125+<dd>Use file-based logging for individual packages</dd>
    dongcarl commented at 4:57 pm on November 12, 2020:
    Could you add more details? For example, where would we expect these log files to appear?
    hebasto commented at 6:32 pm on November 12, 2020:
    If wording in the updated OP is ok, I’ll push it into the docs.
  13. in depends/funcs.mk:89 in cc80061a1d outdated 
    79@@ -79,7 +80,7 @@ $(1)_download_path_fixed=$(subst :,\:,$$($(1)_download_path))
80 # The default behavior for tar will try to set ownership when running as uid 0 and may not succeed, --no-same-owner disables this behavior
81 $(1)_fetch_cmds ?= $(call fetch_file,$(1),$(subst \:,:,$$($(1)_download_path_fixed)),$$($(1)_download_file),$($(1)_file_name),$($(1)_sha256_hash))
82 $(1)_extract_cmds ?= mkdir -p $$($(1)_extract_dir) && echo "$$($(1)_sha256_hash)  $$($(1)_source)" > $$($(1)_extract_dir)/.$$($(1)_file_name).hash &&  $(build_SHA256SUM) -c $$($(1)_extract_dir)/.$$($(1)_file_name).hash && tar --no-same-owner --strip-components=1 -xf $$($(1)_source)
83-$(1)_preprocess_cmds ?=
84+$(1)_preprocess_cmds ?= true
    dongcarl commented at 4:57 pm on November 12, 2020:
    Why is this necessary?
    hebasto commented at 6:23 pm on November 12, 2020:
    To apply $(1)_logging to $(1)_preprocess_cmds even the latter is not set explicitly.
  14. in depends/funcs.mk:192 in cc80061a1d outdated 
    173@@ -173,6 +174,10 @@ endif
174 endef
175 
176 define int_add_cmds
177+ifneq ($(LOG),)
178+$(1)_logging = >>$$($(1)_build_log) 2>&1 || { if test -f $$($(1)_build_log); then cat $$($(1)_build_log); fi; exit 1; }
    dongcarl commented at 4:59 pm on November 12, 2020:
    So this would mean that while the $(1)_*_cmds is running, we wouldn’t see any logs, and only if the cmds fail, would we cat the logs? Am I understanding this correctly?
    hebasto commented at 6:19 pm on November 12, 2020:
    Yes, you’re correct. This behavior is applied only when LOG is set.
  15. laanwj commented at 4:19 am on November 20, 2020: member
    Concept ACK, I think not being verbose unless the level of detail is really needed (say, on a failure) is good for build/test commands. Makes it easier to spot actual problems. Please do make it easy to figure out where the logs went in case they’re needed though :slightly_smiling_face:
  16. DrahtBot added the label Needs rebase on Dec 17, 2020
  17. hebasto force-pushed on Dec 19, 2020
  18. hebasto commented at 8:23 pm on December 19, 2020: member

    Updated 4b676b1757a397ea39e6fabff42c3b30c4489630 -> 46e2ff7c0dd4d15d17dc6d9015f28191d3efc05b (pr19952.03 -> pr19952.04):

    • rebased on top of the recent CI changes
    • addressed @laanwj’s comment:

    Please do make it easy to figure out where the logs went in case they’re needed though

    The example of CI output in case of error during depends build is here.

  19. DrahtBot removed the label Needs rebase on Dec 19, 2020
  20. dongcarl commented at 10:13 pm on April 26, 2021: member

    Hmmm, I’ve been thinking about this… And wondering if we could use tee and process substitute to do:

     0diff --git a/depends/funcs.mk b/depends/funcs.mk
 1index 3ddeaf0378..2de5f67b5c 100644
 2--- a/depends/funcs.mk
 3+++ b/depends/funcs.mk
 4@@ -80,7 +80,7 @@ $(1)_download_path_fixed=$(subst :,\:,$$($(1)_download_path))
 5 # The default behavior for tar will try to set ownership when running as uid 0 and may not succeed, --no-same-owner disables this behavior
 6 $(1)_fetch_cmds ?= $(call fetch_file,$(1),$(subst \:,:,$$($(1)_download_path_fixed)),$$($(1)_download_file),$($(1)_file_name),$($(1)_sha256_hash))
 7 $(1)_extract_cmds ?= mkdir -p $$($(1)_extract_dir) && echo "$$($(1)_sha256_hash)  $$($(1)_source)" > $$($(1)_extract_dir)/.$$($(1)_file_name).hash &&  $(build_SHA256SUM) -c $$($(1)_extract_dir)/.$$($(1)_file_name).hash && tar --no-same-owner --strip-components=1 -xf $$($(1)_source)
 8-$(1)_preprocess_cmds ?= true
 9+$(1)_preprocess_cmds ?=
10 $(1)_build_cmds ?=
11 $(1)_config_cmds ?=
12 $(1)_stage_cmds ?=
13@@ -177,7 +177,7 @@ endef
14 
15 define int_add_cmds
16 ifneq ($(LOG),)
17-$(1)_logging = >>$$($(1)_build_log) 2>&1 || { if test -f $$($(1)_build_log); then cat $$($(1)_build_log); fi; exit 1; }
18+$(1)_logging = exec >> >(tee -a $$($(1)_build_log)) 2>&1;
19 endif
20 
21 $($(1)_fetched):
22@@ -196,23 +196,23 @@ $($(1)_preprocessed): | $($(1)_extracted)
23 	$(AT)echo Preprocessing $(1)...
24 	$(AT)mkdir -p $$(@D) $($(1)_patch_dir)
25 	$(AT)$(foreach patch,$($(1)_patches),cd $(PATCHES_PATH)/$(1); cp $(patch) $($(1)_patch_dir) ;)
26-	$(AT){ cd $$(@D); $(call $(1)_preprocess_cmds, $(1)); } $$($(1)_logging)
27+	$(AT)( $$($(1)_logging) cd $$(@D); $(call $(1)_preprocess_cmds, $(1)) )
28 	$(AT)touch $$@
29 $($(1)_configured): | $($(1)_dependencies) $($(1)_preprocessed)
30 	$(AT)echo Configuring $(1)...
31 	$(AT)rm -rf $(host_prefix); mkdir -p $(host_prefix)/lib; cd $(host_prefix); $(foreach package,$($(1)_all_dependencies), tar --no-same-owner -xf $($(package)_cached); )
32 	$(AT)mkdir -p $$(@D)
33-	$(AT)+{ cd $$(@D); $($(1)_config_env) $(call $(1)_config_cmds, $(1)); } $$($(1)_logging)
34+	$(AT)+( $$($(1)_logging) cd $$(@D); $($(1)_config_env) $(call $(1)_config_cmds, $(1)) )
35 	$(AT)touch $$@
36 $($(1)_built): | $($(1)_configured)
37 	$(AT)echo Building $(1)...
38 	$(AT)mkdir -p $$(@D)
39-	$(AT)+{ cd $$(@D); $($(1)_build_env) $(call $(1)_build_cmds, $(1)); } $$($(1)_logging)
40+	$(AT)+( $$($(1)_logging) cd $$(@D); $($(1)_build_env) $(call $(1)_build_cmds, $(1)) )
41 	$(AT)touch $$@
42 $($(1)_staged): | $($(1)_built)
43 	$(AT)echo Staging $(1)...
44 	$(AT)mkdir -p $($(1)_staging_dir)/$(host_prefix)
45-	$(AT)+{ cd $($(1)_build_dir); $($(1)_stage_env) $(call $(1)_stage_cmds, $(1)); } $$($(1)_logging)
46+	$(AT)+( $$($(1)_logging) cd $($(1)_build_dir); $($(1)_stage_env) $(call $(1)_stage_cmds, $(1)) )
47 	$(AT)rm -rf $($(1)_extract_dir)
48 	$(AT)touch $$@
49 $($(1)_postprocessed): | $($(1)_staged)

    I supposed POSIX sh doesn’t have process substitution… But perhaps for depends we can set .SHELL = bash? Just thinking…

  21. DrahtBot added the label Needs rebase on Nov 15, 2021
  22. hebasto commented at 12:39 pm on January 3, 2022: member

    @dongcarl

    I supposed POSIX sh doesn’t have process substitution… But perhaps for depends we can set .SHELL = bash? Just thinking…

    Right. It does not work in sh or dash scripts. But we should support dash.

  23. hebasto force-pushed on Jan 3, 2022
  24. hebasto commented at 1:10 pm on January 3, 2022: member
    Updated 46e2ff7c0dd4d15d17dc6d9015f28191d3efc05b -> bdd1056fbdd2164f852940afbaf63abe83d1f571 (pr19952.04 -> pr19952.05) due to the conflict with #22283.
  25. hebasto commented at 2:08 pm on January 3, 2022: member
    The PR description has been updated.
  26. DrahtBot removed the label Needs rebase on Jan 3, 2022
  27. DrahtBot added the label Needs rebase on Feb 1, 2022
  28. hebasto force-pushed on Feb 2, 2022
  29. hebasto commented at 3:50 pm on February 2, 2022: member
    Rebased bdd1056fbdd2164f852940afbaf63abe83d1f571 -> 0b9357beb4bb6e39125090204b26a00842f2736e (pr19952.05 -> pr19952.06) due to the conflict with #24212.
  30. DrahtBot removed the label Needs rebase on Feb 2, 2022
  31. DrahtBot added the label Needs rebase on Feb 11, 2022
  32. hebasto force-pushed on Feb 13, 2022
  33. hebasto commented at 10:34 am on February 13, 2022: member
    Rebased 0b9357beb4bb6e39125090204b26a00842f2736e -> 3c5c5839fe611e5bb723c792a24c582df4b0f5c0 (pr19952.06 -> pr19952.07) due to the conflict with #23998.
  34. DrahtBot removed the label Needs rebase on Feb 13, 2022
  35. hebasto commented at 5:54 pm on February 13, 2022: member

    Guix builds:

     0$ find guix-build-$(git rev-parse --short=12 HEAD)/output/ -type f -print0 | env LC_ALL=C sort -z | xargs -r0 sha256sum
 1e0e8d12c43c077aa59be7ef51d0290c1c0a204592b03a024d7e98054547b4298  guix-build-3c5c5839fe61/output/aarch64-linux-gnu/SHA256SUMS.part
 22b04b0c30974d5ec8bb048890a4c89e31afb60b7e638155b73c96a0cb07603a6  guix-build-3c5c5839fe61/output/aarch64-linux-gnu/bitcoin-3c5c5839fe61-aarch64-linux-gnu-debug.tar.gz
 3cd8be3cdd7a54877dce307072b646e3706b57f626e2ea13f38274394aa9ae16f  guix-build-3c5c5839fe61/output/aarch64-linux-gnu/bitcoin-3c5c5839fe61-aarch64-linux-gnu.tar.gz
 49a760fb5b0b1f7a7c1d35cac6b4447eea77823722230d3bf19d8a809ac7affdc  guix-build-3c5c5839fe61/output/arm-linux-gnueabihf/SHA256SUMS.part
 5fc53bb9ec64fa928baf787119174ba2dc8a48f982eec85554073a04cf5a8416f  guix-build-3c5c5839fe61/output/arm-linux-gnueabihf/bitcoin-3c5c5839fe61-arm-linux-gnueabihf-debug.tar.gz
 6c7720367ef22d2f2f622cfdee492d7101eae3633c7b61ee9035837bd267438c0  guix-build-3c5c5839fe61/output/arm-linux-gnueabihf/bitcoin-3c5c5839fe61-arm-linux-gnueabihf.tar.gz
 7e064e261db13ebeafcb9bcc1ca8b1fca16d8105a9e4454b6f240cd055236789b  guix-build-3c5c5839fe61/output/arm64-apple-darwin/SHA256SUMS.part
 83bb7a65fb83ebf0b2d06c2d1d1e9bee096198a0c7f9f7b8c41e0f7d9a4f85650  guix-build-3c5c5839fe61/output/arm64-apple-darwin/bitcoin-3c5c5839fe61-arm64-apple-darwin.tar.gz
 939b9d88cb56e9a2e5626c7673f6bcc2764ac341a9bd9d4cf048900543261013c  guix-build-3c5c5839fe61/output/arm64-apple-darwin/bitcoin-3c5c5839fe61-osx-unsigned.dmg
108d5b5acd83883c53228527ca87be6af46c765243d8bdab073ec2355c110394dd  guix-build-3c5c5839fe61/output/arm64-apple-darwin/bitcoin-3c5c5839fe61-osx-unsigned.tar.gz
112f605b0320c90cd30028724d3684971c0b323135d5bdc3894e826b54aab5a79d  guix-build-3c5c5839fe61/output/dist-archive/bitcoin-3c5c5839fe61.tar.gz
12e4d5faaa07788ed1b454ecbbcdc81d1546239a8356fa3c118dc4d4714ad2e771  guix-build-3c5c5839fe61/output/powerpc64-linux-gnu/SHA256SUMS.part
133a2401d8b989b7f9a714f27d4c8894040d5407e0d4d841adc705ff420a119467  guix-build-3c5c5839fe61/output/powerpc64-linux-gnu/bitcoin-3c5c5839fe61-powerpc64-linux-gnu-debug.tar.gz
14f86901a9f2336201f5a3cac2556ad317e0ef33a1a89303557ee0ceb5ee31c43d  guix-build-3c5c5839fe61/output/powerpc64-linux-gnu/bitcoin-3c5c5839fe61-powerpc64-linux-gnu.tar.gz
15b7302a12063fc85a96077f9048451d737ae3a5f950ad17fc521a63d02063b6b7  guix-build-3c5c5839fe61/output/powerpc64le-linux-gnu/SHA256SUMS.part
16d181f8ee35239094d3dfa953bd8ba3d51a8ee38a4038149a15f400d386587274  guix-build-3c5c5839fe61/output/powerpc64le-linux-gnu/bitcoin-3c5c5839fe61-powerpc64le-linux-gnu-debug.tar.gz
17810d60570ca083217fc2eb378bceb5361605dda47ae2fd76bc693a027ee8409a  guix-build-3c5c5839fe61/output/powerpc64le-linux-gnu/bitcoin-3c5c5839fe61-powerpc64le-linux-gnu.tar.gz
18b2f8d87ae3d8ce4abb121b7f589f9d1db71ee32a456fec3639e7714d0fa7ca5f  guix-build-3c5c5839fe61/output/riscv64-linux-gnu/SHA256SUMS.part
19994d00a6514c704ac177f48790b9acb586f1e56f8aa702b205d532d95a81e2d9  guix-build-3c5c5839fe61/output/riscv64-linux-gnu/bitcoin-3c5c5839fe61-riscv64-linux-gnu-debug.tar.gz
20b4bacd388be1f687404baa31650647e3416b743d0a8ceb4a57121d25750cb0fc  guix-build-3c5c5839fe61/output/riscv64-linux-gnu/bitcoin-3c5c5839fe61-riscv64-linux-gnu.tar.gz
21f285e4da8535f25605f3f16e3f72b8a8f64210ec4686b5aba784ff48842476d9  guix-build-3c5c5839fe61/output/x86_64-apple-darwin/SHA256SUMS.part
22af453058f80b0a933563ab5c3f8003af75e37d79fff823f418ba881c9e53ed81  guix-build-3c5c5839fe61/output/x86_64-apple-darwin/bitcoin-3c5c5839fe61-osx-unsigned.dmg
237eb22cd46617f0c1a55328931e3fa5e7a9bb481ff56dc136fe62854ef08dde99  guix-build-3c5c5839fe61/output/x86_64-apple-darwin/bitcoin-3c5c5839fe61-osx-unsigned.tar.gz
243456ac4af66f2e71473a3d891c9e953a39a49cbd306d8d9d935e50aa50dea8d8  guix-build-3c5c5839fe61/output/x86_64-apple-darwin/bitcoin-3c5c5839fe61-osx64.tar.gz
25d35fde85151a30f780bf69c12e40399ac96810e1cb6dfa7861fb7a48d0be801d  guix-build-3c5c5839fe61/output/x86_64-linux-gnu/SHA256SUMS.part
26b1994d47c85bfa248b0483bc910d805718c2c920571e9e465d5cbe6da3cae65b  guix-build-3c5c5839fe61/output/x86_64-linux-gnu/bitcoin-3c5c5839fe61-x86_64-linux-gnu-debug.tar.gz
27591e0d6d7a706d10b5bcbe93387560b6790dd989f046d0a32ec20e028bbda037  guix-build-3c5c5839fe61/output/x86_64-linux-gnu/bitcoin-3c5c5839fe61-x86_64-linux-gnu.tar.gz
  36. DrahtBot added the label Needs rebase on Mar 10, 2022
  37. hebasto force-pushed on Apr 5, 2022
  38. hebasto force-pushed on Apr 5, 2022
  39. hebasto commented at 6:04 pm on April 5, 2022: member
    Rebased 3c5c5839fe611e5bb723c792a24c582df4b0f5c0 -> eb4b01edf0a4cbe3e7b306f1cb51f03f87f13879 (pr19952.07 -> pr19952.08) due to the conflict with #24522.
  40. hebasto commented at 6:06 pm on April 5, 2022: member
    Friendly ping @dongcarl @MarcoFalke
  41. MarcoFalke commented at 6:33 pm on April 5, 2022: member
    Concept ACK
  42. DrahtBot removed the label Needs rebase on Apr 5, 2022
  43. DrahtBot added the label Needs rebase on Apr 13, 2022
  44. depends: Add file-based logging for individual packages 7f650883b7
  45. ci: Make log verbose in error case only 
    This change silences depends build using LOG=1.
    86c2889518
  46. hebasto force-pushed on Apr 14, 2022
  47. hebasto commented at 11:51 am on April 14, 2022: member
    Rebased eb4b01edf0a4cbe3e7b306f1cb51f03f87f13879 -> 86c2889518df0234b8b28112a9771b85c577eefd (pr19952.08 -> pr19952.09) due to the conflict with #24285.
  48. laanwj commented at 12:21 pm on April 14, 2022: member
    Going to test this.
  49. DrahtBot removed the label Needs rebase on Apr 14, 2022
  50. laanwj commented at 2:26 pm on April 14, 2022: member

    Tested ACK 86c2889518df0234b8b28112a9771b85c577eefd

    I know it’s not the intended use but I patched this into the guix build, as that’s the only depends build i do at the moment:

     0--- a/contrib/guix/libexec/build.sh
 1+++ b/contrib/guix/libexec/build.sh
 2@@ -198,7 +198,7 @@ esac
 3 ####################
 4 
 5 # Build the depends tree, overriding variables that assume multilib gcc
 6-make -C depends --jobs="$JOBS" HOST="$HOST" \
 7+make -C depends --jobs="$JOBS" HOST="$HOST" LOG=1 \
 8                                    ${V:+V=1} \
 9                                    ${SOURCES_PATH+SOURCES_PATH="$SOURCES_PATH"} \
10                                    ${BASE_CACHE+BASE_CACHE="$BASE_CACHE"} \

    The log was clean. A lot less spammy, this is good!:

     0make: Entering directory '/bitcoin/depends'
 1Extracting boost...
 2/home/guest/sources/boost_1_77_0.tar.bz2: OK
 3Preprocessing boost...
 4Configuring boost...
 5Building boost...
 6Staging boost...
 7Postprocessing boost...
 8Caching boost...
 9Extracting libevent...
10/home/guest/sources/libevent-2.1.12-stable.tar.gz: OK
11Preprocessing libevent...
12Configuring libevent...
13⋮ 
14Preprocessing zeromq...
15Configuring zeromq...
16Building zeromq...
17Staging zeromq...
18Postprocessing zeromq...
19Caching zeromq...
20copying packages: boost libevent qt expat libxcb xcb_proto libXau xproto freetype fontconfig libxkbcommon libxcb_util libxcb_util_render libxcb_util_keysyms libxcb_util_image libxcb_util_wm qrencode bdb sqlite miniupnpc libnatpmp systemtap zeromq
21to: /bitcoin/depends/riscv64-linux-gnu

    After the build, the individual log files could be found in the cache:

     0./base-cache/riscv64-linux-gnu/qt/qt-5.15.3-30a361c8788.log
 1./base-cache/riscv64-linux-gnu/libnatpmp/libnatpmp-4536032ae32268a45c073a4d5e91bbab4534773a-59aa9cc4671.log
 2./base-cache/riscv64-linux-gnu/qrencode/qrencode-3.4.4-0469a7d34c9.log
 3./base-cache/riscv64-linux-gnu/boost/boost-1.77.0-1f1d64dbf3c.log
 4./base-cache/riscv64-linux-gnu/libxcb_util_render/libxcb_util_render-0.3.9-95703f260b3.log
 5⋮ 
 6./base-cache/riscv64-linux-gnu/bdb/bdb-4.8.30-cf38a657499.log
 7./base-cache/riscv64-linux-gnu/sqlite/sqlite-3320100-06ea4df8e5e.log
 8./base-cache/riscv64-linux-gnu/libxcb_util_keysyms/libxcb_util_keysyms-0.4.0-baf1d726a20.log
 9./base-cache/riscv64-linux-gnu/fontconfig/fontconfig-2.12.6-181950ffe12.log
10./base-cache/riscv64-linux-gnu/expat/expat-2.4.1-24df0173864.log

    I also corrupted a patch intentionally to see if the logging would still report the error in that case, it does:

     0make: Entering directory '/bitcoin/depends'
 1Extracting qt...
 2/home/guest/sources/qtbase-everywhere-opensource-src-5.15.3.tar.xz: OK
 3/home/guest/sources/qttranslations-everywhere-opensource-src-5.15.3.tar.xz: OK
 4/home/guest/sources/qttools-everywhere-opensource-src-5.15.3.tar.xz: OK
 5Preprocessing qt...
 6patching file qtbase/configure
 7patch: **** malformed patch at line 19: dsoiusaoifasdfiadfiosudfsduf
 8
 9make: *** [funcs.mk:289: /bitcoin/depends/work/build/riscv64-linux-gnu/qt/5.15.3-3a31e8c8789/.stamp_preprocessed] Error 1
10make: Leaving directory '/bitcoin/depends'
  51. laanwj merged this on Apr 14, 2022
  52. laanwj closed this on Apr 14, 2022
  53. hebasto deleted the branch on Apr 14, 2022
  54. sidhujag referenced this in commit 58b98f18f5 on Apr 14, 2022
  55. DrahtBot locked this on Jul 4, 2023


hebasto practicalswift DrahtBot MarcoFalke dongcarl laanwj

Labels
Build system Utils/log/libs

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-07-05 22:12 UTC
This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me