Skip to content

Conversation

@erlingrj
Copy link
Collaborator

@erlingrj erlingrj commented Feb 21, 2025

THis PR adds a lfc --version and lingo --version to CI so we can verify that they were successfully installed, and are available on the path.

This currently fails for macOS

@erlingrj
Copy link
Collaborator Author

@axmmisaka it appears as the fix with adding a && break to the macOS installation just hid the error and that it still does not work

@erlingrj erlingrj changed the title Remove break-statement from macOS installation Verify installation in CI Feb 21, 2025
@axmmisaka
Copy link
Contributor

Let me check what might have happened 😥

@axmmisaka axmmisaka mentioned this pull request Feb 23, 2025
@axmmisaka
Copy link
Contributor

@axmmisaka it appears as the fix with adding a && break to the macOS installation just hid the error and that it still does not work

It's still curl issue. It was impossible to debug because we were suppressing curl output with -s. I'll try to push a upstream fix.

@axmmisaka
Copy link
Contributor

Once lf-lang/installation#31 is merged, can you please stress-check axmmisaka@7f60b1e a bit and see if it could fix the bug... 😥 thanks!

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