Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Use the libseccomp-rs/libseccomp-rs crate instead of youki original libseccomp-rs. #741

Merged
merged 3 commits into from
Mar 5, 2022

Conversation

utam0k
Copy link
Member

@utam0k utam0k commented Feb 27, 2022

Is this proposal acceptable to you?
https://github.com/libseccomp-rs/libseccomp-rs

@utam0k utam0k requested review from yihuaf and Furisto February 27, 2022 08:11
@utam0k utam0k force-pushed the feature/remove-original-libseccomp branch from 6199810 to 1dd3fa7 Compare February 27, 2022 08:11
@utam0k
Copy link
Member Author

utam0k commented Feb 27, 2022

Thanks to @ManaSugi, the author of libseccomp-rs. If you have something(e.g. not ready for production yet) to say, please let me know.

@codecov-commenter
Copy link

codecov-commenter commented Feb 27, 2022

Codecov Report

Merging #741 (7daed7a) into main (00ecbb8) will increase coverage by 0.00%.
The diff coverage is 51.35%.

@@           Coverage Diff            @@
##             main     #741    +/-   ##
========================================
  Coverage   72.23%   72.24%            
========================================
  Files          87       86     -1     
  Lines       11912    11756   -156     
========================================
- Hits         8605     8493   -112     
+ Misses       3307     3263    -44     

@utam0k utam0k force-pushed the feature/remove-original-libseccomp branch from c8d7e53 to 2b0f1f6 Compare February 27, 2022 08:17
Copy link
Contributor

@ManaSugi ManaSugi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@utam0k Thank you for adopting our crate! If there is something you'd like to fix, welcome any suggestions or PRs.

Copy link
Collaborator

@saschagrunert saschagrunert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome, this removes duplication. @utam0k to you wanna fix the open suggestion, otherwise LGTM

Copy link
Collaborator

@Furisto Furisto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

utam0k and others added 3 commits March 5, 2022 22:01
…ibseccomp-rs.

Signed-off-by: utam0k <k0ma@utam0k.jp>
Signed-off-by: utam0k <k0ma@utam0k.jp>
Co-authored-by: Manabu Sugimoto <66765615+ManaSugi@users.noreply.github.com>
@utam0k utam0k force-pushed the feature/remove-original-libseccomp branch from 7daed7a to 5916ff3 Compare March 5, 2022 13:02
@utam0k utam0k merged commit ba9215c into youki-dev:main Mar 5, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants