From: Paul Eggert <eggert@cs.ucla.edu>
To: Adhemerval Zanella <adhemerval.zanella@linaro.org>
Cc: GNU C Library development <libc-alpha@sourceware.org>
Subject: Re: [PATCH v5 0/7] Refactor syslog implementation
Date: Thu, 14 Apr 2022 15:12:51 -0700 [thread overview]
Message-ID: <a11b46db-a87b-6fc3-e883-a78f55ab7416@cs.ucla.edu> (raw)
In-Reply-To: <3ff164e7-c247-4275-a8d0-799c104b049e@linaro.org>
On 4/14/22 06:42, Adhemerval Zanella wrote:
>
> On 09/04/2022 03:44, Paul Eggert wrote:
>> If all you did is change gmtime back to localtime it's probably OK.
>>
>> Can you send it to me as just two diffs? One between glibc and what you want. The other between glibc under your previous proposal and glibc under your latest proposal. That will save time for me in reviewing. Thanks.
> Thanks Paul, both diff are in attachment. If you may reply on libc-alpha
> that you are ok I appreciate.
Thanks, this new version looks good.
next prev parent reply other threads:[~2022-04-14 22:12 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-03-21 21:19 Adhemerval Zanella
2022-03-21 21:19 ` [PATCH v5 1/7] support: Add xmkfifo Adhemerval Zanella
2022-03-21 21:19 ` [PATCH v5 2/7] misc: Add syslog test Adhemerval Zanella
2022-03-21 21:19 ` [PATCH v5 3/7] misc: syslog: Fix indentation and style Adhemerval Zanella
2022-03-21 21:19 ` [PATCH v5 4/7] misc: syslog: Simplify implementation Adhemerval Zanella
2022-03-21 21:19 ` [PATCH v5 5/7] misc: syslog: Use fixed-sized buffer and remove memstream Adhemerval Zanella
2022-03-21 21:19 ` [PATCH v5 6/7] misc: syslog: Move SYSLOG_NAME to USE_MISC (BZ #16355) Adhemerval Zanella
2022-03-21 21:19 ` [PATCH v5 7/7] misc: Use 64 bit time_t interfaces on syslog Adhemerval Zanella
[not found] ` <4c23c3a5-59fa-caa5-00d6-7392f6220b55@linaro.org>
[not found] ` <5780fdae-5b8e-e9ff-2e92-91343649a968@cs.ucla.edu>
[not found] ` <3ff164e7-c247-4275-a8d0-799c104b049e@linaro.org>
2022-04-14 22:12 ` Paul Eggert [this message]
2022-04-15 14:00 ` [PATCH v5 0/7] Refactor syslog implementation Adhemerval Zanella
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=a11b46db-a87b-6fc3-e883-a78f55ab7416@cs.ucla.edu \
--to=eggert@cs.ucla.edu \
--cc=adhemerval.zanella@linaro.org \
--cc=libc-alpha@sourceware.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).