Skip to content

gskit: remove#11460

Closed
bagder wants to merge 1 commit intomasterfrom
bagder/rm-gskit
Closed

gskit: remove#11460
bagder wants to merge 1 commit intomasterfrom
bagder/rm-gskit

Conversation

@bagder
Copy link
Member

@bagder bagder commented Jul 18, 2023

Motivation

We remove support for building curl with the gskit TLS library in August 2023.

  • This is a niche TLS library, only running on some IBM systems
  • no regular curl contributors use this backend
  • no CI builds use or verify this backend
  • gskit, or the curl adaption for it, lacks many modern TLS features making it an inferior solution
  • build breakages in this code take weeks or more to get detected
  • fixing gskit code is mostly done "flying blind"

Discussion

Debated here: https://curl.se/mail/lib-2023-07/0015.html

To bring it back?

This backend could be brought back to supported status if we see efforts into making sure this backend reaches similar level of support and attention as other TLS backends do. Probably including both support for modern TLS features and CI builds.

@bagder bagder added TLS feature-window A merge of this requires an open feature window labels Jul 18, 2023
@bagder bagder closed this in 78d6232 Aug 7, 2023
@bagder bagder deleted the bagder/rm-gskit branch August 7, 2023 19:00
ptitSeb pushed a commit to wasix-org/curl that referenced this pull request Sep 25, 2023
We remove support for building curl with gskit.

 - This is a niche TLS library, only running on some IBM systems
 - no regular curl contributors use this backend
 - no CI builds use or verify this backend
 - gskit, or the curl adaption for it, lacks many modern TLS features
   making it an inferior solution
 - build breakages in this code take weeks or more to get detected
 - fixing gskit code is mostly done "flying blind"

This removal has been advertized in DEPRECATED in Jan 2, 2023 and it has
been mentioned on the curl-library mailing list.

It could be brought back, this is not a ban. Given proper effort and
will, gskit support is welcome back into the curl TLS backend family.

Closes curl#11460
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cmdline tool feature-window A merge of this requires an open feature window libcurl API tests TLS

Development

Successfully merging this pull request may close these issues.

1 participant