Buy commercial curl support from WolfSSL. We help you work
out your issues, debug your libcurl applications, use the API, port to new
platforms, add new features and more. With a team lead by the curl founder
himself.
Re: [PATCH v4] setopt: make curl_easy_vsetopt public
- Contemporary messages sorted: [ by date ] [ by thread ] [ by subject ] [ by author ] [ by messages with attachments ]
From: Drew DeVault via curl-library <curl-library_at_lists.haxx.se>
Date: Tue, 29 Mar 2022 09:39:31 +0200
On Tue Mar 29, 2022 at 9:32 AM CEST, Daniel Stenberg wrote:
> I'm sorry, but whose job would you say it is to forward outputs from GitHub CI
> jobs from your patch to the mailing list because you won't go there and check
> yourself?
I would say it's the project's responsibility to make the contribution
process as smooth as possible. lists.sr.ht facilitates forwarding CI
results to mailing lists, for instance. You've made contributions via
email a second-class citizen, thus my feedback on the process. I have no
desire to use a nonfree platform like GitHub to contribute to curl.
> Here's V4 as a PR - and it causes CI failures too right now:
>
> https://github.com/curl/curl/pull/8645
The first failure looks unrelated to my change:
https://github.com/curl/curl/runs/5732971450?check_suite_focus=true
The others look more useful. Will send a v5 which corrects the issue.
Date: Tue, 29 Mar 2022 09:39:31 +0200
On Tue Mar 29, 2022 at 9:32 AM CEST, Daniel Stenberg wrote:
> I'm sorry, but whose job would you say it is to forward outputs from GitHub CI
> jobs from your patch to the mailing list because you won't go there and check
> yourself?
I would say it's the project's responsibility to make the contribution
process as smooth as possible. lists.sr.ht facilitates forwarding CI
results to mailing lists, for instance. You've made contributions via
email a second-class citizen, thus my feedback on the process. I have no
desire to use a nonfree platform like GitHub to contribute to curl.
> Here's V4 as a PR - and it causes CI failures too right now:
>
> https://github.com/curl/curl/pull/8645
The first failure looks unrelated to my change:
https://github.com/curl/curl/runs/5732971450?check_suite_focus=true
The others look more useful. Will send a v5 which corrects the issue.
-- Unsubscribe: https://lists.haxx.se/listinfo/curl-library Etiquette: https://curl.haxx.se/mail/etiquette.htmlReceived on 2022-03-29