fix docker build#2015
Conversation
There was a problem hiding this comment.
Code Review
This pull request updates the IF50_aggregator_common.sh installer script to set the HOMEBREW_NO_SANDBOX_LINUX=1 environment variable when installing cyclonedx-cli via Homebrew. The review feedback highlights a lack of error handling for the installation command and points out that the existing error handling in surrounding lines uses subshells, which prevents the main script from exiting on failure. A code suggestion is provided to address these issues using curly braces.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
There was a problem hiding this comment.
Pull request overview
This PR aims to fix a Docker build issue by adjusting how cyclonedx-cli is installed via Linuxbrew during the installer flow.
Changes:
- Disable Homebrew’s Linux sandbox during
cyclonedx-cliinstallation by settingHOMEBREW_NO_SANDBOX_LINUX=1.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Bug fix
What is the current behavior? (You can also link to an open issue here)
What is the new behavior (if this is a feature change)? If possible add a screenshot.
Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
Other information: