lib: use LYD_VALIDATE_MULTI_ERROR only if available #14831
Conversation
LYD_VALIDATE_MULTI_ERROR was added in libyang 2.1.36. The currently enforced minimum of libyang is 2.0.0. Stick an #ifdef around it. Fixes: 51a2a4b ("lib: print all errors when validating a config") Cc: Igor Ryzhov <[email protected]> Signed-off-by: David Lamparter <[email protected]>
|
Well, the fix makes sense, but I think we already actually require even higher version because of some other fixes in libyang that we rely on. @choppsv1 I think you recently changed some code that requires newer libyang? |
|
@idryzhov there's a bunch of version stuff going on, there's a range where things are broken in libyang that doesn't appear to have been packaged. We should up our requirement to 2.1.80 I believe. |
but we'll need to see how that affects OS release packages and if we care or not (i.e., just require build form source, or use frrouting package if the OS package is too old). |
LYD_VALIDATE_MULTI_ERROR was added in libyang 2.1.36. The currently enforced minimum of libyang is 2.0.0. Stick an #ifdef around it.
Fixes #14800