92cf602e3809 - main - Copy strcasestr.c from libc to libkern.

Baptiste Daroussin bapt at FreeBSD.org
Fri Jan 8 09:27:14 UTC 2021


On Fri, Jan 08, 2021 at 02:49:31AM +0300, Vladimir Kondratyev wrote:
> On 08.01.2021 02:41, Ravi Pokala wrote:
> > -----Original Message-----
> > From: <owner-src-committers at freebsd.org> on behalf of Vladimir Kondratyev <vladimir at kondratyev.su>
> > Date: 2021-01-07, Thursday at 15:33
> > To: Jessica Clarke <jrtc27 at freebsd.org>, Vladimir Kondratyev <wulf at FreeBSD.org>
> > Cc: "src-committers at freebsd.org" <src-committers at FreeBSD.org>, "dev-commits-src-all at freebsd.org" <dev-commits-src-all at FreeBSD.org>, "dev-commits-src-main at freebsd.org" <dev-commits-src-main at FreeBSD.org>
> > Subject: Re: git: 92cf602e3809 - main - Copy strcasestr.c from libc to libkern.
> > 
> >     On 08.01.2021 02:27, Jessica Clarke wrote:
> >     > On 7 Jan 2021, at 23:20, Vladimir Kondratyev <wulf at FreeBSD.org> wrote:
> >     >> +#include <ctype.h>
> >     >> +#include <string.h>
> >     >> +#include "xlocale_private.h"
> >     >> +
> >     >> +/*
> >     >> + * Find the first occurrence of find in s, ignore case.
> >     >> + */
> >     >> +char *
> >     >> +strcasestr_l(const char *s, const char *find, locale_t locale)
> >     > 
> >     > Locales in the kernel? This feels wrong.
> >     > 
> >     > Jess
> >     > 
> > 
> >     strcasestr.c was copied from userland unmodified just to allow
> >     "git log --follow" to work. Next commit removed locale support from this
> >     file and connected it to build.
> > 
> > Unfortunately, about two-dozen commit messages from you arrived at the same time, and some were out of order; without incrementing change numbers in the subject line, figuring out the actual ordering is non-trivial. :-p
> > 
> > -Ravi (rpokala@)
> > 
> 
> I hope, commit hook will be improved some day. Pushing of two-dozen
> commits one by one is too boring.

The history is in the right order and gitlog/cgit/anything that reads the log
show everything in the right order.

mails are asynchronuous, expecting the mail to always arrive in a given order is
wrong ;)

pushing all those commits at once was imho the right thing to do.

Best regards,
Bapt
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.freebsd.org/pipermail/dev-commits-src-all/attachments/20210108/ec24f354/attachment-0001.sig>


More information about the dev-commits-src-all mailing list