Date: Mon, 15 Jan 2024 09:37:48 -0700 From: Warner Losh <imp@bsdimp.com> To: John Baldwin <jhb@freebsd.org> Cc: Warner Losh <imp@freebsd.org>, src-committers@freebsd.org, dev-commits-src-all@freebsd.org, dev-commits-src-main@freebsd.org Subject: Re: git: df834e06bbc7 - main - git-arc: Just strip escape sequences form arc log Message-ID: <CANCZdfoNWuX%2BG6gTjGPRugyCxNkGH=wZ1kUF4z93mEQ1qmXxmw@mail.gmail.com> In-Reply-To: <de314243-7cac-4f1c-b013-5c6c8cadd0bc@FreeBSD.org> References: <202401150224.40F2OSBS093537@gitrepo.freebsd.org> <de314243-7cac-4f1c-b013-5c6c8cadd0bc@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
[-- Attachment #1 --] On Mon, Jan 15, 2024 at 9:30 AM John Baldwin <jhb@freebsd.org> wrote: > On 1/14/24 6:24 PM, Warner Losh wrote: > > The branch main has been updated by imp: > > > > URL: > https://cgit.FreeBSD.org/src/commit/?id=df834e06bbc78c03cb6b35f074ae43673c9ce463 > > > > commit df834e06bbc78c03cb6b35f074ae43673c9ce463 > > Author: Warner Losh <imp@FreeBSD.org> > > AuthorDate: 2024-01-15 02:22:04 +0000 > > Commit: Warner Losh <imp@FreeBSD.org> > > CommitDate: 2024-01-15 02:23:45 +0000 > > > > git-arc: Just strip escape sequences form arc log > > > > Just strip escape sequences and Warning: lines from 'arc list' > output. I > > upgraded and they changed and git arc list broke. This restores its > > functionality. > > > > Note: jhb didn't like this, so if others object, I'll fix... > > > > Sponsored by: Netflix > > Reviewed by: markj > > Differential Revision: https://reviews.freebsd.org/D36553 > > Namely, it breaks the color output for reviews in different states, so > Needs Review > shows up as purple, Approved as green, etc. If you find the color output > useful > you'll have to revert this locally. > > Also, your last update in the review said that git arc was no longer > broken for you > back in May implying this commit wasn't needed for git arc to work for you? > Yes and no. I was mistaken. This change is still needed, but its scope can be reduced. The trouble is that it's at the first for the other things I committed and getting it out of the way was a pain (I tried). Since I didn't want to delay things any further, I went ahead and committed and thought I'd fix stuff if there were objections.I'll do that now. Warner [-- Attachment #2 --] <div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jan 15, 2024 at 9:30 AM John Baldwin <<a href="mailto:jhb@freebsd.org">jhb@freebsd.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 1/14/24 6:24 PM, Warner Losh wrote:<br> > The branch main has been updated by imp:<br> > <br> > URL: <a href="https://cgit.FreeBSD.org/src/commit/?id=df834e06bbc78c03cb6b35f074ae43673c9ce463" rel="noreferrer" target="_blank">https://cgit.FreeBSD.org/src/commit/?id=df834e06bbc78c03cb6b35f074ae43673c9ce463</a><br>; > <br> > commit df834e06bbc78c03cb6b35f074ae43673c9ce463<br> > Author: Warner Losh <imp@FreeBSD.org><br> > AuthorDate: 2024-01-15 02:22:04 +0000<br> > Commit: Warner Losh <imp@FreeBSD.org><br> > CommitDate: 2024-01-15 02:23:45 +0000<br> > <br> > git-arc: Just strip escape sequences form arc log<br> > <br> > Just strip escape sequences and Warning: lines from 'arc list' output. I<br> > upgraded and they changed and git arc list broke. This restores its<br> > functionality.<br> > <br> > Note: jhb didn't like this, so if others object, I'll fix...<br> > <br> > Sponsored by: Netflix<br> > Reviewed by: markj<br> > Differential Revision: <a href="https://reviews.freebsd.org/D36553" rel="noreferrer" target="_blank">https://reviews.freebsd.org/D36553</a><br>; <br> Namely, it breaks the color output for reviews in different states, so Needs Review<br> shows up as purple, Approved as green, etc. If you find the color output useful<br> you'll have to revert this locally.<br> <br> Also, your last update in the review said that git arc was no longer broken for you<br> back in May implying this commit wasn't needed for git arc to work for you?<br></blockquote><div><br></div><div>Yes and no. I was mistaken. This change is still needed, but its scope can be reduced.</div><div>The trouble is that it's at the first for the other things I committed and getting it out of the</div><div>way was a pain (I tried). Since I didn't want to delay things any further, I went ahead and</div><div>committed and thought I'd fix stuff if there were objections.I'll do that now.<br></div><div><br></div><div>Warner <br></div></div></div>
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANCZdfoNWuX%2BG6gTjGPRugyCxNkGH=wZ1kUF4z93mEQ1qmXxmw>