Nsoblath/issue 212 #217
Nsoblath/issue 212 #217
Conversation
…ically linking Python in Mac and Linux.
|
Update: Added explicit linking to thread library, libdl, and libutil (the latter two for non-Windows). Test builds work. Unfortunately the mac and linux builds both are still dynamically linked to Python 😒 The verbose printout from the linking command doesn't seem to show us linking against libpython3.9.so:
|
|
Update [actually from 3/21/22]: On Linux the static linking to Python is actually working. On Mac, it appears to be statically linking libpython3.9.a, but the Framework still shows up when I call |
nsoblath
force-pushed
the
nsoblath/issue_212
branch
from
e4d9f58 to
4deb770
Compare
|
Update: Mac build solved by linking to the Currently the test jobs are not even starting because of some problem with the Here's the libraries dynamically linked to the apbs binary in the Mac build: And here are the libraries dynamically linked to the apbs binary in the linux build: |
* Fixed syntax errors in checking for max python version (#208) * Fixed syntax errors in checking for max python version * Switch OS specification to absolute versions and reduce places where those are specified * Update Windows config to Visual Studio 17 2022 * skip ci -- Updated releases.rst with update info * [skip ci] Added release note Co-authored-by: Noah Oblath <noah.oblath@pnnl.gov> * Change OS specification in build workflow (#210) * Switch OS specification to absolute versions and reduce places where those are specified * Update Windows config to Visual Studio 17 2022 * skip ci -- Updated releases.rst with update info Co-authored-by: Noah Oblath <noah.oblath@pnnl.gov> Co-authored-by: Darren Curtis <Darren.Curtis@pnnl.gov> * Nsoblath/issue 212 (#217) * Turn off debugging steps * Only install python in Windows for "Test Usage" job, as we'll be statically linking Python in Mac and Linux. * Search for static python libs in linux and mac when doing a static build * Turn on tmate to check library linking * Find threads explicitly * Tmate only on failure * Verbose build for debugging * Added two more missing libraries: libdl and libutil * Enable tmate for the mac use-test job * Test with Python 3.10 * Test with Python 3.10 revealed we needed libintl. So that's added now. Should be mac-specific * Can't include_directories() on an empty variable * Typo fix * Find Iconv * Add linking of APBS to CoreFoundation framework under macOS * Remove debugging * Removing some of the debugging tools used in testing the build. * Switch back to the standard build with Python 3.9 * Removed last tmate debugging * [skip ci] Document changes Co-authored-by: Noah Oblath <noah.oblath@pnnl.gov> * Nsoblath/issue 222 (#223) * First committed version of Dockerfile.centos * Add alt config workflow * Alt config build: test docker builds * In Dockerfile, specify APBS_STATIC_BUILD instead of BUILD_SHARED_LIBS * Missing backslash found * Fixing bugs in both Dockerfiles * Dockerfile bug fixing * Complete missing pieces of ensuring a position-independent-code build * Minor fixes * Switch to a shared-library build in the Ubuntu Docker build * Rename Dockerfile --> Dockerfile.ubuntu * Override APBS_STATIC_BUILD when needed Co-authored-by: Noah Oblath <noah.oblath@pnnl.gov> * Remove all mention of ENABLE_FETK (#225) Co-authored-by: Noah Oblath <noah.oblath@pnnl.gov> * Nsoblath/issue 226: FETK update (#227) * Address missing linkages * Cleanup FETK import * Multithreaded build in docker now controlled by a build argument * Install superlu as a dependency in the Ubuntu docker build * Update the FETK version in use by the automatic builds (still on a development branch) * Add in debugging tmate session for mac * Manually build arpack to get the static lib (which seems to have disappeared?) * Remove -j flags * Set fortran compiler for arpack build * Engaging brain, fixing mac-build commands * Comment out tmate session * Update the FETK commit to be the main branch, pre-release * Update the changelog * Updated default FETK build to commit v1.9.3 (57195e) Co-authored-by: Noah Oblath <noah.oblath@pnnl.gov> * Nsoblath/new issue 207: Addressing python version test (#229) * Fix python test max-version test in apbs/CMakeLists.txt * Turn on the alternate non-docker builds, with updates from the main build * Fixing the python max version test in apbs/tests/CMakeLists.txt * Fixing the python max version test in apbs/tools/python/CMakeLists.txt * Test commit: remove python-max-version definition * Simplify the Python search by looking for an exact version * Add a default PYTHON_VERSION * Adapt the various files involved in building APBS to the new Python version setup * Didn't setup the python version correctly in the GHA builds Co-authored-by: Noah Oblath <noah.oblath@pnnl.gov> * Updated installation documentation (#231) Co-authored-by: Noah Oblath <noah.oblath@pnnl.gov> * Updated license, version, and release info for pending release Co-authored-by: Noah Oblath <noah.oblath@pnnl.gov> Co-authored-by: Darren Curtis <Darren.Curtis@pnnl.gov>
|
Unfortunately, this still seems to be an issue in the 3.4.1 release. Incidentally, I also needed to pass |
|
@alexreg should we reopen this issue? |
|
@sobolevnrm If you would, please. |
Statically link Python in Mac and Linux builds.
Closes #212