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

Remove remaining reference to log4cxx #40

Merged
merged 4 commits into from
Jul 27, 2018

Conversation

florent-lamiraux
Copy link
Member

No description provided.

@bchretien bchretien self-assigned this Jun 29, 2018
@bchretien
Copy link
Member

roboptim/roboptim-core-plugin-ipopt#38 merged, I'm re-running the CI build.

@bchretien bchretien added the bug label Jun 29, 2018
@bchretien
Copy link
Member

The CI build fails because of log4cxx logs that are generated in debug only. These should be removed as well.

@florent-lamiraux
Copy link
Member Author

Travis tests fail for reason out of the scope of this pull request:

  • 246.3: The command "pip install --user codecov" failed and exited with 127 during,
  • 246.4: line 4061, failure during compilation of roboptim-core,
  • 246.5: line 6605, failure in system headers,
  • 246.6: The command "pip install --user codecov" failed and exited with 127 during.

As the current master branch does not compile, is it possible to merge this pull request ?

@florent-lamiraux florent-lamiraux merged commit e13630d into roboptim:master Jul 27, 2018
@bchretien
Copy link
Member

There has been some issues with the OSX build (they removed the IPOPT formula and did not merge it in the core repository...). I haven't succeeded in fixing it for roboptim-core-plugin-ipopt, so I'll just ignore the failure here as well. The rest looks fine, thanks!

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

Successfully merging this pull request may close these issues.

2 participants