Skip to content
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

[lets-rust] Get CI for Linux/macOS working #187

Closed
wants to merge 18 commits into from

Conversation

kivikakk
Copy link
Collaborator

@kivikakk kivikakk commented Aug 5, 2022

Was too curious and explored this messily. Still need to install dev tools on Windows or something?

The trick was:

  1. not to chdir, just stay put wherever mini_portile puts us; and,
  2. build comrak's c-api as a staticlib only; wasn't required for the Mac build to work (see 9565657 build logs), maybe the build prefers .a to .dylib, but on Linux it's like "hey where's my libcomrak_ffi.so".

If this looks good to you, I say we merge kivikakk/comrak@c-api-staticlib and then squash this all into #185 (minus the branch name change).

@kivikakk kivikakk closed this Aug 5, 2022
@kivikakk kivikakk deleted the lets-rust-where_r_u branch August 5, 2022 08:51
@kivikakk
Copy link
Collaborator Author

kivikakk commented Aug 5, 2022

Squashed into #185 as 9962914!

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.

1 participant