Skip to content

Conversation

@ajpotts
Copy link
Contributor

@ajpotts ajpotts commented Dec 12, 2024

This PR modifies the Makefile such that
make install-deps DEP_INSTALL_DIR=<directory>
can be called, and the install script will attempt to look for the necessary files in DEP_INSTALL_DIR before calling out to the network. This will be useful for offline builds.

make install-arrow is being handled under #3933.

Closes #3939: install-deps to work offline

@ajpotts ajpotts force-pushed the 3939_install_deps_to_work_offline branch from a55b3e5 to 6425ce3 Compare December 13, 2024 15:19
Copy link
Contributor

@jaketrookman jaketrookman left a comment

Choose a reason for hiding this comment

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

Looks good

Copy link
Contributor

@drculhane drculhane left a comment

Choose a reason for hiding this comment

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

I tested this on Tuxedo (an Ubuntu variant) and Alma Linux (which our customer suggested). I did an install once with the Internet, to ensure the files where there. Then I disconnected from the Internet, and ran make install-zmq, make install-hdf5, make install-iconv and make install-idn2, all with the appropriate DEP_BUILD_DIR= argument. They all worked.

@ajpotts ajpotts force-pushed the 3939_install_deps_to_work_offline branch from dd6ddb0 to 2d7c708 Compare December 13, 2024 17:21
@ajpotts ajpotts enabled auto-merge December 13, 2024 17:21
@ajpotts ajpotts added this pull request to the merge queue Dec 13, 2024
Merged via the queue into Bears-R-Us:master with commit 18525f6 Dec 13, 2024
16 checks passed
jabraham17 pushed a commit to jabraham17/arkouda that referenced this pull request Jan 21, 2025
Co-authored-by: Amanda Potts <ajpotts@users.noreply.github.com>
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.

install-deps to work offline

3 participants