Skip to content

Update docs, bump cmake version#575

Merged
fnc12 merged 7 commits intofnc12:devfrom
Leon0402:feature/dependencyManagment
Aug 3, 2020
Merged

Update docs, bump cmake version#575
fnc12 merged 7 commits intofnc12:devfrom
Leon0402:feature/dependencyManagment

Conversation

@Leon0402
Copy link
Copy Markdown
Contributor

@Leon0402 Leon0402 commented Aug 3, 2020

Hi,

I updated the docs and bumped the cmake version, which should fix the travis issue. How can I enable travis here, so I can see if the issue is actually resolved?

Copy link
Copy Markdown
Owner

@fnc12 fnc12 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Two issues:

  1. linux is still failing https://travis-ci.org . If it is not related to this PR we can skip it for now for you need to fix it. Or I can remove your old commit which broke linux CI compilation;
  2. please remove beerpay footer

@Leon0402
Copy link
Copy Markdown
Contributor Author

Leon0402 commented Aug 3, 2020

@fnc12 Where can I see the failing travis build? Can you enable it here, so I can see the output? Otherwise it's hard to fix

@fnc12
Copy link
Copy Markdown
Owner

fnc12 commented Aug 3, 2020

@Leon0402 Leon0402 marked this pull request as draft August 3, 2020 10:52
@Leon0402
Copy link
Copy Markdown
Contributor Author

Leon0402 commented Aug 3, 2020

@Leon0402
Copy link
Copy Markdown
Contributor Author

Leon0402 commented Aug 3, 2020

So I just had a look we need cmake 3.14 in order for this to work. But travis currently does only support cmake 3.12 ... ubuntu 20.04 is on the way for travis, but not currently available yet.
So I will add a ppa for cmake later this day, hope that's fine.

@fnc12
Copy link
Copy Markdown
Owner

fnc12 commented Aug 3, 2020

Why doesn't it show up here?

Don't know. I need to investigate it.

@Leon0402 Leon0402 marked this pull request as ready for review August 3, 2020 12:04
@Leon0402
Copy link
Copy Markdown
Contributor Author

Leon0402 commented Aug 3, 2020

Fixed, should be alright now!

@fnc12
Copy link
Copy Markdown
Owner

fnc12 commented Aug 3, 2020

thank you

@fnc12 fnc12 merged commit d97eedf into fnc12:dev Aug 3, 2020
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

Successfully merging this pull request may close these issues.

2 participants