Skip to content

Allow RPATH adjustment to be removed and various CMake fixes#1029

Merged
jhale merged 19 commits into
mainfrom
jhale-patch-1
Jun 24, 2026
Merged

Allow RPATH adjustment to be removed and various CMake fixes#1029
jhale merged 19 commits into
mainfrom
jhale-patch-1

Conversation

@jhale

@jhale jhale commented Jun 22, 2026

Copy link
Copy Markdown
Member

RPATH adjustments can be disabled in Python interface using -Ccmake.args=-DBASIX_SET_INSTALL_RPATH=False. This is possibly useful for packages and users who prefer dynamic library discovery. Many other small fixes.

Comment thread .github/workflows/dolfinx-tests.yml Fixed
- name: Install Basix
run: pip install --no-build-isolation .
run: |
pip install scikit-build-core
- name: Install Basix
run: pip install --no-build-isolation .
run: |
pip install scikit-build-core
run: |
pip install scikit-build-core
python -m scikit_build_core.build requires | python -c "import sys, json; print(' '.join(json.load(sys.stdin)))" | xargs pip install
pip install --no-build-isolation .
run: |
pip install scikit-build-core
python -m scikit_build_core.build requires | python -c "import sys, json; print(' '.join(json.load(sys.stdin)))" | xargs pip install
pip install --no-build-isolation .
Comment thread .github/workflows/dolfinx-tests.yml Fixed
@jhale jhale changed the title Remove RPATH fiddling Allow RPATH adjustment to be removed and various CMake fixes Jun 24, 2026
@jhale jhale enabled auto-merge June 24, 2026 09:08
@jhale jhale added this pull request to the merge queue Jun 24, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jun 24, 2026
@jhale jhale added this pull request to the merge queue Jun 24, 2026
Merged via the queue into main with commit 43386e3 Jun 24, 2026
30 checks passed
@jhale jhale deleted the jhale-patch-1 branch June 24, 2026 09:38
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.

2 participants