Skip to content

[closes #56] Use peregrine to compile to multiple targets - #57

Merged
omelkonian merged 2 commits into
masterfrom
orestis/peregrine-output
Feb 23, 2026
Merged

omelkonian merged 2 commits into
masterfrom
orestis/peregrine-output

Conversation

@omelkonian

Copy link
Copy Markdown
Collaborator

No description provided.

@omelkonian
omelkonian force-pushed the orestis/peregrine-output branch from 6031457 to 406d150 Compare February 20, 2026 18:39
@omelkonian
omelkonian marked this pull request as draft February 20, 2026 19:51
@4ever2

4ever2 commented Feb 20, 2026

Copy link
Copy Markdown
Contributor

Good addition. Compiling Peregrine takes a while, unfortunately caching doesn't work well with the setup-ocaml action. If this becomes a problem, we could consider publishing archives containing just the ml sources needed to build Peregrine.

Maybe it would be worth adding a --out-dir option to Peregrine, this would simplify the compile script a bit since the ext switch wouldn't be needed.

@omelkonian
omelkonian force-pushed the orestis/peregrine-output branch from 406d150 to dfb9334 Compare February 23, 2026 19:24
@omelkonian
omelkonian marked this pull request as ready for review February 23, 2026 20:39
@omelkonian
omelkonian merged commit f9f8faa into master Feb 23, 2026
1 check passed
@omelkonian
omelkonian deleted the orestis/peregrine-output branch February 23, 2026 20:40
@omelkonian

Copy link
Copy Markdown
Collaborator Author

@4ever2 Indeed it's a bit tedious to have to wait a full hour; please do provide a CI caching fix if you are aware of a solution (e.g. using another action, publishing archives).

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