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

fix quoting of error messages in catkin/cmake/em/order_packages.cmake.em #667

Closed
rhaschke opened this issue Jun 27, 2014 · 6 comments
Closed

Comments

@rhaschke
Copy link
Contributor

I get the following order_packages.cmake:
# generated from catkin/cmake/em/order_packages.cmake.em message(FATAL_ERROR "Invalid package manifest "/homes/rhaschke/src/ros/rtt/install_isolated/lib/ruby/1.9.1/x86_64-linux/orogen/templates/typekit/package.xml": The manifest contains invalid XML: not well-formed (invalid token): line 2, column 9")

which triggers the cmake error:

Argument not separated from preceding token by whitespace.

@tfoote
Copy link
Member

tfoote commented Jun 27, 2014

Note i believe this is an orogen package.xml which is different from a ros one.

@rhaschke
Copy link
Contributor Author

I know, that the package.xml is wrong. However, catkin should better
deal with catkin-generated exception that contains double-quotes.
These shouldn't be written literally into the order_packages.cmake file.

@dirk-thomas
Copy link
Member

Could it be that the message you are referring to ("Argument not separated from preceding token by whitespace.") is only a CMake warning and that you are using CMake 2.8.12 (where this warning has been introduced)?

Can you please provide more information to reproduce the problem? What exactly is in your workspace and what command are you invoking?

@rhaschke
Copy link
Contributor Author

rhaschke commented Jul 3, 2014

Indeed, I'm using cmake 2.8.12.2 on Ubuntu Trusty. The message is only a
warning - but annoying. It originates from a non-ROS package.xml (attached).
I'm suggesting to fix the error handling routine in catkin, that finally
generated the error message indicating the malformed package.xml.

Kind regards, Robert

On 29.06.2014 02:43, Dirk Thomas wrote:

Could it be that the message you are referring to ("Argument not
separated from preceding token by whitespace.") is only a CMake
warning and that you are using CMake 2.8.12 (where this warning has
been introduced)?

Can you please provide more information to reproduce the problem? What
exactly is in your workspace and what command are you invoking?


Reply to this email directly or view it on GitHub
#667 (comment).

Robert Haschke


Dr. Robert Haschke
Faculty of Technology, Neuroinformatics Group, CITEC-3.309
Bielefeld University, 33615 Bielefeld, Germany
phone: +49-521-106 12122
fax: +49-521-106 6011
mobile: +49-157-817 365 13
WWW: www.neuroinformatik.de

@dirk-thomas
Copy link
Member

You can not attach files when replying to the GitHub issue emails. The quoting has already been addressed in the referenced PR #668.

Please verify if the problem has been fixed with an up-to-date checkout of catkin in your workspace and post your results. If the issue is still there please post the content of the package.xml in question.

@rhaschke
Copy link
Contributor Author

rhaschke commented Jul 4, 2014

Thanks, the patch fixed the issue.

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

No branches or pull requests

3 participants