Skip to content

Improvements to unit test setup#363

Closed
vegard wants to merge 6 commits intobitcoin:masterfrom
vegard:testing
Closed

Improvements to unit test setup#363
vegard wants to merge 6 commits intobitcoin:masterfrom
vegard:testing

Conversation

@vegard
Copy link
Copy Markdown
Contributor

@vegard vegard commented Jun 29, 2011

I made some changes in anticipation of unit tests for scripts and OP_CHECKSIG, etc.

Feedback is welcome.

vegard added 6 commits June 29, 2011 12:54
There were some problems with the existing testing setup:

 - Makefile rules for test-file compilation used CFLAGS instead of
   CXXFLAGS in makefile.unix
 - Main test file included other test files verbatim using #include
 - Test files included bitcoin headers using #include "../foo.h"
   (this breaks when headers include other headers), so add -I. to
   the flags to make #include "foo.h" include the top-level foo.h.

(Only tested on Linux)
This is needed for the script unit tests.
@gavinandresen
Copy link
Copy Markdown
Contributor

Nice! Thanks!

@TheBlueMatt
Copy link
Copy Markdown
Contributor

Looks cool, however the -I. stuff kills MinGW building, so NACK that part.

@gavinandresen
Copy link
Copy Markdown
Contributor

Is there a standard makefile syntax for "the directory that the makefile is in" ?

(it has been many years since I wrestled with makefiles....)

@gavinandresen
Copy link
Copy Markdown
Contributor

vegard: ping? I'd very much like to pull this, will you have time soon to fix the mingw and undefined symbol problems?

@vegard
Copy link
Copy Markdown
Contributor Author

vegard commented Jul 18, 2011

On 18 July 2011 03:06, gavinandresen
reply@reply.github.com
wrote:

vegard: ping?  I'd very much like to pull this, will you have time soon to fix the mingw and undefined symbol problems?

I'm not sure what to do about them. I don't have windows or mingw, and
I don't see any undefined symbols here.

I can try to have a look next week.

@gavinandresen
Copy link
Copy Markdown
Contributor

I'd suggest jumping on IRC and see if you can recruit somebody to help test/fix... (BlueMatt is very helpful).

@gavinandresen
Copy link
Copy Markdown
Contributor

Pulled versions of these, Matt fixed them up.

lateminer pushed a commit to lateminer/bitcoin that referenced this pull request Oct 16, 2019
Losangelosgenetics pushed a commit to Losangelosgenetics/bitcoin that referenced this pull request Mar 12, 2020
cryptapus pushed a commit to cryptapus/bitcoin that referenced this pull request May 3, 2021
…gitian keys

4c72d1f remove midnight's gitian signing key from list of gitian keys (Midnight Magic)

Pull request description:

  as per subject..

ACKs for top commit:
  domob1812:
    ACK 4c72d1f.

Tree-SHA512: 5cd18ad676c5ff01c194734a3c2c2257850c10e61d8a8b5fabc4423c8ebcfce9f25d03d0fc222b16c64bdac3c7395e4fc075e77afccd082c01ff3888dd3536d5
rajarshimaitra pushed a commit to rajarshimaitra/bitcoin that referenced this pull request Aug 5, 2021
Node Operations - Submarine Swap
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants