Skip to content

Fix compile and c build #1185

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.

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

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

atticusofsparta
Copy link

@atticusofsparta atticusofsparta commented Mar 15, 2025

Updates deno cliffy dep and removes unused c deps

Also adds an action to verify that template languages (c and lua for now) can be created and build

@atticusofsparta atticusofsparta force-pushed the fix-compile-and-c-build branch 7 times, most recently from ebcb99b to c5cf925 Compare March 15, 2025 23:31
@atticusofsparta atticusofsparta force-pushed the fix-compile-and-c-build branch 13 times, most recently from e2bf556 to 579724e Compare March 16, 2025 00:22
@atticusofsparta atticusofsparta force-pushed the fix-compile-and-c-build branch from 579724e to 6654a0b Compare March 16, 2025 00:26

def inject_c_files(definition: Definition, c_program: str, c_source_files: list, link_libraries: list):
def inject_c_files(definition: Definition, c_program: str, c_source_files: list):
Copy link
Contributor

Choose a reason for hiding this comment

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

I would really like this fix to be merged in.

@MichaelBuhler
Copy link
Contributor

@atticusofsparta Why are the check (build tests) failing?

@MichaelBuhler
Copy link
Contributor

Closes #1205.

@MichaelBuhler
Copy link
Contributor

Duplicates #1116 and #1119

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.

3 participants