This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH] Avoid redundant shift character in iconv output at block boundary (bug 17197)
- From: Martin Sebor <msebor at gmail dot com>
- To: Andreas Schwab <schwab at suse dot de>
- Cc: libc-alpha at sourceware dot org
- Date: Mon, 14 Dec 2015 09:23:10 -0700
- Subject: Re: [PATCH] Avoid redundant shift character in iconv output at block boundary (bug 17197)
- Authentication-results: sourceware.org; auth=none
- References: <mvm7fo1pbwy dot fsf at hawking dot suse dot de> <5668C219 dot 8070606 at gmail dot com> <mvmsi3abtbl dot fsf at hawking dot suse dot de> <5669C008 dot 7050902 at gmail dot com> <mvmr3ipa03u dot fsf at hawking dot suse dot de>
On 12/14/2015 01:46 AM, Andreas Schwab wrote:
Martin Sebor <msebor@gmail.com> writes:
Great, thank you for confirming that. Will you be committing
the patch sometime soon or is something standing in the way?
Lack of review.
Yes, I see you submitted the patch for review a number of times with
no response.
For what it's worth, I've reviewed and successfully tested the patch.
I don't know the glibc commit process well enough to tell if someone
else needs to review and approve it and who that might be in this
case. However, based on my reading of the Consensus page on the wiki
I would expect anyone to be able to commit this patch with only no
approval (I don't see the iconvdata component mentioned on in the
Reviewers by component section on the MAINTAINERS page):
Anyone can commit a locale related change where a bugzilla issue
exists, government sources are cited, common uses are cited, and
if there is an original author for the locale, that original author
ACKs the change. No developer review required. Post the patch and
ChangeLog to libc-alpha with a short message and then push the
commit.
Do you agree it's sufficient to go ahead with the commit or do you
think someone else needs to review and approve the patch? If the
latter, do you know who that might be?
Incidentally, while searching for more background on this bug I
came across a commit from 2014 of the same patch into the openSUSE
repository:
https://www.mail-archive.com/opensuse-commit@opensuse.org/msg61322.html
If another review is required, hopefully that will help increase
confidence in the patch so that it can be approved for commit here.
Thanks
Martin