msbuild: Add missing classes to the bench_bitcoin project #16747

pull sipsorcery wants to merge 1 commits into bitcoin:master from sipsorcery:add_missing_bench changing 3 files +198 −78
  1. sipsorcery commented at 8:27 PM on August 28, 2019: member

    Adds six missing classes to the msbuild bench_bitcoin project.

  2. MarcoFalke commented at 8:33 PM on August 28, 2019: member

    Why can't the wildcard /*.cpp be used as in other build files?

  3. sipsorcery commented at 8:58 PM on August 28, 2019: member

    Based on a quick inspection it seems like the bench_bitcoin.vcxproj file could be auto generated from the makefile. There may have been a reason why @ken2812221 excluded it when he wrote the original python script. I'll put in on my list to look into.

  4. fanquake added the label Windows on Aug 28, 2019
  5. Added the bench_bitcoin project to the list automatically produced by the msvc-autogen python script. 751aec28e2
  6. fanquake commented at 4:10 AM on August 29, 2019: member

    @sipsorcery Does that mean you want to close this and follow up once you've had time to look, or is this still worth merging now/as-is?

  7. sipsorcery force-pushed on Aug 29, 2019
  8. sipsorcery commented at 6:58 AM on August 29, 2019: member

    Will resubmit the msvc-autogen fix on a different PR.

  9. sipsorcery closed this on Aug 29, 2019

  10. fanquake referenced this in commit dcfd85acb4 on Aug 29, 2019
  11. DrahtBot locked this on Dec 16, 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: 2026-04-17 03:14 UTC

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