Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

compilation failure in src/test/alert_tests.cpp #20

Open
SachaSchlegel opened this issue Jan 8, 2018 · 3 comments
Open

compilation failure in src/test/alert_tests.cpp #20

SachaSchlegel opened this issue Jan 8, 2018 · 3 comments

Comments

@SachaSchlegel
Copy link

Running on commit: 2ba090b
Linux Ubuntu
uname -a: Linux MorgenLandBrain 4.13.0-21-generic #24-Ubuntu SMP Mon Dec 18 17:29:16 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux

nach git ckeckout ...


[Makefile.txt](https://github.com/faircoin/faircoin/files/1613379/Makefile.txt)
[Makefile.txt](https://github.com/faircoin/faircoin/files/1613381/Makefile.txt)
[config.log](https://github.com/faircoin/faircoin/files/1613382/config.log)
make clean
./autogen.sh
./configure
make

...
test/alert_tests.cpp: In member function ‘void Alert_tests::PartitionAlert::test_method()’:
test/alert_tests.cpp:203:55: error: ‘const struct Consensus::Params’ has no member named ‘nPowTargetSpacing’
     int64_t nPowTargetSpacing = params.GetConsensus().nPowTargetSpacing;
                                                       ^~~~~~~~~~~~~~~~~
test/alert_tests.cpp:224:5: error: ‘PartitionCheck’ was not declared in this scope
     PartitionCheck(falseFunc, csDummy, &indexDummy[99], nPowTargetSpacing);
     ^~~~~~~~~~~~~~
test/alert_tests.cpp:224:5: note: suggested alternative: ‘PartitionAlert’
     PartitionCheck(falseFunc, csDummy, &indexDummy[99], nPowTargetSpacing);
     ^~~~~~~~~~~~~~
     PartitionAlert


@thokon00
Copy link

thokon00 commented Jan 9, 2018

Thanks for the report. This is known issue, though. The test suite has not been ported to FairCoin yet. I hope someone can do it in 2018. There is a note at the bottom of the README.md about how to disable the test suite.

@jsalatas
Copy link

jsalatas commented Sep 1, 2018

Could you please provide more details on what needs to be done? I may be interested (and hopefully capable) of doing that.

BTW: I just noticed that while building fairchains and apparently it's the same issue. Right?

Thanks!

@user-invalid
Copy link

Ran into the same issue.
Configuring with --disable-tests --disable-bench as suggested in the README.md solved the problem.
Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants