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

GO: Adding eventfd2 syscall. #1203

Merged
merged 1 commit into from
Feb 23, 2025

Conversation

jozuenoon
Copy link
Contributor

Adding eventfd2 syscall, apparently used in Go from 1.23 golang/go#65443

I'm currently unable to submit Go code to judge due to 'RTE (eventfd2 syscall disallowed)'

@dmoj-build
Copy link
Collaborator

Can one of the admins verify this patch?

@kiritofeng
Copy link
Member

ok to test

 I'm currently unable to submit Go code to judge due to 'RTE (eventfd2 syscall disallowed)'

Signed-off-by: Witold Konior <witold.konior@qradium.com>
@jozuenoon jozuenoon force-pushed the go/add_eventfd2_syscall branch from f54bf79 to 3fee5c8 Compare February 23, 2025 21:38
Copy link

codecov bot commented Feb 23, 2025

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 50.63%. Comparing base (315ede5) to head (3fee5c8).
Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
dmoj/executors/GO.py 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           master    #1203       +/-   ##
===========================================
- Coverage   82.56%   50.63%   -31.93%     
===========================================
  Files         146      146               
  Lines        5666     5666               
===========================================
- Hits         4678     2869     -1809     
- Misses        988     2797     +1809     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Member

@kiritofeng kiritofeng left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@kiritofeng kiritofeng merged commit fa02703 into DMOJ:master Feb 23, 2025
10 checks passed
# 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.

3 participants