Skip to content

Commit

Permalink
Set HOMEBREW_NO_INSTALLED_DEPENDENTS_CHECK to 1 on the runner whi…
Browse files Browse the repository at this point in the history
…le installing OCaml

Signed-off-by: Sora Morimoto <[email protected]>
  • Loading branch information
smorimoto committed Dec 9, 2023
1 parent 7ac5a7d commit 3f244ea
Show file tree
Hide file tree
Showing 4 changed files with 20 additions and 0 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ and this project adheres to
### Added

- Add opam support for the Dependency submission API.
- Set `HOMEBREW_NO_INSTALLED_DEPENDENTS_CHECK` to `1` on the runner while installing OCaml.

### Removed

Expand Down
7 changes: 7 additions & 0 deletions dist/index.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

11 changes: 11 additions & 0 deletions packages/setup-ocaml/src/installer.ts
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,18 @@ export async function installer() {
await restoreCygwinCache();
}
const opamCacheHit = await restoreOpamCache();
const HOMEBREW_NO_INSTALLED_DEPENDENTS_CHECK =
process.env["HOMEBREW_NO_INSTALLED_DEPENDENTS_CHECK"];
if (PLATFORM === "macos") {
core.exportVariable("HOMEBREW_NO_INSTALLED_DEPENDENTS_CHECK", 1);
}
await setupOpam();
if (PLATFORM === "macos") {
core.exportVariable(
"HOMEBREW_NO_INSTALLED_DEPENDENTS_CHECK",
HOMEBREW_NO_INSTALLED_DEPENDENTS_CHECK,
);
}
await repositoryRemoveAll();
await repositoryAddAll(OPAM_REPOSITORIES);
const ocamlCompiler = await resolveCompiler(OCAML_COMPILER);
Expand Down
1 change: 1 addition & 0 deletions packages/setup-ocaml/turbo.json
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@
"passThroughEnv": [
"GITHUB_ACTIONS",
"GITHUB_WORKSPACE",
"HOMEBREW_NO_INSTALLED_DEPENDENTS_CHECK",
"PATH",
"USERPROFILE",
"XDG_CACHE_HOME"
Expand Down

0 comments on commit 3f244ea

Please sign in to comment.