Skip to content

Fall back to /sbin/ldconfig for get_library_path - #126

Merged
guillon merged 2 commits into
xtc-tools:mainfrom
guillon:dev/cguillon/fix-library-path
Sep 2, 2026
Merged

Fall back to /sbin/ldconfig for get_library_path#126
guillon merged 2 commits into
xtc-tools:mainfrom
guillon:dev/cguillon/fix-library-path

Conversation

@guillon

@guillon guillon commented Sep 2, 2026

Copy link
Copy Markdown
Member

Motivation

When PATH does not contain /sbin fallback to /sbin/ldconfig on get_library_path.

Also moved ext_tools test under tests/pytest/unit/ext_tools/.

Description

Upated behavior for path search:

  • absolute
  • LD_LIBRARY_PATH
  • LIBRARY_PATH
  • ldconfig in path
  • /sbin/ldconfig

Commits

Ref commits.

Discussion

n/a

@guillon
guillon requested a review from qaco September 2, 2026 12:10
@guillon guillon self-assigned this Sep 2, 2026
@guillon

guillon commented Sep 2, 2026

Copy link
Copy Markdown
Member Author

@superlopuh , @qaco FYI.

@guillon
guillon enabled auto-merge September 2, 2026 12:18
@codecov

codecov Bot commented Sep 2, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@guillon
guillon merged commit 0323fa9 into xtc-tools:main Sep 2, 2026
5 checks passed
@guillon
guillon deleted the dev/cguillon/fix-library-path branch September 2, 2026 12:32
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