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

app-colcon.meta vs colcon.meta #220

Open
flabrosse opened this issue Jan 5, 2024 · 3 comments
Open

app-colcon.meta vs colcon.meta #220

flabrosse opened this issue Jan 5, 2024 · 3 comments

Comments

@flabrosse
Copy link
Contributor

This is not really an issue, but more to document what I found out after spending a lot of time having compilation errors.

A lot of the online documentation mentions the need for a file named colcon.meta, eg to use a different transport mechanism. Not much of the documentation mentions a file named app-colcon.meta.

Renaming my colcon.meta to app-colcon.meta solved my compilation problem relating to RMW_UXRCE_TRANSPORT_CUSTOM not being defined.

@pablogs9
Copy link
Member

pablogs9 commented Jan 8, 2024

Both shall be used as you can see here:

--metas $(EXTENSIONS_DIR)/colcon.meta $(APP_COLCON_META) \

In that sense, colcon.meta is intended to configure the library, and app-colcon.meta is a sort of override for application-specific configurations in the number of entities (like this)

Make sure you clean the library completely before modifying it and changing colcon meta parameters.

@flabrosse
Copy link
Contributor Author

Yes, of course. My point is that there is documentation out there that specifies colcon.meta in an application, not in the library. I initially had a colcon.meta, inherited from a previous developer, that worked with the galactic version. When I changed to humble, it was complaining about the transport not being configured. Changing the file to app-colcon.meta fixed it.

Yes, systematically was doing clean-microros, clean and fullclean.

@pablogs9
Copy link
Member

pablogs9 commented Jan 8, 2024

AFAIK the only documentation that this package has is its own README.
And you are right, app-colcon.meta is not documented there.

Contributions are welcomed, please feel free to open a PR to include a paragraph mentioning it.

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

2 participants