-
Notifications
You must be signed in to change notification settings - Fork 11
PR to make the repository compiled on Humble #44
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
base: master
Are you sure you want to change the base?
Conversation
Fix compilation build for missing required dependency
|
@nim65s @saikishor could you have a final look to this PR for merging ? Thx in advance. |
|
@olivier-stasse I might have some changes locally. I can recheck and post here in a week. Next week I'll be at ERF |
| [submodule "cmake"] | ||
| path = cmake | ||
| url = git://github.com/jrl-umi3218/jrl-cmakemodules.git | ||
| url = https://github.com/jrl-umi3218/jrl-cmakemodules |
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.
Do we need this change?
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.
Thanks @saikishor for the feedback. For some reason, the git link on my side is timing out where as the https link allows a fast submodule pull. For this reason the following command:
git clone --recurse-submodules http://github.com/roboptim/roboptim-core-plugin-ipopt.git
is not working due to a long timeout.
The travis submodule for me is also a problem. I had to individually init and update the cmake module.
- If you confirm that you have the same problem then I could remove also travis which is deprecated.
- If you do not have the problem, then I wonder what is the problem on my side.
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.
I'll check and get back
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.
Yes, we need this change, ref https://github.blog/security/application-security/improving-git-protocol-security-github/#no-more-unauthenticated-git
This PR implements the following modification to make the repository compiled under Humble: