appveyor: script improvement #14241

pull ken2812221 wants to merge 1 commits into bitcoin:master from ken2812221:2018-09-17-appveyor-reduce-cache changing 1 files +21 −15
  1. ken2812221 commented at 2:11 PM on September 17, 2018: contributor
    • Reduce cache size from 1.55GB to 170MB by disabling whole program optimization.
    • The *.iobj *.ipdb cache is no longer required because of above change.
    • Clearing cache hit stats instead of deleting clcache stats.txt file
    • Enable access time on Windows so clcache would work correctly
    • Reduce max cache size from 2GB to 512MB (Uncompressed)
    • Remove unnecessary boost-interprocess vcpkg package
    • Split some command into multiple lines to make it easier to know what commands do.
  2. fanquake added the label Tests on Sep 17, 2018
  3. ken2812221 force-pushed on Sep 17, 2018
  4. DrahtBot commented at 2:39 PM on September 17, 2018: contributor

    <!--e57a25ab6845829454e8d69fc972939a-->Reviewers, this pull request conflicts with the following ones:

    • #14372 (msvc: build secp256k1 and leveldb locally by ken2812221)
    • #13787 (Test for Windows encoding issue by ken2812221)

    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.

  5. DrahtBot cross-referenced this on Sep 17, 2018 from issue tests: Run functional test on Windows and enable it on Appveyor by ken2812221
  6. DrahtBot cross-referenced this on Sep 20, 2018 from issue tests: Write the notification message to different files to avoid race condition in feature_notifications.py by ken2812221
  7. DrahtBot added the label Needs rebase on Sep 24, 2018
  8. ken2812221 force-pushed on Sep 24, 2018
  9. DrahtBot removed the label Needs rebase on Sep 24, 2018
  10. DrahtBot added the label Needs rebase on Sep 25, 2018
  11. ken2812221 force-pushed on Sep 25, 2018
  12. DrahtBot removed the label Needs rebase on Sep 25, 2018
  13. ken2812221 force-pushed on Sep 26, 2018
  14. ken2812221 force-pushed on Sep 26, 2018
  15. ken2812221 force-pushed on Sep 26, 2018
  16. DrahtBot cross-referenced this on Sep 27, 2018 from issue utils: Convert Windows args to utf-8 string by ken2812221
  17. DrahtBot commented at 8:46 AM on September 28, 2018: contributor

    <!--32850dd3fdea838b4049e64f46995ea2-->

    Coverage Change (pull 14241) Reference (master)
    Lines +0.0485 % 87.0361 %
    Functions +0.1390 % 84.1130 %
    Branches +0.0171 % 51.5451 %
  18. DrahtBot cross-referenced this on Sep 30, 2018 from issue Test for Windows encoding issue by ken2812221
  19. ken2812221 force-pushed on Oct 2, 2018
  20. ken2812221 force-pushed on Oct 2, 2018
  21. ken2812221 renamed this:
    appveyor: trivial build cache improvement
    appveyor: script improvement
    on Oct 2, 2018
  22. DrahtBot cross-referenced this on Oct 2, 2018 from issue msvc: build secp256k1 and leveldb locally by ken2812221
  23. DrahtBot cross-referenced this on Oct 3, 2018 from issue test: Add missing call to skip_if_no_cli() by practicalswift
  24. DrahtBot added the label Needs rebase on Oct 4, 2018
  25. jb55 commented at 2:40 AM on October 4, 2018: contributor

    utACK af1dcecefa09c43ca8fb50203d1b3f439cf33291

  26. ken2812221 force-pushed on Oct 4, 2018
  27. DrahtBot removed the label Needs rebase on Oct 4, 2018
  28. appveyor: trivial build cache modifications 59a50c2179
  29. in .appveyor.yml:26 in f01ad1ec6c outdated 
      25 | @@ -27,28 +26,34 @@ install:
  26 |  - cmd: vcpkg upgrade --no-dry-run
    ras0219-msft commented at 1:10 AM on October 5, 2018:

    It would be better to replace the install+upgrade with

    >vcpkg remove --recurse --outdated
>vcpkg install %ALL_PACKAGES%

    Otherwise, you'll be rebuilding packages that you no longer use (like boost interprocess).

    You should also run ./bootstrap-vcpkg.bat after doing a git pull, because there may be important tool changes.

    ken2812221 commented at 5:55 AM on October 5, 2018:

    Awesome

    My original thought is that we could get rid of boost-interprocess by clear the cache, but if we use the command you provide, we don't have to clear the cache anymore. Just have to wait for next boost-interprocess update and it would be removed automatically.

  30. ken2812221 force-pushed on Oct 5, 2018
  31. MarcoFalke merged this on Oct 8, 2018
  32. MarcoFalke closed this on Oct 8, 2018
  33. MarcoFalke referenced this in commit d9acd8648e on Oct 8, 2018
  34. ken2812221 deleted the branch on Oct 8, 2018
  35. Bushstar cross-referenced this on Oct 11, 2018 from issue Updates from bitcoin/master by Bushstar
  36. bitcoin locked this on Sep 8, 2021
Contributors
ken2812221DrahtBotjb55ras0219-msft
Labels
Tests
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: 2026-05-19 11:54 UTC
This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me