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

[FRR]: Use stg in a proper way. #3589

Merged
merged 3 commits into from
Oct 15, 2019
Merged

Conversation

wangshengjun
Copy link
Contributor

Signed-off-by: wangshengjun [email protected]

- What I did
use stg branch instead of git branch.
- How I did it

- How to verify it

- Description for the changelog

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

@@ -4,20 +4,24 @@ SHELL = /bin/bash

MAIN_TARGET = $(FRR)
DERIVED_TARGET = $(FRR_PYTHONTOOLS) $(FRR_DBG) $(FRR_SNMP) $(FRR_SNMP_DBG)
BRANCH = $(shell date +%Y%m%d\.%H%M%S)
FRR_VERSION = 7.1
Copy link
Collaborator

@nikos-github nikos-github Oct 11, 2019

Choose a reason for hiding this comment

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

You don't need to define it. It's exported from rules/frr.mk: export FRR_VERSION FRR_SUBVERSION along with the subversion so we can just use it.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for your comment! I will remove it.

@pavel-shirshov pavel-shirshov merged commit 34d8842 into sonic-net:master Oct 15, 2019
lguohan pushed a commit that referenced this pull request Nov 4, 2019
…3692)

Fixed Makefile of FRR. Before we had issues after #3589:

- When you want to rebuild frr with new changes you get error "branch frr/7.1 is already exist".
- When your patch list is empty stg undo gives an error
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.

4 participants