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: member
    • #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 added the label Needs rebase on Sep 24, 2018
  6. ken2812221 force-pushed on Sep 24, 2018
  7. DrahtBot removed the label Needs rebase on Sep 24, 2018
  8. DrahtBot added the label Needs rebase on Sep 25, 2018
  9. ken2812221 force-pushed on Sep 25, 2018
  10. DrahtBot removed the label Needs rebase on Sep 25, 2018
  11. ken2812221 force-pushed on Sep 26, 2018
  12. ken2812221 force-pushed on Sep 26, 2018
  13. ken2812221 force-pushed on Sep 26, 2018
  14. DrahtBot commented at 8:46 am on September 28, 2018: member
    Coverage Change (pull 14241) Reference (master)
    Lines +0.0485 % 87.0361 %
    Functions +0.1390 % 84.1130 %
    Branches +0.0171 % 51.5451 %
  15. ken2812221 force-pushed on Oct 2, 2018
  16. ken2812221 force-pushed on Oct 2, 2018
  17. ken2812221 renamed this:
    appveyor: trivial build cache improvement
    appveyor: script improvement
    on Oct 2, 2018
  18. DrahtBot added the label Needs rebase on Oct 4, 2018
  19. jb55 commented at 2:40 am on October 4, 2018: member
    utACK af1dcecefa09c43ca8fb50203d1b3f439cf33291
  20. ken2812221 force-pushed on Oct 4, 2018
  21. DrahtBot removed the label Needs rebase on Oct 4, 2018
  22. appveyor: trivial build cache modifications 59a50c2179
  23. 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

    0>vcpkg remove --recurse --outdated
    1>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.

  24. ken2812221 force-pushed on Oct 5, 2018
  25. MarcoFalke merged this on Oct 8, 2018
  26. MarcoFalke closed this on Oct 8, 2018

  27. MarcoFalke referenced this in commit d9acd8648e on Oct 8, 2018
  28. ken2812221 deleted the branch on Oct 8, 2018
  29. MarcoFalke locked this on Sep 8, 2021

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-09-28 22:12 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me