Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

ENH: Make brew work in quite mode to silence warnings#280

Merged
thewtex merged 1 commit into
InsightSoftwareConsortium:masterInsightSoftwareConsortium/ITKPythonPackage:masterfrom
jhlegarreta:BrewQuietjhlegarreta/ITKPythonPackage:BrewQuietCopy head branch name to clipboard
Jul 28, 2024
Merged

ENH: Make brew work in quite mode to silence warnings#280
thewtex merged 1 commit into
InsightSoftwareConsortium:masterInsightSoftwareConsortium/ITKPythonPackage:masterfrom
jhlegarreta:BrewQuietjhlegarreta/ITKPythonPackage:BrewQuietCopy head branch name to clipboard

Conversation

@jhlegarreta

Copy link
Copy Markdown
Member

Make brew work in quite mode to silence warnings.

Fixes:

Warning: Treating doxygen as a formula. For the cask, use homebrew/cask/doxygen or specify the `--cask` flag.

and

Warning: zstd 1.5.6 is already installed and up-to-date.
To reinstall 1.5.6, run:
  brew reinstall zstd

and other similar warnings raised for example in:
https://github.com/InsightSoftwareConsortium/ITKAnalyzeObjectMap/actions/runs/10012233814

Make `brew` work in quite mode to silence warnings.

Fixes:
```
Warning: Treating doxygen as a formula. For the cask, use homebrew/cask/doxygen or specify the `--cask` flag.
```

and
```
Warning: zstd 1.5.6 is already installed and up-to-date.
To reinstall 1.5.6, run:
  brew reinstall zstd
```

and other similar warnings raised for example in:
https://github.com/InsightSoftwareConsortium/ITKAnalyzeObjectMap/actions/runs/10012233814
@jhlegarreta

Copy link
Copy Markdown
Member Author

@thewtex Have not tested.

Should we filter only the target warnings, something like e.g.

install_list=(zstd aria2 gnu-tar doxygen ninja cmake)

for item in "${install_list[@]}"; do
  if brew info "${item}" | grep --quiet 'Not installed'; then
    brew install "${item}"
  else
    brew upgrade "${item}"
  fi
done

However, the cask warning would still be shown unless we do brew install --quiet "${item}".

@thewtex

thewtex commented Jul 28, 2024

Copy link
Copy Markdown
Member

@jhlegarreta thanks!

Should we filter only the target warnings, something like e.g.

Would be a reasonable, refinement.

@thewtex thewtex merged commit 103f655 into InsightSoftwareConsortium:master Jul 28, 2024
@jhlegarreta jhlegarreta deleted the BrewQuiet branch July 28, 2024 21:58
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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.