Skip to content

Support Iron distro#2787

Merged
Nir-Az merged 3 commits intorealsenseai:ros2-developmentfrom
SamerKhshiboun:add-iron
Jun 22, 2023
Merged

Support Iron distro#2787
Nir-Az merged 3 commits intorealsenseai:ros2-developmentfrom
SamerKhshiboun:add-iron

Conversation

@SamerKhshiboun
Copy link
Contributor

@SamerKhshiboun SamerKhshiboun commented Jun 21, 2023

Tracked by RSDEV-353

  • Iron to be added to pre-release when we release librealsense package in ROS build farm, till then we will get error in the script

@SamerKhshiboun SamerKhshiboun force-pushed the add-iron branch 5 times, most recently from dfa9899 to 79956fe Compare June 21, 2023 21:03
@SamerKhshiboun SamerKhshiboun changed the title DRAFT - Add iron Support Iron distro Jun 21, 2023
@SamerKhshiboun SamerKhshiboun self-assigned this Jun 21, 2023
@SamerKhshiboun SamerKhshiboun requested a review from Nir-Az June 21, 2023 21:49
fail-fast: false
matrix:
ros_distro: [humble, foxy]
ros_distro: [humble]
Copy link
Contributor Author

@SamerKhshiboun SamerKhshiboun Jun 21, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

will add iron here only when it builds and be ready on the ROS2 public package index
meanwhile, if we add it here, it will fail, since it will not find librealsense2 package for iron.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I say let's add it and ignore Iron failure, it should be fixed once it's merged , right?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

@Nir-Az Nir-Az merged commit e6e7f95 into realsenseai:ros2-development Jun 22, 2023
@SamerKhshiboun SamerKhshiboun deleted the add-iron branch July 25, 2023 10:27
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