[glibc] regex: copy back from Gnulib
Paul Eggert
eggert@cs.ucla.edu
Wed Sep 22 01:06:59 GMT 2021
On 9/21/21 2:44 PM, Carlos O'Donell wrote:
> In the future please post such changes for review on the list (I don't see it on
> libc-alpha, patchwork, or see any Reviewed-by: tags).
I posted it on libc-alpha a while ago; that's where Joseph picked it up.
https://sourceware.org/pipermail/libc-alpha/2021-August/130571.html
> I would like to review such changes as part of our normal code review.
I'd also like reviews. Unfortunately it's hard to get good regex
reviews. The code is hairy and nobody really understands it, other than
maybe its long-gone author.
> Could you please also update SHARED-FILES with a "# Merged from gnulib YYYY-MM-DD"
> marker for updated files?
Sure, done by committing the attached.
It is unfortunate that glibc has a file for this (SHARED-FILES) that I
didn't know about until now, while Gnulib has its own file in a fancier
format for the same thing (config/srclist.txt), so that I need to
remember to update both. Plus, they surely disagree with each other. At
some point I'll try to find time to iron out the discrepancies.
PS. When I first tried to commit the change I got the following
offputting diagnostic. Is there any chance we could change the
commit-message encoding from ISO-8895-15 to UTF-8? Requiring ISO-8895-15
seems so 2nd-millennium; even downgrading to requiring ASCII would be
better, as ASCII is UTF-8-compatible.
> 3-penguin $ git push
> Enumerating objects: 5, done.
> Counting objects: 100% (5/5), done.
> Delta compression using up to 4 threads
> Compressing objects: 100% (3/3), done.
> Writing objects: 100% (3/3), 459 bytes | 229.00 KiB/s, done.
> Total 3 (delta 2), reused 0 (delta 0), pack-reused 0
> remote: *** Invalid revision history for commit 3e45d44dd2922890aec56fbd415621cb0b56a95f:
> remote: *** It contains characters not in the ISO-8859-15 charset.
> remote: ***
> remote: *** Below is the first line where this was detected (line 1):
> remote: *** | Mention today’s regex merge in SHARED-FILES
> remote: *** ^
> remote: *** |
> remote: ***
> remote: *** Please amend the commit's revision history to remove it
> remote: *** and try again.
> remote: error: hook declined to update refs/heads/master
> To sourceware.org:/git/glibc.git
> ! [remote rejected] master -> master (hook declined)
> error: failed to push some refs to 'sourceware.org:/git/glibc.git'
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Mention-today-s-regex-merge-in-SHARED-FILES.patch
Type: text/x-patch
Size: 1577 bytes
Desc: not available
URL: <https://sourceware.org/pipermail/libc-alpha/attachments/20210921/21a8654a/attachment.bin>
More information about the Libc-alpha
mailing list