Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Qt6Base] Upgrade to OpenSSL 3 #7619

Merged
merged 5 commits into from
Nov 5, 2023
Merged

Conversation

simonbyrne
Copy link
Contributor

I'm not exactly sure if this is the correct fix?

Fixes #7531.

cc: @barche

I'm not exactly sure if this is the correct fix?

Fixes JuliaPackaging#7531.
Copy link
Member

@giordano giordano left a comment

Choose a reason for hiding this comment

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

No

  1. you MUST change the version number when you change compat bounds
  2. compat bounds MUST be there

@simonbyrne
Copy link
Contributor Author

Okay, well we're a couple of versions behind (6.5.3 and 6.6.0 are both available: https://download.qt.io/archive/qt/), so should we just update those?

@giordano
Copy link
Member

giordano commented Nov 2, 2023

Yup, that'd do it.

@simonbyrne
Copy link
Contributor Author

Okay, no idea if this is the right way to do this. @barche does this look right?

simonbyrne and others added 2 commits November 2, 2023 11:27
Co-authored-by: Mosè Giordano <giordano@users.noreply.github.com>
@simonbyrne
Copy link
Contributor Author

okay, i'm out of ideas...

@barche
Copy link
Contributor

barche commented Nov 2, 2023

I’m away from my computers until Sunday, but 6.5.3 is the right version to choose, going to 6.6 would require rebuilding GR.

@barche
Copy link
Contributor

barche commented Nov 2, 2023

Looks like the new version requires some extra patching to deal with the AVX stuff.

@simonbyrne
Copy link
Contributor Author

thanks @barche!

@giordano giordano merged commit b524ac3 into JuliaPackaging:master Nov 5, 2023
4 checks passed
@yuyichao
Copy link

yuyichao commented Nov 5, 2023

I assume it's unintentional that this removes everything but x64 linux musl? JuliaBinaryWrappers/Qt6Base_jll.jl@382b0ad

@yuyichao
Copy link

yuyichao commented Nov 5, 2023

OK I saw #7628 (comment)

amontoison pushed a commit to amontoison/Yggdrasil that referenced this pull request Nov 27, 2023
* [Qt6Base] Upgrade to OpenSSL 3

I'm not exactly sure if this is the correct fix?

Fixes JuliaPackaging#7531.

* Update build_tarballs.jl

* Update Q/Qt6Base/build_tarballs.jl

Co-authored-by: Mosè Giordano <giordano@users.noreply.github.com>

* Update build_tarballs.jl

* Fix haswell error

---------

Co-authored-by: Mosè Giordano <giordano@users.noreply.github.com>
Co-authored-by: Bart Janssens <bart@bartjanssens.org>
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.

Bump Qt6Base to use OpenSSL 3
4 participants