[PATCH] ChangeLog: Correct two dates.

classic Classic list List threaded Threaded
12 messages Options
Reply | Threaded
Open this post in threaded view
|

[PATCH] ChangeLog: Correct two dates.

Rafal Luzynski
OK for master?

This is trivial and very nit-picky but I'd better ask before just in
case the dates are exactly as the authors wanted them to be.

It's only in master and only after 2.29 release.  Also I'm not touching
one-off errors as long as they don't break the chronological order.
After all, these dates might have been correct in some time zones. :)

Regards,

Rafal


--------------------------------------------------------------------
From: Rafal Luzynski <[hidden email]>
Date: Wed, 27 Mar 2019 12:46:24 +0100
Subject: [PATCH] ChangeLog: Correct two dates.

---
 ChangeLog | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index bd76c1e..b90af4b 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,4 +1,4 @@
-2019-02-26  Adhemerval Zanella  <[hidden email]>
+2019-03-26  Adhemerval Zanella  <[hidden email]>
 
  * math/math.h (fpclassify, isfinite, isnormal, isnan): Use builtin for
  clang 2.8.
@@ -184,7 +184,7 @@
  (HWCAP_S390_VXRS_EXT2, HWCAP_S390_VXRS_PDE, HWCAP_S390_SORT,
  HWCAP_S390_DFLT): Define.
 
-2019-03-05  Gabriel F. T. Gomes  <[hidden email]>
+2019-03-21  Gabriel F. T. Gomes  <[hidden email]>
 
  [BZ #24372]
  * iconv/iconvconfig.c (write_output): Add parentheses to get rid
--
2.7.5
Reply | Threaded
Open this post in threaded view
|

Re: [PATCH] ChangeLog: Correct two dates.

Carlos O'Donell-5
On 3/27/19 7:53 AM, Rafal Luzynski wrote:
> OK for master?
>
> This is trivial and very nit-picky but I'd better ask before just in
> case the dates are exactly as the authors wanted them to be.
>
> It's only in master and only after 2.29 release.  Also I'm not touching
> one-off errors as long as they don't break the chronological order.
> After all, these dates might have been correct in some time zones. :)

No, these dates were obviously mistakes.

Please correct them. They should match checkin dates in git.

Reviewed-by: Carlos O'Donell <[hidden email]>  
 

> --------------------------------------------------------------------
> From: Rafal Luzynski <[hidden email]>
> Date: Wed, 27 Mar 2019 12:46:24 +0100
> Subject: [PATCH] ChangeLog: Correct two dates.
>
> ---
>   ChangeLog | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/ChangeLog b/ChangeLog
> index bd76c1e..b90af4b 100644
> --- a/ChangeLog
> +++ b/ChangeLog
> @@ -1,4 +1,4 @@
> -2019-02-26  Adhemerval Zanella  <[hidden email]>
> +2019-03-26  Adhemerval Zanella  <[hidden email]>
>  
>   * math/math.h (fpclassify, isfinite, isnormal, isnan): Use builtin for
>   clang 2.8.
> @@ -184,7 +184,7 @@
>   (HWCAP_S390_VXRS_EXT2, HWCAP_S390_VXRS_PDE, HWCAP_S390_SORT,
>   HWCAP_S390_DFLT): Define.
>  
> -2019-03-05  Gabriel F. T. Gomes  <[hidden email]>
> +2019-03-21  Gabriel F. T. Gomes  <[hidden email]>
>  
>   [BZ #24372]
>   * iconv/iconvconfig.c (write_output): Add parentheses to get rid
>


--
Cheers,
Carlos.
Reply | Threaded
Open this post in threaded view
|

Re: [PATCH] ChangeLog: Correct two dates.

Rafal Luzynski
Carlos,

27.03.2019 14:41 Carlos O'Donell <[hidden email]> wrote:

> On 3/27/19 7:53 AM, Rafal Luzynski wrote:
> > [...]
> > It's only in master and only after 2.29 release.  Also I'm not touching
> > one-off errors as long as they don't break the chronological order.
> > After all, these dates might have been correct in some time zones. :)
>
> No, these dates were obviously mistakes.
>
> Please correct them. They should match checkin dates in git.
>
> Reviewed-by: Carlos O'Donell <[hidden email]>  

Thanks for your reply.  However, your answer is ambiguous, probably
because my questions were ambiguous.  After reading twice I must ask
if I understand you correctly.  Do you mean that those corrections
are OK but I should also correct more dates if I discover that the
dates do not match?

Does your answer mean this patch is OK to push but I should prepare
another similar patch and send it for a review?

Regards,

Rafal
Reply | Threaded
Open this post in threaded view
|

Re: [PATCH] ChangeLog: Correct two dates.

Carlos O'Donell-5
On 3/27/19 9:52 AM, Rafal Luzynski wrote:

> Carlos,
>
> 27.03.2019 14:41 Carlos O'Donell <[hidden email]> wrote:
>> On 3/27/19 7:53 AM, Rafal Luzynski wrote:
>>> [...]
>>> It's only in master and only after 2.29 release.  Also I'm not touching
>>> one-off errors as long as they don't break the chronological order.
>>> After all, these dates might have been correct in some time zones. :)
>>
>> No, these dates were obviously mistakes.
>>
>> Please correct them. They should match checkin dates in git.
>>
>> Reviewed-by: Carlos O'Donell <[hidden email]>
>
> Thanks for your reply.  However, your answer is ambiguous, probably
> because my questions were ambiguous.  After reading twice I must ask
> if I understand you correctly.  Do you mean that those corrections
> are OK but I should also correct more dates if I discover that the
> dates do not match?

My 'Reviewed-by' indicates my belief that your patch is OK for master.

> Does your answer mean this patch is OK to push but I should prepare
> another similar patch and send it for a review?

Yes it is OK to push your current patch.

Yes it is OK to push *other* patches that correct the dates to match
the git commit dates.

In the future we'll generate the Changelog entry from the git commit
and then the dates will never be wrong.

Right Siddhesh?

--
Cheers,
Carlos.
Reply | Threaded
Open this post in threaded view
|

Re: [PATCH] ChangeLog: Correct two dates.

Zack Weinberg-2
On Wed, Mar 27, 2019 at 10:33 AM Carlos O'Donell <[hidden email]> wrote:
> Yes it is OK to push *other* patches that correct the dates to match
> the git commit dates.

For the record, I would have assumed that correcting the date in a
ChangeLog entry to match the git commit date was like correcting a
typo in a comment: no approval required, just post the patch and then
go ahead and commit it.

zw
Reply | Threaded
Open this post in threaded view
|

Added ChangeLog date update to Consensus "Trivial Bug-Fix Changes"

Carlos O'Donell-5
On 3/27/19 10:36 AM, Zack Weinberg wrote:
> On Wed, Mar 27, 2019 at 10:33 AM Carlos O'Donell <[hidden email]> wrote:
>> Yes it is OK to push *other* patches that correct the dates to match
>> the git commit dates.
>
> For the record, I would have assumed that correcting the date in a
> ChangeLog entry to match the git commit date was like correcting a
> typo in a comment: no approval required, just post the patch and then
> go ahead and commit it.

I agree.

I've added it to Consensus.

https://sourceware.org/glibc/wiki/Consensus#Trivial_Bug-Fix_Changes
~~~
  * Anyone can commit a change to master fixing the date in a ChangeLog
entry to match the git commit date. There is no need to post the patch
for review, simply make the change and then push the commit. This is
only allowed for master where consensus is that dates should be
correct. There is still no clear consensus around what the ChangeLog
date should be for a cherry-picked change into a non-master branch.
~~~

--
Cheers,
Carlos.
Reply | Threaded
Open this post in threaded view
|

Re: [PATCH] ChangeLog: Correct two dates.

Gabriel F. T. Gomes-2
In reply to this post by Rafal Luzynski
On Wed, Mar 27 2019, Rafal Luzynski wrote:
>  
> -2019-03-05  Gabriel F. T. Gomes  <[hidden email]>
> +2019-03-21  Gabriel F. T. Gomes  <[hidden email]>

Sorry about that.  It was definitely a mistake from my part (I probably
copied the line from a previous ChangeLog entry with my name on it).

Thanks.
Reply | Threaded
Open this post in threaded view
|

Re: [PATCH] ChangeLog: Correct two dates.

Siddhesh Poyarekar-8
In reply to this post by Carlos O'Donell-5
On 27/03/19 8:02 PM, Carlos O'Donell wrote:
> In the future we'll generate the Changelog entry from the git commit
> and then the dates will never be wrong.
>
> Right Siddhesh?

That is correct, provided of course that the git commit log dates are
correct ;)

Siddhesh
Reply | Threaded
Open this post in threaded view
|

Re: [PATCH] ChangeLog: Correct two dates.

Adhemerval Zanella-2
In reply to this post by Rafal Luzynski


On 27/03/2019 08:53, Rafal Luzynski wrote:

