Skip to content

update prefix on install#157

Merged
MikeFultonDev merged 2 commits intomainfrom
prefixswizzle
Feb 4, 2023
Merged

update prefix on install#157
MikeFultonDev merged 2 commits intomainfrom
prefixswizzle

Conversation

@MikeFultonDev
Copy link
Copy Markdown
Collaborator

Add an additional update to the hardcoded values in the installed code, as follows:

  • keep the update so that ZOPEN_INSTALL_ROOT replaced that hard-coded product installation directory
  • add a new update so that ZOPEN_INSTALL_PREFIX replaces hard-code prefix locations (e.g. to another pre-req tool)

On install, have setup now:

  • update ZOPEN_INSTALL_ROOT to the location the code is now installed into
  • update ZOPEN_INSTALL_PREFIX to the parent location of where the code is installed into

This was tested with pipelineport which previously had a '/jenkins/prod' prefix left around causing problems.
This should clean up other tools as well and mean we can eliminate some environment variables I hope (for things like autoconf)

@MikeFultonDev MikeFultonDev added the enhancement New feature or request label Feb 4, 2023
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.

Nice! LGTM

@MikeFultonDev MikeFultonDev merged commit 8808ff1 into main Feb 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants