Skip to content

bpo-45808: Fix check for crypt_r on platforms without crypt.h - #29563

Closed
tiran wants to merge 1 commit into
python:mainfrom
tiran:bpo-45808-crypt_r
Closed

bpo-45808: Fix check for crypt_r on platforms without crypt.h#29563
tiran wants to merge 1 commit into
python:mainfrom
tiran:bpo-45808-crypt_r

Conversation

@tiran

@tirantiran commented Nov 15, 2021•
edited by bedevere-bot
Loading

Copy link
Copy Markdown
Member

@tiran
bpo-45808: Fix check for crypt_r on platforms without crypt.h
86a3a59
Signed-off-by: Christian Heimes <christian@python.org>
@tiran

tiran commented Nov 15, 2021•
edited
Loading

Copy link
Copy Markdown
MemberAuthor

The PR also introduced LIBCRYPT. I need the variable in another PR later.

The check kinda worked on FreeBSD because it picks up the unrelated file /usr/include/rpcsvc/crypt.h.

@tirantiran added the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Nov 15, 2021
@bedevere-bot

bedevere-bot commented Nov 15, 2021

Copy link
Copy Markdown

🤖 New build scheduled with the buildbot fleet by @tiran for commit 86a3a59 🤖

If you want to schedule another build, you need to add the ":hammer: test-with-buildbots" label again.

@bedevere-botbedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Nov 15, 2021
@tiran

tiran commented Nov 23, 2021

Copy link
Copy Markdown
MemberAuthor

Superseded by #29725

@tirantiran closed this Nov 23, 2021
@tirantiran mentioned this pull request Sep 11, 2022
Closed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

awaiting core review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@tiran@bedevere-bot@the-knights-who-say-ni