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.

Sign up for GitHub

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

Jump to bottom

libnl merge #1653

Merged
merged 3 commits into from
Jul 23, 2024
Merged

libnl merge #1653

merged 3 commits into from
Jul 23, 2024

Conversation

tpashkin
Copy link
Collaborator

tpashkin and others added 3 commits July 19, 2024 16:03
@tpashkin
import libnl to contrib; issue-1166
ffefd23
@tpashkin
refactor netlink device constructor and Start method
c2614b8 
* move index parsing from constructor to Start method, making it nothrow
* restructure endpoint_proxy/server code to reduce nesting
@tpashkin
provide cmake files and missing linux headers
12ab2f2
@tpashkin tpashkin added the blockstore Add this label to run only cloud/blockstore build and tests on PR label Jul 22, 2024
@tpashkin
Copy link
Collaborator Author

Не стал разделять CMakeLists по платформам, как это сделано для автогенерированных файлов. Не вижу особого смысла, так как libnl всё равно нигде кроме линукса не соберётся, а сборку под арм мне негде тестировать

@tpashkin
Copy link
Collaborator Author

tpashkin commented Jul 22, 2024 •
edited
Loading

Чтобы собралась nl-route-3, пришлось портировать из main 3 линуксовых хэдера:

  • linux/ip.h
  • linux/if_tunnel.h
  • linux/stddef.h – этот был старый, я его обновил

@github-actions GitHub Actions
Copy link
Contributor

Note

This is an automated comment that will be appended during run.

🟢 default-linux-x86-64-relwithdebinfo: all tests PASSED for commit 12ab2f2.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
1551 1551 0 0 0 0

@tpashkin tpashkin merged commit e7846e5 into stable-23-3 Jul 23, 2024
9 of 12 checks passed
@tpashkin tpashkin deleted the users/tpashkin/stable-23-3/libnl-contrib branch July 23, 2024 08:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@budevg budevg budevg approved these changes

@drbasic drbasic drbasic approved these changes

@qkrorlqr qkrorlqr Awaiting requested review from qkrorlqr

Assignees
No one assigned
Labels
blockstore Add this label to run only cloud/blockstore build and tests on PR
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@tpashkin @budevg @drbasic