Skip to content

Conversation

@YvesDup
Copy link
Contributor

@YvesDup YvesDup commented Apr 21, 2025

Description

Because the line 68 from

try:
from multiprocessing.sharedctypes import Value, copy
HAS_SHAREDCTYPES = True
except ImportError:
HAS_SHAREDCTYPES = False
failed in AMD64 FreeBSD14/15 buildbot.

the res = Value('b', 0) line of the test_rlock_locked_2processes raise an exception.

Change to res = self.Value('b', 0) as in test_lock_locked_2processes test.

Linked PRs

See PR #132586

@YvesDup YvesDup requested a review from gpshead as a code owner April 21, 2025 15:28
@bedevere-app bedevere-app bot added awaiting review tests Tests in the Lib/test dir labels Apr 21, 2025
@YvesDup YvesDup changed the title gh-132719: Fix AMD64 FreeBSD14/15 3.x failures - test_rlock_2processes used aa unknown Value gh-132719: Fix AMD64 FreeBSD14/15 3.x failures - test_rlock_2processes used an unknown Value Apr 21, 2025
@YvesDup YvesDup changed the title gh-132719: Fix AMD64 FreeBSD14/15 3.x failures - test_rlock_2processes used an unknown Value gh-132719: Fix AMD64 FreeBSD14/15 3.x failures - test_rlock_locked_2processes used an unknown Value Apr 21, 2025
@gpshead
Copy link
Member

gpshead commented Apr 21, 2025

!buildbot .BSD.

@bedevere-bot
Copy link

🤖 New build scheduled with the buildbot fleet by @gpshead for commit 358fe9d 🤖

Results will be shown at:

https://buildbot.python.org/all/#/grid?branch=refs%2Fpull%2F132774%2Fmerge

The command will test the builders whose names match following regular expression: .*BSD.*

The builders matched are:

  • AMD64 FreeBSD Refleaks PR
  • AMD64 FreeBSD14 PR
  • AMD64 FreeBSD PR
  • AMD64 FreeBSD15 PR

@gpshead gpshead merged commit c8e0b6e into python:main Apr 21, 2025
40 of 42 checks passed
@YvesDup YvesDup deleted the sem-macosx-multiprocessing-locked-test branch May 22, 2025 17:03
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.

3 participants