Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

GH-99108: Make vectorized versions of Blake2 available on x86, too#125244

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
zware merged 6 commits intopython:mainfrommsprotz:protz_hacl_refresh
Oct 17, 2024

Conversation

msprotz
Copy link
Contributor

@msprotzmsprotz commentedOct 10, 2024
edited by bedevere-appbot
Loading

Per#99108 (comment): trying a quick fix to see if this builds.

@gpshead can you buildbot this for Debian x86? Thanks.

If it works, I'll patch & fix upstream, then pull it in accordingly into CPython. Thank you!

@msprotz
Copy link
ContributorAuthor

(The SBOM build will fail for the time being.)

@encukou
Copy link
Member

!buildbot x86

@bedevere-bot
Copy link

🤖 New build scheduled with the buildbot fleet by@encukou for commit091fd62 🤖

The command will test the builders whose names match following regular expression:x86

The builders matched are:

  • x86-64 MacOS Intel NoGIL PR
  • x86-64 macOS PR
  • x86 Debian Installed with X PR
  • x86-64 MacOS Intel ASAN NoGIL PR
  • x86 Debian Non-Debug with X PR

@msprotz
Copy link
ContributorAuthor

Looks like this compiles but gives an unrelated error in tests.@gpshead@encukou do you agree? If so, I'll fix it upstream

@encukou
Copy link
Member

Yup, I agree!
I'll need to fix that ctypes error now that it's uncovered :)

gpshead reacted with laugh emoji

@vstinner
Copy link
Member

buildbot/x86 Debian Installed with X PR — Build done.
buildbot/x86 Debian Non-Debug with X PR — Build done.

Tests failed on these two workers, but it's a known issue:#121938

This PR fix the build on these two workers:#125535

Copy link
Member

@vstinnervstinner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

LGTM

@vstinner
Copy link
Member

(The SBOM build will fail for the time being.)

I ranmake regen-sbom.

@vstinnervstinnerenabled auto-merge (squash)October 15, 2024 15:50
@vstinner
Copy link
Member

If it works, I'll patch & fix upstream, then pull it in accordingly into CPython. Thank you!

Yeah, please fix upstream ;-)

@msprotz
Copy link
ContributorAuthor

The fix just got merged a couple hours ago upstream, I need to run a build to refresh the correct folder, and then I'll push the refresh here (along with make regen-sbom). Thanks!

gpshead reacted with thumbs up emoji

@vstinner
Copy link
Member

@gpshead@encukou: Why did you block merging this PR? Do you prefer to wait until upstream is fixed, and then update Hacl in Python?

@encukou
Copy link
Member

Yes.

vstinner reacted with thumbs up emoji

@msprotz
Copy link
ContributorAuthor

Ok this is pushed, let's see what CI says.@vstinner FYI, this would be an SBOM violation to push a local fix to an external vendored library, so this is why we have to propagate the fix upstream first... thanks!

gpshead reacted with thumbs up emoji

@vstinner
Copy link
Member

FYI, this would be an SBOM violation to push a local fix to an external vendored library

I didn't know that. We made many changes in mimalloc vendored copy.

@msprotz
Copy link
ContributorAuthor

Maybe mimalloc gets a different treatment? The SBOM CI target fails if the vendored copy of HACL* is not exactly identical to upstream...

@gpshead
Copy link
Member

For this one we set it up to be upstream-first and just track specific upstream versions. it works since we've got active maintainers. :)

vstinner reacted with thumbs up emoji

@zware
Copy link
Member

!buildbot x86 Deb

@bedevere-bot
Copy link

🤖 New build scheduled with the buildbot fleet by@zware for commitacd04e6 🤖

The command will test the builders whose names match following regular expression:x86 Deb

The builders matched are:

  • x86 Debian Non-Debug with X PR
  • x86 Debian Installed with X PR

@zware
Copy link
Member

This fixes the build issue on x86 Debian. Not sure what's up with theCI job checking the SBOM, though. I can't reproduce that failure locally (cc@sethmlarson for sbom insight).

befeleme pushed a commit to fedora-python/cpython that referenced this pull requestOct 17, 2024
@vstinner
Copy link
Member

@msprotz: Can you re-runmake regen-sbom?

@msprotz
Copy link
ContributorAuthor

I thought I had run it. Let me run it again.

Co-authored-by: Zachary Ware <zachary.ware@gmail.com>
@msprotz
Copy link
ContributorAuthor

I committed the suggested change, butmake regen-sbom generates no diff on my machine still. Let's see if this fixes CI

@zware
Copy link
Member

I could have left a clearer note, sorry. It turned out that the generated files check was failing onregen-sbom becauseTools/build/generate_sbom.py behaves differently ifCI is defined in the environment. In that case, it actually does the download and regenerates the checksum.

@msprotz
Copy link
ContributorAuthor

Ah ok! Good to know for next time, thank you.

zware reacted with thumbs up emoji

@zwarezwareenabled auto-merge (squash)October 17, 2024 14:52
@zwarezware merged commit528bbab intopython:mainOct 17, 2024
37 checks passed
ebonnal pushed a commit to ebonnal/cpython that referenced this pull requestJan 12, 2025
…too (python#125244)Accomplished by updating HACL* vendored code fromhacl-star/hacl-star@a6a0949 tohacl-star/hacl-star@315a9e4Co-authored-by: Victor Stinner <vstinner@python.org>Co-authored-by: Zachary Ware <zach@python.org>
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@zwarezwarezware left review comments

@vstinnervstinnervstinner approved these changes

@gpsheadgpsheadAwaiting requested review from gpsheadgpshead is a code owner

@sethmlarsonsethmlarsonAwaiting requested review from sethmlarsonsethmlarson is a code owner

Assignees
No one assigned
Labels
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

6 participants
@msprotz@encukou@bedevere-bot@vstinner@gpshead@zware

[8]ページ先頭

©2009-2025 Movatter.jp