[Buildroot] [PATCH 1/1] systemd: imported fix for journald SIGABRTs.

Maxime Hadjinlian maxime.hadjinlian at gmail.com
Tue Oct 13 08:15:09 UTC 2015


Hi Jordan

On Tue, Oct 13, 2015 at 9:53 AM, Thomas Petazzoni <
thomas.petazzoni at free-electrons.com> wrote:

> Maxime, Jordan,
>
> On Tue, 13 Oct 2015 09:47:20 +0200, Maxime Hadjinlian wrote:
>
> > On Tue, Oct 13, 2015 at 5:58 AM, Jordan Yelloz <jordan at yelloz.me> wrote:
> >
> > > Prior to applying this patch, assertion errors would reliably be
> displayed
> > > on the
> > > console. These failed assertions led to crashes in systemd-journald on
> > > startup.
> > >
> > > This patch fixes systemd bug #512:
> > > <https://github.com/systemd/systemd/issues/512>
> > >
> > NAK on this one.
>
> I clearly don't think "NAK on this one" is the appropriate way to react
> to a contribution, especially when the contribution in question is only
> the second one from Jordan.
>
> Here is a much more appropriate response for Jordan:
>
I apologies about the tone of my mail, I though you were an older
contributor I seems to recalled your name, anyway, that's not an excuse for
this.
Don't take it bad and please send many more patches :)

>
> """
> Jordan,
>
> Thanks for your patch! However, it turns out that another patch was
> already posted some time ago to bump the systemd version from 221 to
> 226 (see https://patchwork.ozlabs.org/patch/525867/). Since systemd
> v226 already includes the fix for the problem you're pointing to, it
> seems like a better idea to upgrade systemd to v226 rather than
> backporting just this one fix.
>
> Consequently, we will mark your patch as "Rejected" in our patch
> tracking system, as we will favor the systemd update instead. However,
> what you could do is test the patch at
> https://patchwork.ozlabs.org/patch/525867/ and if it works for you,
> report a Tested-by.
>
> Thanks!
> """
>
> Best regards,
>
> Thomas
> --
> Thomas Petazzoni, CTO, Free Electrons
> Embedded Linux, Kernel and Android engineering
> http://free-electrons.com
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20151013/67dc5dc4/attachment.html>


More information about the buildroot mailing list