Skip to content

Conversation

@ARH-MNAJS
Copy link
Contributor

@ARH-MNAJS ARH-MNAJS commented Oct 7, 2025

@pateljannat This PR introduces support for the C programming language by following the existing pattern for compiled runtimes like Go and Rust. It adds a new runtimes/c directory containing a Dockerfile and a run.sh script to handle the compile-then-run logic. The Makefile and server configuration have been updated to integrate the new runtime, and a new YAML test case is included for validation.

Please review for merging. Closes Issue #22

@pateljannat
Copy link
Contributor

@ARH-MNAJS Tests failing

@ARH-MNAJS
Copy link
Contributor Author

ARH-MNAJS commented Oct 10, 2025

@ARH-MNAJS Tests failing

@pateljannat I've pushed a fix for the test failure. The gcc: error: main.c: Permission denied error was caused by a file permission issue inside the CI container, where the script couldn't read the mounted source file. I've updated the run.sh script to copy the source file to /tmp before compiling, which resolves this problem.

@ARH-MNAJS ARH-MNAJS requested a review from pateljannat October 10, 2025 09:12
@pateljannat
Copy link
Contributor

@ARH-MNAJS Its still giving the permission denied error

ARH-MNAJS and others added 4 commits October 10, 2025 20:00
Added Manual trigger for testing
- Streamline run.sh by removing unnecessary copy operations
- Improve Dockerfile structure and user permission handling
@ARH-MNAJS
Copy link
Contributor Author

@ARH-MNAJS Its still giving the permission denied error

Fixed C runtime failing in CI with "Permission denied" errors by simplifying the Dockerfile to run as root instead of using a non-root appuser.
The issue was caused by UID/GID mismatches between host-mounted files and the container user, preventing gcc from reading source files. Removed user creation and privilege-dropping logic from both Dockerfile and run.sh. C runtime tests now pass successfully (test_c_hello.yml ✅).

The 3 remaining test failures (JavaScript runtime and 2 exec tests) are pre-existing issues unrelated to these changes. Overall test results: 14/17 passing

@pateljannat pateljannat merged commit 1617569 into fossunited:main Oct 13, 2025
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants