The #curl feature window opened today and will remain open for three weeks. This rather huge change just landed:

github.com/curl/curl/pull/1512…

#curl
in reply to daniel:// stenberg://

As an outsider, just following briefly due to curiosity ... I would suggest commit messages including a rationale for such huge changes. It helps people reading the commit log, especially those who might not follow out-of-tree discussions closely.

From the diff I understand what the feature was and that it was experimental with a heavy maintenance load. But going from there to why easily ends up into speculation.

in reply to daniel:// stenberg://

yeah, I didn't mean the commit message should be elaborate. Just give enough insight that you know where to look further for more info.

In this case, I think something like this could have been enough: "This code has been experimental for long while and is not a feature we see value in for the foreseeable future. Removing it to reduce maintenance efforts."

in reply to 🔗 David Sommerseth

@dazo the explanation has existed in the DEPRECATE docs file in the repository since August, and it has been repeated in every release video I've done since. It should not be news to anyone following the project. Also, the PR has existed for a few moths as well and not a single one asked for that clarification. So it was not done in the commit.