Skip to content

Updated ml-metadata build script #8137

Open
vrushanaik wants to merge 3 commits intoppc64le:masterfrom
vrushanaik:mlmetadata
Open

Updated ml-metadata build script #8137
vrushanaik wants to merge 3 commits intoppc64le:masterfrom
vrushanaik:mlmetadata

Conversation

@vrushanaik
Copy link
Copy Markdown
Contributor

Issue:

Wheel file was getting store in the working directory but the wrapper wasn't able to detect it .

Fix:

Previously, the script exited immediately after successful test execution, preventing the wrapper from detecting wheel in the working directory.

Replaced:

exit 0

With:

return 0 2>/dev/null || exit 0

This allows sourced execution to return control to the wrapper, while still exiting normally when run directly. No changes to build or test logic.

Checklist

  • Have you checked and followed all the points mention in the CONTRIBUTING.MD
  • Have you validated script on UBI 9 container
  • Did you run the script(s) on fresh container with set -e option enabled and observe success ?
  • Did you have Legal approvals for patch files ?

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.

1 participant