> OK for master?
>
> This is trivial and very nit-picky but I'd better ask before just in
> case the dates are exactly as the authors wanted them to be.
>
> It's only in master and only after 2.29 release.  Also I'm not touching
> one-off errors as long as they don't break the chronological order.
> After all, these dates might have been correct in some time zones. :)
>
> Regards,
>
> Rafal
>
>

Thanks for catching it.

> --------------------------------------------------------------------
> From: Rafal Luzynski <[hidden email]>
> Date: Wed, 27 Mar 2019 12:46:24 +0100
> Subject: [PATCH] ChangeLog: Correct two dates.
>
> ---
>  ChangeLog | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/ChangeLog b/ChangeLog
> index bd76c1e..b90af4b 100644
> --- a/ChangeLog
> +++ b/ChangeLog
> @@ -1,4 +1,4 @@
> -2019-02-26  Adhemerval Zanella  <[hidden email]>
> +2019-03-26  Adhemerval Zanella  <[hidden email]>
>  
>   * math/math.h (fpclassify, isfinite, isnormal, isnan): Use builtin for
>   clang 2.8.
> @@ -184,7 +184,7 @@
>   (HWCAP_S390_VXRS_EXT2, HWCAP_S390_VXRS_PDE, HWCAP_S390_SORT,
>   HWCAP_S390_DFLT): Define.
>  
> -2019-03-05  Gabriel F. T. Gomes  <[hidden email]>
> +2019-03-21  Gabriel F. T. Gomes  <[hidden email]>
>  
>   [BZ #24372]
>   * iconv/iconvconfig.c (write_output): Add parentheses to get rid
>
Reply | Threaded
Open this post in threaded view
|

Re: [PATCH] ChangeLog: Correct two dates.

Rafal Luzynski
In reply to this post by Carlos O'Donell-5
27.03.2019 15:32 Carlos O'Donell <[hidden email]> wrote:
> On 3/27/19 9:52 AM, Rafal Luzynski wrote:
> [...]
> > Does your answer mean this patch is OK to push but I should prepare
> > another similar patch and send it for a review?
>
> Yes it is OK to push your current patch.

Thank you, now I understand.  Pushed to master.

Regards,

Rafal
Reply | Threaded
Open this post in threaded view
|

Re: [PATCH] ChangeLog: Correct two dates.

Rafal Luzynski
In reply to this post by Gabriel F. T. Gomes-2
27.03.2019 16:52 "Gabriel F. T. Gomes" <[hidden email]> wrote:
> [...]
> Sorry about that.  It was definitely a mistake from my part (I probably
> copied the line from a previous ChangeLog entry with my name on it).
>
> Thanks.

I apologize those people who are explicitly mentioned in these patches.
It was not my intention to associate your names with omissions, it is
just obviously impossible to avoid.

Regards,

Rafal
Reply | Threaded
Open this post in threaded view
|

[PATCH COMMITTED] ChangeLog: Correct more dates.

Rafal Luzynski
In reply to this post by Rafal Luzynski
---
 ChangeLog | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index b90af4b..19d3e90 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1102,7 +1102,7 @@
  (wait_tid): Adjust arguments and call futex_wait with ctid_val
  as assumed current value of ctid_ptr.
 
-2019-02-15  Florian Weimer  <[hidden email]>
+2019-02-16  Florian Weimer  <[hidden email]>
 
  Check that non-sysdeps headers have wrapper headers.
  * scripts/check-wrapper-headers.py: New file.
@@ -1111,7 +1111,7 @@
  * Rules (tests-special): Add check-wrapper-headers.out.
  (check-wrapper-headers.out): New target.
 
-2019-02-15  Florian Weimer  <[hidden email]>
+2019-02-16  Florian Weimer  <[hidden email]>
 
  Add missing header wrappers under include/.
  * include/ar.h: New file.
@@ -1134,7 +1134,7 @@
  * include/sys/vfs.h: Likewise.
  * include/wait.h: Likewise.
 
-2019-02-15  Florian Weimer  <[hidden email]>
+2019-02-16  Florian Weimer  <[hidden email]>
 
  * nptl_db/proc_service.h: Move to ...
  * sysdeps/nptl/proc_service.h: ... here.
@@ -1465,7 +1465,7 @@
  Likewise.
  * sysdeps/unix/sysv/linux/tst-setgetname.c (gettid): Remove.
 
-2019-02-07  Florian Weimer  <[hidden email]>
+2019-02-08  Florian Weimer  <[hidden email]>
 
  * rt/clock-compat.c (COMPAT_REDIRECT): Turn librt forwarders into
  compatibility symbols.
--
2.7.5