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

[sonic-frr/Makefile]: Checkout gracefully if branch exists already. #8289

Merged
merged 2 commits into from
Aug 15, 2021

Conversation

praveen-li
Copy link
Member

This is useful to avoid "branch exists" error while rebuild FRR.

Signed-off-by: Praveen Chaudhary[email protected]

Why I did it

If branch exist already, today FRR build fails. Branch may exist on rebuild.
With this PR, we will Checkout branch gracefully if branch exists already.

How I did it

Use git checkout $(FRR_BRANCH) || git checkout -b $(FRR_BRANCH) origin/$(FRR_BRANCH) in src/sonic-frr/Makefile.

How to verify it

Rebuild works fine. Fresh build works fine as well.

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106

Description for the changelog

A picture of a cute animal (not mandatory but encouraged)

This is useful to avoid "branch exists" error while rebuild FRR.

Signed-off-by: Praveen Chaudhary<[email protected]>
@praveen-li praveen-li requested a review from lguohan as a code owner July 30, 2021 01:20
@lguohan lguohan requested a review from shi-su August 3, 2021 16:40
src/sonic-frr/Makefile Outdated Show resolved Hide resolved
Signed-off-by: Praveen Chaudhary <[email protected]>
@lguohan lguohan merged commit 144a7bc into sonic-net:master Aug 15, 2021
praveen-li pushed a commit to praveen-li/sonic-buildimage that referenced this pull request Feb 15, 2022
…#8289)

If branch exist already, today FRR build fails. Branch may exist on rebuild.
With this PR, we will Checkout branch gracefully if branch exists already.

Signed-off-by: Praveen Chaudhary<[email protected]>
mssonicbld pushed a commit to mssonicbld/sonic-buildimage that referenced this pull request Jun 12, 2023
…#8289)

If branch exist already, today FRR build fails. Branch may exist on rebuild.
With this PR, we will Checkout branch gracefully if branch exists already.

Signed-off-by: Praveen Chaudhary<[email protected]>
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202106: #15434

@liushilongbuaa
Copy link
Contributor

Approve for 202106 branch to avoid build failure.

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.

5 participants