Dear Michal
Thanks for the bug report, unfortunately I'm unable to reproduce it here
using git 2.20.1 (see below). Knowing a little about how the todo list
is created I cannot think how multibyte characters would break it. What
command line were you using to start the rebase?
Best Wishes
Phillip
$ git clone --shallow-since='11-1-2019'
https://github.com/illumos/illumos-gate.git
Cloning into 'illumos-gate'...
remote: Enumerating objects: 54348, done.
remote: Counting objects: 100% (54348/54348), done.
remote: Compressing objects: 100% (39008/39008), done.
remote: Total 54348 (delta 19671), reused 31187 (delta 11884), pack-reused 0
Receiving objects: 100% (54348/54348), 148.91 MiB | 1.18 MiB/s, done.
Resolving deltas: 100% (19671/19671), done.
Checking out files: 100% (47352/47352), done.
$ cd ~/src/illumos-gate
$ git log --oneline -1
f482e26c (HEAD -> master, origin/master, origin/HEAD) 10233 dboot:
process_module() is missing newline at the end of the string Reviewed
by: Sebastian Wiedenroth <wiedi@xxxxxxxxxx> Reviewed by: Jason King
<jbk@xxxxxxxxxx> Reviewed by: Andy Stormont
<astormont@xxxxxxxxxxxxxxxxxx> Approved by: Robert Mustacchi <rm@xxxxxxxxxx>
$ GIT_SEQUENCE_EDITOR=cat git rebase -i
08487eea68a2fa501b5042131c6db068089f82e1
hint: Waiting for your editor to close the file...
pick 1ea94c75 10202 loader: use screen-#rows to find bottom left
co-ordinates Reviewed by: Toomas Soome <tsoome@xxxxxx> Reviewed by:
Gergő Mihály Doma <domag02@xxxxxxxxx> Approved by: Robert Mustacchi
<rm@xxxxxxxxxx>
pick cadd68ea 10078 smatch fixes for UCB Reviewed by: Andy Fiddaman
<omnios@xxxxxxxxxxxxx> Reviewed by: Toomas Soome <tsoome@xxxxxx>
Reviewed by: Peter Tribble <peter.tribble@xxxxxxxxx> Approved by: Robert
Mustacchi <rm@xxxxxxxxxx>
pick 05ede3db 10079 smatch Makefile changes for usr/src/lib Reviewed by:
Andy Fiddaman <andy@xxxxxxxxxxxx> Approved by: Robert Mustacchi
<rm@xxxxxxxxxx>
pick 5661bb76 10080 smatch Makefile changes for usr/src/cmd Reviewed by:
Andy Fiddaman <andy@xxxxxxxxxxxx> Approved by: Robert Mustacchi
<rm@xxxxxxxxxx>
pick 15c07adc 10081 smatch indenting fixes for usr/src/uts Reviewed by:
Toomas Soome <tsoome@xxxxxx> Reviewed by: Peter Tribble
<peter.tribble@xxxxxxxxx> Reviewed by: Andy Fiddaman <andy@xxxxxxxxxxxx>
Approved by: Robert Mustacchi <rm@xxxxxxxxxx>
pick c653bb47 10197 build smatch in parallel Reviewed by: Andy Fiddaman
<andy@xxxxxxxxxxxx> Approved by: Robert Mustacchi <rm@xxxxxxxxxx>
pick 161294fe 10217 mdb: r9w isn't r8w 10218 CONV_CAP_VAL_HW2_BUFSIZE
wasn't updated 10219 AV_386_2_AVX512VPOPCDQ elfcap name should be
consistent Reviewed by: Dan McDonald <danmcd@xxxxxxxxxx> Reviewed by:
Rob Johnston <rob.johnston@xxxxxxxxxx> Reviewed by: Patrick Mooney
<patrick.mooney@xxxxxxxxxx> Reviewed by: Andy Fiddaman
<andy@xxxxxxxxxxxx> Approved by: Dan McDonald <danmcd@xxxxxxxxxx>
pick f482e26c 10233 dboot: process_module() is missing newline at the
end of the string Reviewed by: Sebastian Wiedenroth <wiedi@xxxxxxxxxx>
Reviewed by: Jason King <jbk@xxxxxxxxxx> Reviewed by: Andy Stormont
<astormont@xxxxxxxxxxxxxxxxxx> Approved by: Robert Mustacchi <rm@xxxxxxxxxx>
# Rebase 08487eea..f482e26c onto 08487eea (8 commands)
#
# Commands:
# p, pick <commit> = use commit
# r, reword <commit> = use commit, but edit the commit message
# e, edit <commit> = use commit, but stop for amending
# s, squash <commit> = use commit, but meld into previous commit
# f, fixup <commit> = like "squash", but discard this commit's log message
# x, exec <command> = run command (the rest of the line) using shell
# b, break = stop here (continue rebase later with 'git rebase --continue')
# d, drop <commit> = remove commit
# l, label <label> = label current HEAD with a name
# t, reset <label> = reset HEAD to a label
# m, merge [-C <commit> | -c <commit>] <label> [# <oneline>]
# . create a merge commit using the original merge commit's
# . message (or the oneline, if no original merge commit was
# . specified). Use -c <commit> to reword the commit message.
#
# These lines can be re-ordered; they are executed from top to bottom.
#
# If you remove a line here THAT COMMIT WILL BE LOST.
#
# However, if you remove everything, the rebase will be aborted.
#
# Note that empty commits are commented out
Successfully rebased and updated refs/heads/master.
On 15/01/2019 19:29, Michal Nowak wrote:
Hello,
on OpenIndiana 2018.10 (illumos kernel) line of the interactive rebase
text after a particular name (Gergő Mihály Doma) is broken:
pick 1ea94c756c 10202 loader: use screen-#rows to find bottom left
co-ordinates Reviewed by: Toomas Soome <tsoome@xxxxxx> Reviewed by:
Gergő Mihály Doma <domag02@xxxxxxxxx> Approved by: Robert Mustacchi
<rm@xxxxxxxxxx>
p
pick cadd68ea00 10078 smatch fixes for UCB Reviewed by: Andy Fiddaman
<omnios@xxxxxxxxxxxxx> Reviewed by: Toomas Soome <tsoome@xxxxxx>
Reviewed by: Peter Tribble <peter.tribble@xxxxxxxxx> Approved by: Robert
Mustacchi <rm@xxxxxxxxxx>
Source: https://github.com/illumos/illumos-gate.git
See the second item which only has "p".
This is with LC_ALL=en_US.UTF-8 (same with cs_CZ.UTF-8), with LC_ALL=C
the text is fine.
I checked various editors (vim, nano, cat), they are all the same.
`git log` (PAGER is set to `/usr/bin/less -ins`) shows the particular
commit correctly.
I tested following git version: 2.16.5, 2.19.2, 2.20.1, git.next branch.
Here's the configure output:
https://paste.ec/paste/oDdydNxZ#r3avan8BL+8lldFMo928cw3eXSServTsSkGPW4jRBOd
Michal
