-
Notifications
You must be signed in to change notification settings - Fork 90
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
Add appveyor CI support #128
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like a start. But all the test executables fail during linking with ld.exe: cannot find -lrt
.
api_test
is about to be removed, so never mind that.
cmake --build . -j 2 --target real128_test; | ||
cmake --build . -j 2 --target ecc_test; | ||
cmake --build . -j 2 --target bip_lock; | ||
echo "Build finished!" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please run the tests after building. See .travis.yml
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Btw, shell scripts do not need a semicolon after each command. Only if another command follows on the same line.
7d1cb43
to
2a59f5e
Compare
Merge upstream
@cwyyprog are you still working on this? |
No, I'm not working on this due to lack of time.
On Sat, Nov 2, 2019 at 4:23 PM +0800, "Peter Conrad" <notifications@github.com> wrote:
@cwyyprog are you still working on this?
We've made quite a few changes to CMakeLists.txt, including the windows build.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
Appveyor CI build configuration using mingw64/msys2. I've tested it on my branch, and the latest build log is here https://ci.appveyor.com/project/cwyyprog/bitshares-fc . The api.exe takes more time than Appveyor's limit, so is not in the build targets.