thom311 / libnl

Netlink Library Suite
GNU Lesser General Public License v2.1
423 stars 311 forks source link

route: Add extra support for bridge vlan and dynamic bridge table. #372

Closed Cordell-O closed 8 months ago

thom311 commented 10 months ago

Thank you for the patch!

There are some issues in CI (compiler warnings). Are you able to address those?

you can reformat the files by running ./tools/clang-format.sh. That requires that you have a suitable clang-format version installed. Alternatively, you can use ./tools/clang-format-container.sh (which requires a working podman and will build a Fedora container). If both is too cumbersome, you can also ignore the formatting issue, and I will reformat before merging.

Cordell-O commented 9 months ago

Sorry for the slow response time, ive address all feedback, unsure about the test suite any feedback is appreciated.

thom311 commented 8 months ago

lgtm. Merged to main.

Thank you for your contribution!!