-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
building-frr-for-centos7: fix frr config path error #16515
base: master
Are you sure you want to change the base?
Conversation
5be0535
to
e4916cc
Compare
FRR sets sysconfdir based upon what is passed in on |
e4916cc
to
6b0b14f
Compare
Yes, so change it in this PR. Thank a lot. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need to apply such a change for all the examples we have?
6b0b14f
to
1b60b21
Compare
Thanks! |
1b60b21
to
bb06a47
Compare
add --sysconfdir=/etc. Signed-off-by: xuweiwei <[email protected]>
bb06a47
to
d75df74
Compare
can I get an output of your config.log as I asked for? I would like to understand how we got here and this would help and that is why I asked for it |
Thanks for your review.
By default, it config sysconfdir='/usr/local/etc/frr' , But it use /etc/frr/ as frr confdir in the document .
|
According to the
building-frr-for-centos7
document, the following errors were found during building: