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

suggest what to do when CATKIN_NONHOMOGENEOUS_WORKSPACE #681

Closed
wants to merge 1 commit into from

Conversation

k-okada
Copy link
Contributor

@k-okada k-okada commented Sep 8, 2014

@jbohren
Copy link
Member

jbohren commented Sep 8, 2014

@k-okada it's currently in beta, but I'd suggest trying the new top-level build tool from the catkin_tools package: http://catkin-tools.readthedocs.org/en/latest/

This tool can build catkin and cmake packages simultaneously in the same workspace and can do this in parallel.

Also there are a bunch of additional features in my branch which are still pending review:

http://jbohren-ct.readthedocs.org/en/metadata-file/index.html
https://github.com/jbohren-forks/catkin_tools

@k-okada
Copy link
Contributor Author

k-okada commented Sep 8, 2014

@jbohren thanks for info, I'm aware of that, because dave colman was here until last month and he always show me how your tools are excellent. Do you have plan to release that package as deb or pip? This is my fault, but I have psychological barrier to install new tools from source.

@jbohren
Copy link
Member

jbohren commented Sep 8, 2014

thanks for info, I'm aware of that, because dave colman was here until last month and he always show me how your tools are excellent. Do you have plan to release that package as deb or pip? This is my fault, but I have psychological barrier to install new tools from source.

Well it will definitely be released after my pull request ( catkin/catkin_tools#80 ) is merged, but that's still pending review by @wjwwood. That probably won't happen until after ROSCon.

@wjwwood
Copy link
Member

wjwwood commented Sep 8, 2014

We aren't going to suggest catkin_tools over cmi until several things are fixed in catkin_tools, see:

Do you have plan to release that package as deb or pip?

It has already been released once, via apt-get get it is python-catkintools (this will be fixed to be python-catkin-tools in the next release) and via pip it is catkin_tools.

As the original issue, I think it would be a good hint to have in the error message, please consider a pull request:

https://github.com/ros/catkin/blob/indigo-devel/cmake/catkin_workspace.cmake#L88

As @jbohren pointed out, until after ROSCon (this week) we will probably be booked solid.

@dirk-thomas
Copy link
Member

Thank you, merged (with slight modification) in c58b1d7.

@k-okada
Copy link
Contributor Author

k-okada commented Sep 19, 2014

thank you for merging and correct English ;-),
Try "the" XYZ "instead". I'll use this expression next time.

@k-okada k-okada deleted the add_warning_comment branch September 19, 2014 00:38
@k-okada
Copy link
Contributor Author

k-okada commented Oct 24, 2014

I just want to say catkin_tools is great, thanks!!! @jbohren ⭐⭐⭐⭐⭐

cwecht pushed a commit to cwecht/catkin that referenced this pull request Mar 20, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants