Skip to content

FIX Use ctypes.util.dllist again on Windows and macOS - #288

Open
cakedev0 wants to merge 1 commit into
joblib:masterfrom
cakedev0:fix-dllist-condition
Open

cakedev0 wants to merge 1 commit into
joblib:masterfrom
cakedev0:fix-dllist-condition

Conversation

@cakedev0

Copy link
Copy Markdown
Contributor

#250 changed the dllist import condition to sys.platform == "linux" and sys.version_info[:2] >= (3, 15). Windows and macOS on Python 3.14+ therefore stopped using ctypes.util.dllist, which 3.7.0 used there (#219), and fell back to the older listing code. This looks unintended: #250 is only about Linux. This PR restores the 3.7.0 behavior on Windows and macOS and keeps the Linux restriction.

The new test fails on master and passes with this PR on macOS with Python 3.14. The PR also drops the stdlib_dllist() test helper, which still mirrored the 3.7.0 condition and wrongly skipped a Windows test.

This regression hasn't been released, so there is no changelog entry.

🤖 Generated with Claude Code

joblib#250 restricted the dllist import to Linux with Python 3.15+, so Windows
and macOS on Python 3.14+ fell back to the older listing code. Restore
the 3.7.0 behavior there and keep the Linux restriction.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

This branch has not been deployed

No deployments
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