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

lib: small fix for NBC #define name change in libyang3 #16514

Closed

Conversation

choppsv1
Copy link
Contributor

@choppsv1 choppsv1 commented Aug 5, 2024

No description provided.

@choppsv1 choppsv1 force-pushed the chopps/libyang3-fix branch from eea3877 to 150ead7 Compare August 5, 2024 00:44
@choppsv1
Copy link
Contributor Author

choppsv1 commented Aug 5, 2024

same fix but done differently than #16511

@choppsv1
Copy link
Contributor Author

choppsv1 commented Aug 5, 2024

https://github.com/Mergifyio backport stable/10.1

Copy link

mergify bot commented Aug 5, 2024

backport stable/10.1

🟠 Waiting for conditions to match

  • merged [📌 backport requirement]

@vjardin
Copy link
Contributor

vjardin commented Aug 6, 2024

OK for me,

side note, it matches the documentation of libyang migration:
https://github.com/CESNET/libyang/blob/fba28260f382d81cf8f4b91b24cd717b52324fc2/doc/transition_2_3.dox#L40

@choppsv1
Copy link
Contributor Author

fixed by #16511

@choppsv1 choppsv1 closed this Aug 10, 2024
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.

2 participants