Skip to content

gh-154443: Fix test_makedev on DragonFly BSD#154444

Merged
serhiy-storchaka merged 1 commit into
python:mainfrom
serhiy-storchaka:nodev-dragonfly
Jul 22, 2026
Merged

gh-154443: Fix test_makedev on DragonFly BSD#154444
serhiy-storchaka merged 1 commit into
python:mainfrom
serhiy-storchaka:nodev-dragonfly

Conversation

@serhiy-storchaka

@serhiy-storchaka serhiy-storchaka commented Jul 22, 2026

Copy link
Copy Markdown
Member

major() and minor() on DragonFly BSD have no special case for NODEV, they just extract bit fields, so major(NODEV) is 255, not NODEV. Remove DragonFly BSD from the list of platforms preserving NODEV.

major() and minor() do not preserve NODEV on DragonFly BSD.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@bedevere-app bedevere-app Bot added the tests Tests in the Lib/test dir label Jul 22, 2026
@serhiy-storchaka serhiy-storchaka added needs backport to 3.15 pre-release feature fixes, bugs and security fixes skip news labels Jul 22, 2026
@serhiy-storchaka
serhiy-storchaka enabled auto-merge (squash) July 22, 2026 08:20
@serhiy-storchaka
serhiy-storchaka merged commit 2cf8e59 into python:main Jul 22, 2026
54 of 55 checks passed
@miss-islington-app

Copy link
Copy Markdown

Thanks @serhiy-storchaka for the PR 🌮🎉.. I'm working now to backport this PR to: 3.15.
🐍🍒⛏🤖

@bedevere-app

bedevere-app Bot commented Jul 22, 2026

Copy link
Copy Markdown

GH-154450 is a backport of this pull request to the 3.15 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.15 pre-release feature fixes, bugs and security fixes label Jul 22, 2026
@serhiy-storchaka
serhiy-storchaka deleted the nodev-dragonfly branch July 22, 2026 08:37
serhiy-storchaka added a commit that referenced this pull request Jul 22, 2026
…54450)

major() and minor() do not preserve NODEV on DragonFly BSD.
(cherry picked from commit 2cf8e59)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip news tests Tests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant