Re: [Ovmsdev] [cesanta/mongoose] LWIP SO_REUSEADDR & ESP32 logging (#894) (fwd)
What happened? Did Mark's commit 58942ce96c5e3e14edb53c804ffcdd4127e226e7 finally wake up cesanta? Are we going to resubmit all of our changes as separate pull requests? One person does that, or each author? And to the extent that they may be interdependent, so we need for them to be accepted in order? -- Steve ---------- Forwarded message ---------- Date: Wed, 12 Feb 2020 01:35:43 -0800 From: Sergey Lyubka <notifications@github.com> Reply-To: cesanta/mongoose <reply+ABG5G6FIED7L3BL25LSLI4F4KD4W7EVBNHHBDAFYO4@reply.github.com> To: cesanta/mongoose <mongoose@noreply.github.com> Cc: Stephen Casner <casner@acm.org>, Mention <mention@noreply.github.com> Subject: Re: [cesanta/mongoose] LWIP SO_REUSEADDR & ESP32 logging (#894) This PR contains several unrelated pieces - please split, one PR per single functionality piece. The PR should be against the `dev` branch, and one PR must have only one commit. -- You are receiving this because you were mentioned. Reply to this email directly or view it on GitHub: https://github.com/cesanta/mongoose/pull/894#issuecomment-585116086
It is ridiculous. I replied as follows:
This PR contains several unrelated pieces - please split, one PR per single functionality piece. The PR should be against the dev branch, and one PR must have only one commit.
Is there any point? There are 33 pull requests here, dating back to 5 years to 2015. Only three have any response, and this one has sat dead since October 2018. It seems Cesanta is simply not interested in community improvements? We could go to the work to re-factor all these improvements and bug fixes into individual PRs, but there seems little point unless they get some attention. The way we are making our changes, to mongoose.h and mongoose.c directly, isn’t really compatible with how they do things anyway. We should probably be using their tools/(un)amalgam.py scripts. Let’s see what they reply (to my somewhat rude comment), and go from there. There are so few changes been made to cesanta/mongoose that it may just be easier for us to pull from them. Regards, Mark.
On 13 Feb 2020, at 4:49 AM, Stephen Casner <casner@acm.org> wrote:
What happened? Did Mark's commit 58942ce96c5e3e14edb53c804ffcdd4127e226e7 finally wake up cesanta? Are we going to resubmit all of our changes as separate pull requests? One person does that, or each author? And to the extent that they may be interdependent, so we need for them to be accepted in order?
-- Steve
---------- Forwarded message ---------- Date: Wed, 12 Feb 2020 01:35:43 -0800 From: Sergey Lyubka <notifications@github.com> Reply-To: cesanta/mongoose <reply+ABG5G6FIED7L3BL25LSLI4F4KD4W7EVBNHHBDAFYO4@reply.github.com> To: cesanta/mongoose <mongoose@noreply.github.com> Cc: Stephen Casner <casner@acm.org>, Mention <mention@noreply.github.com> Subject: Re: [cesanta/mongoose] LWIP SO_REUSEADDR & ESP32 logging (#894)
This PR contains several unrelated pieces - please split, one PR per single functionality piece. The PR should be against the `dev` branch, and one PR must have only one commit.
-- You are receiving this because you were mentioned. Reply to this email directly or view it on GitHub: https://github.com/cesanta/mongoose/pull/894#issuecomment-585116086 _______________________________________________ OvmsDev mailing list OvmsDev@lists.openvehicles.com http://lists.openvehicles.com/mailman/listinfo/ovmsdev
Ack. The response was an apology, but I'll rather spend my time on our own issues first. I asked them about separate PRs with the initial commit. The PR mostly contains Steve's and my changes, I think they are all well done & documented and could also be merged by cherry picking if necessary. Regards, Michael Am 13.02.20 um 00:53 schrieb Mark Webb-Johnson:
It is ridiculous. I replied as follows:
This PR contains several unrelated pieces - please split, one PR per single functionality piece. The PR should be against the dev branch, and one PR must have only one commit.
Is there any point? There are 33 pull requests here, dating back to 5 years to 2015. Only three have any response, and this one has sat dead since October 2018. It seems Cesanta is simply not interested in community improvements? We could go to the work to re-factor all these improvements and bug fixes into individual PRs, but there seems little point unless they get some attention.
The way we are making our changes, to mongoose.h and mongoose.c directly, isn’t really compatible with how they do things anyway. We should probably be using their tools/(un)amalgam.py scripts.
Let’s see what they reply (to my somewhat rude comment), and go from there. There are so few changes been made to cesanta/mongoose that it may just be easier for us to pull from them.
Regards, Mark.
On 13 Feb 2020, at 4:49 AM, Stephen Casner <casner@acm.org <mailto:casner@acm.org>> wrote:
What happened? Did Mark's commit 58942ce96c5e3e14edb53c804ffcdd4127e226e7 finally wake up cesanta? Are we going to resubmit all of our changes as separate pull requests? One person does that, or each author? And to the extent that they may be interdependent, so we need for them to be accepted in order?
-- Steve
---------- Forwarded message ---------- Date: Wed, 12 Feb 2020 01:35:43 -0800 From: Sergey Lyubka <notifications@github.com <mailto:notifications@github.com>> Reply-To: cesanta/mongoose <reply+ABG5G6FIED7L3BL25LSLI4F4KD4W7EVBNHHBDAFYO4@reply.github.com <mailto:reply+ABG5G6FIED7L3BL25LSLI4F4KD4W7EVBNHHBDAFYO4@reply.github.com>> To: cesanta/mongoose <mongoose@noreply.github.com <mailto:mongoose@noreply.github.com>> Cc: Stephen Casner <casner@acm.org <mailto:casner@acm.org>>, Mention <mention@noreply.github.com <mailto:mention@noreply.github.com>> Subject: Re: [cesanta/mongoose] LWIP SO_REUSEADDR & ESP32 logging (#894)
This PR contains several unrelated pieces - please split, one PR per single functionality piece. The PR should be against the `dev` branch, and one PR must have only one commit.
-- You are receiving this because you were mentioned. Reply to this email directly or view it on GitHub: https://github.com/cesanta/mongoose/pull/894#issuecomment-585116086 _______________________________________________ OvmsDev mailing list OvmsDev@lists.openvehicles.com http://lists.openvehicles.com/mailman/listinfo/ovmsdev
_______________________________________________ OvmsDev mailing list OvmsDev@lists.openvehicles.com http://lists.openvehicles.com/mailman/listinfo/ovmsdev
-- Michael Balzer * Helkenberger Weg 9 * D-58256 Ennepetal Fon 02333 / 833 5735 * Handy 0176 / 206 989 26
participants (3)
-
Mark Webb-Johnson -
Michael Balzer -
Stephen Casner