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

[21256] Fix issue with Fast DDS and Fast DDS python deduced branches #838

Merged
merged 1 commit into from
Jul 2, 2024

Conversation

JesusPoderoso
Copy link
Contributor

@JesusPoderoso JesusPoderoso commented Jul 2, 2024

Description

In the previous PR

The Fast DDS and Fast DDS python related branches were not used as expected:
Instead of using the provided branch, it was downloading the .repos file from that branch (so the base reference was used instead of the desired branch).
This hotfix PR performs a checkout before downloading the repos included in the .repos file, so the actual branch used is the desired one.
In the case of the Fast DDS Python, its dependencies are already downloaded, so there is no need to download them. It has been changed to just cloning that repo and checking out to the desired branch.

Important: when fixing the conflicts in the backports, please ensure:

  • Change fastdds.repos to fastrtps.repos
  • change src/fastdds to src/fastrtps

@Mergifyio backport 2.14.x 2.13.x 2.10.x 2.6.x

Contributor Checklist

  • Commit messages follow the project guidelines.
  • N/A Code snippets related to the added documentation have been provided.
  • N/A Documentation tests pass locally.
  • Applicable backports have been included in the description.

Reviewer Checklist

  • The PR has a milestone assigned.
  • The title and description correctly express the PR's purpose.
  • Check contributor checklist is correct.
  • CI passes without warnings or errors.

@JesusPoderoso JesusPoderoso added this to the v3.0.0 milestone Jul 2, 2024
@JesusPoderoso JesusPoderoso self-assigned this Jul 2, 2024
@JesusPoderoso JesusPoderoso requested a review from elianalf July 2, 2024 06:34
@JesusPoderoso
Copy link
Contributor Author

@Mergifyio backport 2.14.x 2.13.x 2.10.x 2.6.x

Copy link

mergify bot commented Jul 2, 2024

backport 2.14.x 2.13.x 2.10.x 2.6.x

✅ Backports have been created

@EduPonz EduPonz merged commit 6a53b9e into master Jul 2, 2024
4 checks passed
@EduPonz EduPonz deleted the hotfix/21256 branch July 2, 2024 12:14
mergify bot pushed a commit that referenced this pull request Jul 2, 2024
Signed-off-by: JesusPoderoso <[email protected]>
(cherry picked from commit 6a53b9e)

# Conflicts:
#	.github/workflows/reusable-ubuntu-ci.yml
mergify bot pushed a commit that referenced this pull request Jul 2, 2024
Signed-off-by: JesusPoderoso <[email protected]>
(cherry picked from commit 6a53b9e)

# Conflicts:
#	.github/workflows/reusable-ubuntu-ci.yml
mergify bot pushed a commit that referenced this pull request Jul 2, 2024
Signed-off-by: JesusPoderoso <[email protected]>
(cherry picked from commit 6a53b9e)

# Conflicts:
#	.github/workflows/reusable-ubuntu-ci.yml
mergify bot pushed a commit that referenced this pull request Jul 2, 2024
Signed-off-by: JesusPoderoso <[email protected]>
(cherry picked from commit 6a53b9e)

# Conflicts:
#	.github/workflows/reusable-ubuntu-ci.yml
JesusPoderoso added a commit that referenced this pull request Jul 3, 2024
Signed-off-by: JesusPoderoso <[email protected]>
(cherry picked from commit 6a53b9e)
JesusPoderoso added a commit that referenced this pull request Jul 3, 2024
Signed-off-by: JesusPoderoso <[email protected]>
(cherry picked from commit 6a53b9e)
JesusPoderoso added a commit that referenced this pull request Jul 3, 2024
Signed-off-by: JesusPoderoso <[email protected]>
(cherry picked from commit 6a53b9e)
JesusPoderoso added a commit that referenced this pull request Jul 3, 2024
Signed-off-by: JesusPoderoso <[email protected]>
(cherry picked from commit 6a53b9e)
JesusPoderoso added a commit that referenced this pull request Jul 3, 2024
…839)

Signed-off-by: JesusPoderoso <[email protected]>
(cherry picked from commit 6a53b9e)

Co-authored-by: Jesús Poderoso <[email protected]>
EduPonz pushed a commit that referenced this pull request Jul 3, 2024
…841)

Signed-off-by: JesusPoderoso <[email protected]>
(cherry picked from commit 6a53b9e)

Co-authored-by: Jesús Poderoso <[email protected]>
EduPonz pushed a commit that referenced this pull request Jul 3, 2024
…842)

Signed-off-by: JesusPoderoso <[email protected]>
(cherry picked from commit 6a53b9e)

Co-authored-by: Jesús Poderoso <[email protected]>
EduPonz pushed a commit that referenced this pull request Jul 3, 2024
…840)

Signed-off-by: JesusPoderoso <[email protected]>
(cherry picked from commit 6a53b9e)

Co-authored-by: Jesús Poderoso <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants