Skip to content

Adjust controller_manager library destination#177

Closed
bmagyar wants to merge 1 commit intoros-controls:masterfrom
bmagyar:adjust_library_destination
Closed

Adjust controller_manager library destination#177
bmagyar wants to merge 1 commit intoros-controls:masterfrom
bmagyar:adjust_library_destination

Conversation

@bmagyar
Copy link
Copy Markdown
Member

@bmagyar bmagyar commented Oct 3, 2020

No description provided.

@bmagyar bmagyar requested review from Karsten1987 and destogl October 3, 2020 07:24
@destogl
Copy link
Copy Markdown
Member

destogl commented Oct 3, 2020

with this changes I get the following error:

Package 'controller_manager' exports the library 'controller_manager' which
  couldn't be found

The PR is related to #172

@Karsten1987
Copy link
Copy Markdown
Contributor

I don't think there's anything to change here. The install directory is correct for this target.

@destogl destogl mentioned this pull request Oct 7, 2020
4 tasks
@destogl
Copy link
Copy Markdown
Member

destogl commented Oct 8, 2020

And what about export?

Should we there prefix ${PROJECT_NAME}?

@Karsten1987
Copy link
Copy Markdown
Contributor

And what about export?

starting with Foxy, we can rely on modern cmake within ament. I wouldn't bother with this right now and could see this as a follow up PR which addresses this altogether on all repositories.

@bmagyar
Copy link
Copy Markdown
Member Author

bmagyar commented Oct 20, 2020

agreed

@bmagyar bmagyar closed this Oct 20, 2020
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

Successfully merging this pull request may close these issues.

3 participants