Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

hpcc: Improve multi-arch installation #279

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

tad4
Copy link

@tad4 tad4 commented Apr 29, 2023

Dear @michaellarabel:

While running hpcc-1.2.8 on an Apple M1 Pro, the installation script couldn't find an installed Linear Algebra (LA) library—that's supported by HPCC—to pass to the linker for AArch64 platforms—even though an HPCC supported LA library was available.

This pull request improves multi-arch hpcc test installation by:

  1. Using PTS' OS_ARCH variable during hpcc's install-phase on supported Linux distros—i.e., Debian, Ubuntu, and Red Hat;
  2. Exits the install-phase early if a required toolchain to build hpcc is not found;
  3. Writes a unique error code to install-exit-status; and
  4. Echos a short message to the console to assist when running PTS' debug-install.

Please review and consider adding these improvements to the install script for hpcc.
Sincerely,
Tad

tad4 and others added 8 commits April 26, 2023 18:47
LA toolchain variable configuration has been updated to be CPU architecture-independent.
All hard-coded architecture-dependent paths have been revised to use PTS' `OS_ARCH` variable when searching for—as well as configuring—toolchain variables.
- If either an MPI or Linear Algebra (LA) toolchain can **not** be found, then `hpcc` can **not** be built.
- Error codes have been defined to assist users in understanding why the install script failed.
- Avoid any potential collisions with other _error identifiers_.
- Script-specific _error identifier_ is written to `install-exit-status`.
….0.1` downloads to reflect new GitHub repo name".

This reverts commit dba209a.
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