Skip to content

Add Porffor JS compiler #1605

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 1 commit into
base: main
Choose a base branch
from
Open

Conversation

lleyton
Copy link

@lleyton lleyton commented May 7, 2025

@mattgodbolt
Copy link
Member

This fails to install:

2025-05-12 12:22:42,661 lib.installation_context INFO     Moving from staging (/opt/compiler-explorer/staging/90aa4977-9d3b-44cc-b43d-bfc2f941c0a9/package) to final destination (/opt/compiler-explorer/porffor-0.57.24)
/opt/compiler-explorer/porffor-0.57.24/porf: 2: node: not found
2025-05-12 12:22:42,665 lib.ce_install  ERROR    compilers/javascript/porffor 0.57.24 installed OK, but doesn't appear as installed after
0 packages installed OK, 0 skipped, and 1 failed installation
Failed:
  compilers/javascript/porffor 0.57.24

@mattgodbolt
Copy link
Member

(we don't have node installed globally)

@mattgodbolt
Copy link
Member

We may have to work around this by using check_file instead of check_exe; the installation environment doesn't have node, nor will it be on the path. Either that or we declare a dependency on node of some sort and run it there. Either way the compiler explorer code will hit the same problem: unpacking javascript tarballs isn't something we've done before.

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