Fix compilation with C++ #258

pull vinniefalco wants to merge 1 commits into bitcoin-core:master from vinniefalco:master changing 1 files +3 −3
  1. vinniefalco commented at 1:35 PM on June 18, 2015: contributor

    This is needed to compile the library as C++

  2. Fix compilation with C++ e2a07c78a1
  3. apoelstra commented at 3:54 PM on June 18, 2015: contributor

    How do you compile as C++? Is this something we can get Travis to do?

  4. sipa commented at 3:56 PM on June 18, 2015: contributor

    Judging by earlier comments by Vinnie, I assume he is directly #include'ing the .c source inside another project's C++ source code.

  5. vinniefalco commented at 4:01 PM on June 18, 2015: contributor

    @sipa Right

  6. apoelstra commented at 8:57 PM on June 29, 2015: contributor

    ACK

  7. gmaxwell commented at 9:01 PM on June 29, 2015: contributor

    For testing you can override CC=g++ and bypass the compile test in configure. I don't have a way to do the test without patching right now but perhaps its possible. A CI build that does this would be nice, otherwise it'll just be periodic fixes.

  8. sipa merged this on Jul 14, 2015
  9. sipa closed this on Jul 14, 2015

  10. sipa referenced this in commit b0a60e6d33 on Jul 14, 2015

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin-core/secp256k1. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-15 02:15 UTC

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