-
Notifications
You must be signed in to change notification settings - Fork 18
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
Update the FC submodule with the changes for GRPH-4 #121
Conversation
Fixes issue #119 |
trying, thanks. |
Failing for me:
|
@oxarbitrage Did you update the submodule before compiling? Since it shows undefined references, it may also be that you have to recompile from scratch. |
Yes i did that, still failing with the error i sent from here. Any other can give it a try ? |
Did you try compiling in a new (empty) directory? Apart from that, it could be that you're linking against and old version of |
hmm, here are the commands i am using for a clean installation:
|
Using the steps above I can finally this. Maybe it's the difference between compiling in-source and out-of-source, as I was doing before. I will try to fix this and update this PR. |
@oxarbitrage I found the problem and fixed it, now it should compile successfully. Thank you for your help. |
@RoshanSyed has made the CI available here : https://gitlab.com/PBSA/peerplays/pipelines |
@RoshanSyed I'm assuming we're not moving the main repo to GitLab. In this case, can you make the CI update the commit status on GitHub as well? It's good to see the checkmark in the PR. |
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.
thank you @Noughmad
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.
LGTM
fc
submodule