Skip to content

Delete manifest.xml (backport #26)#28

Merged
ivanpauno merged 1 commit intofoxyfrom
mergify/bp/foxy/pr-26
Sep 21, 2022
Merged

Delete manifest.xml (backport #26)#28
ivanpauno merged 1 commit intofoxyfrom
mergify/bp/foxy/pr-26

Conversation

@mergify
Copy link
Copy Markdown

@mergify mergify Bot commented Sep 21, 2022

This is an automatic backport of pull request #26 done by Mergify.


Mergify commands and options

More conditions and actions can be found in the documentation.

You can also trigger Mergify actions by commenting on this pull request:

  • @Mergifyio refresh will re-evaluate the rules
  • @Mergifyio rebase will rebase this PR on its base branch
  • @Mergifyio update will merge the base branch into this PR
  • @Mergifyio backport <destination> will backport this PR on <destination> branch

Additionally, on Mergify dashboard you can:

  • look at your merge queues
  • generate the Mergify configuration with the config editor.

Finally, you can contact us on https://mergify.com

Remove manifest.xml. For some reason when trying to install dependencies via rosdep, the dependencies defined in manifest.xml are used first. Because of that dependencies such as eigen3_cmake_module are not installed. This causes compilation via colcon to fail.

(cherry picked from commit f48062a)
@mergify mergify Bot mentioned this pull request Sep 21, 2022
@ivanpauno
Copy link
Copy Markdown
Member

ivanpauno commented Sep 21, 2022

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Windows Build Status

@ivanpauno ivanpauno merged commit 5f1ca5e into foxy Sep 21, 2022
@delete-merged-branch delete-merged-branch Bot deleted the mergify/bp/foxy/pr-26 branch September 21, 2022 15:23
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.

2 participants