Honor ZOPEN_INSTALL_MINIMAL in zopen-build install command generation

[Copilot]

What type of PR is this? (check all applicable)

• Refactor
• Feature
• Bug Fix
• Optimization
• Content Update

Category

• zopen build framework
• zopen package manager
• Documentation
• CI/CD
• Tools

Description

bin/zopen-build documented ZOPEN_INSTALL_MINIMAL behavior by convention, but the normal install path did not honor it when constructing ZOPEN_INSTALL_CMD. This updates install command generation so default behavior stays unchanged while minimal mode drops inline compiler/linker assignments and relies on the exported environment instead.

• Install command behavior

  • preserve the current non-CMake, non-skip default by appending ${ZOPEN_EXTRA_INSTALL_CC_OPTS} when ZOPEN_INSTALL_MINIMAL is unset
  • explicitly set ZOPEN_INSTALL_MINIMAL=no in that default/full path
  • when ZOPEN_INSTALL_MINIMAL is set, build ZOPEN_INSTALL_CMD without inline CC / CPPFLAGS / CFLAGS / CXX / CXXFLAGS / LDFLAGS assignments

• Scope control

  • leave CMake install handling unchanged
  • leave skip install handling unchanged

• Help text

  • add ZOPEN_INSTALL_MINIMAL alongside the other *_MINIMAL environment settings in zopen-build -h

• Regression coverage

  • add focused tests for:
    • default/full install command generation
    • minimal install command generation
    • help text presence for ZOPEN_INSTALL_MINIMAL

# default/full path
ZOPEN_INSTALL_CMD="\"${ZOPEN_INSTALL}\" ${ZOPEN_INSTALL_OPTS} ${verboseOpts} ${ZOPEN_EXTRA_INSTALL_CC_OPTS}"
ZOPEN_INSTALL_MINIMAL="no"

# minimal path
ZOPEN_INSTALL_CMD="\"${ZOPEN_INSTALL}\" ${ZOPEN_INSTALL_OPTS}"
ZOPEN_INSTALL_MINIMAL="yes"

Related Issues

• Related Issue #

[optional] Are there any post-deployment tasks or follow-up actions required?

None.

[Copilot]

Copilot wasn't able to review any files in this pull request.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[Copilot]

🟡 Changes recommended

Isolate the default test environment variable and update the published reference documentation.

Get a fresh assessment by requesting another Copilot review.

Review details

• Files reviewed: 1/2 changed files
• Comments generated: 1
• Review effort level: Lite

[Copilot]

🔵 Needs a closer look

Fix the test’s environment isolation and clarify the full set of variables omitted in minimal mode.

Review details

Suppressed comments (1)

tools/test_zopen_build.py:93

• The default-path test inherits ZOPEN_INSTALL_MINIMAL from the process environment, so a runner that happens to export this variable will exercise the minimal branch instead of the advertised default/unset branch and can fail spuriously (or stop detecting a regression in the full path). Unset it in the generated shell script before applying the optional test value.

                {install_minimal_line}

• Files reviewed: 1/2 changed files
• Comments generated: 0 new
• Review effort level: Lite

[Copilot]

Copilot wasn't able to review any files in this pull request.