Skip to content

zopen-build: add ZOPEN_SETUP_NO_REPLACE to skip path replacement#1195

Merged
sachintu47 merged 1 commit into
mainfrom
setup_nohardcodepath
May 26, 2026
Merged

zopen-build: add ZOPEN_SETUP_NO_REPLACE to skip path replacement#1195
sachintu47 merged 1 commit into
mainfrom
setup_nohardcodepath

Conversation

@sachintu47
Copy link
Copy Markdown
Member

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

zopen-build: add ZOPEN_SETUP_NO_REPLACE to skip path replacement

Related Issues

  • Related Issue #
  • Closes #

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

@augmentcode
Copy link
Copy Markdown

augmentcode Bot commented May 26, 2026

🤖 Augment PR Summary

Summary: Adds a ZOPEN_SETUP_NO_REPLACE environment flag to allow skipping hardcoded-path placeholder replacement during installation.

Changes:

  • Documents the new flag in bin/zopen-build help text
  • Short-circuits replaceHardcodedPaths when the flag is set
  • Ensures the generated setup.sh only includes runtime replacement logic when replacements were performed

🤖 Was this summary useful? React with 👍 or 👎

Copy link
Copy Markdown

@augmentcode augmentcode Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review completed. No suggestions at this time.

Comment augment review to trigger a new review at any time.

Copy link
Copy Markdown
Member

@IgorTodorovskiIBM IgorTodorovskiIBM left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sachintu47 sachintu47 merged commit de6e0b2 into main May 26, 2026
0 of 2 checks passed
@sachintu47 sachintu47 deleted the setup_nohardcodepath branch May 26, 2026 17:11
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