Skip to content

Commit

Permalink
[setup-r] sets HOMEBREW_NO_INSTALLED_DEPENDENTS_CHECK=true
Browse files Browse the repository at this point in the history
On macOS to avoid lengthy homebrew updates that are most probably
not needed. See #810 for an example failure.
  • Loading branch information
gaborcsardi committed Mar 20, 2024
1 parent 194cd6f commit 340528c
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 2 deletions.
2 changes: 1 addition & 1 deletion setup-r/lib/installer.js
Original file line number Diff line number Diff line change
Expand Up @@ -265,7 +265,7 @@ function acquireUtilsMacOS() {
return __awaiter(this, void 0, void 0, function* () {
// qpdf is needed by `--as-cran`
try {
yield exec.exec("brew", ["install", "qpdf", "pkgconfig", "checkbashisms", "ghostscript"]);
yield exec.exec("brew", ["install", "qpdf", "pkgconfig", "checkbashisms", "ghostscript"], { env: { "HOMEBREW_NO_INSTALLED_DEPENDENTS_CHECK": "true" } });
}
catch (error) {
core.debug(`${error}`);
Expand Down
6 changes: 5 additions & 1 deletion setup-r/src/installer.ts
Original file line number Diff line number Diff line change
Expand Up @@ -240,7 +240,11 @@ async function acquireFortranMacOSOld(): Promise<string> {
async function acquireUtilsMacOS() {
// qpdf is needed by `--as-cran`
try {
await exec.exec("brew", ["install", "qpdf", "pkgconfig", "checkbashisms", "ghostscript"]);
await exec.exec(
"brew",
["install", "qpdf", "pkgconfig", "checkbashisms", "ghostscript"],
{ env: { "HOMEBREW_NO_INSTALLED_DEPENDENTS_CHECK": "true" } }
);
} catch (error) {
core.debug(`${error}`);

Expand Down

0 comments on commit 340528c

Please sign in to comment.