[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Progress on merging Eglot?
From: |
Payas Relekar |
Subject: |
Re: Progress on merging Eglot? |
Date: |
Mon, 19 Sep 2022 09:55:58 +0530 |
Thank you! Indeed moving to subdirectory makes history apply to
correct file and shows up as expected. Was also quite straightforward
with git-filter-repo, so here's the updates:
Updates gist:
https://gist.github.com/bhankas/422c34db2e987d4dc3f243e26ba156a3
Updated repo:
https://github.com/bhankas/emacs
Updated history:
https://github.com/bhankas/emacs/commits/master/lisp/progmodes/eglot.el
Now the only remaining issue is commit length. Can someone please
advice on how we can go about it? My brute force thinking is to simply
add newline where length exceeds, delimited at word boundary, but
obviously that's not good enough nor covers sufficient edge cases.
Perhaps a Python/Perl wizard has a one/two liner to take that pain
away?
Thanks,
Payas
On Mon, 19 Sep, 2022, 08:08 Stefan Monnier, <monnier@iro.umontreal.ca> wrote:
>
> > # filter eglot with only eglot.el and find+replace # in commit messages
> > git filter-repo --path eglot.el --message-callback '
> > return re.sub(b" #", b" https://github.com/joaotavora/eglot/issues/",
> >message)
> > return re.sub(b",#", b",https://github.com/joaotavora/eglot/issues/",
> >message)
> > return re.sub(b"(#", b"(https://github.com/joaotavora/eglot/issues/",
> >message)
> > return re.sub(b"Fix#", b"Fix:
> >https://github.com/joaotavora/eglot/issues/", message)
> > return re.sub(b"github#", b"Github:
> >https://github.com/joaotavora/eglot/issues/", message)
>
> I suspect you can get a slightly better result if you tweak this filter
> script so as to rename `eglot.el` to `lisp/progmodes/eglot.el` in each commit.
>
> Then you can also skip the `git read-tree`.
>
>
> Stefan
>
- Re: Progress on merging Eglot?, (continued)
- Re: Progress on merging Eglot?, Payas Relekar, 2022/09/18
- Re: Progress on merging Eglot?, Philip Kaludercic, 2022/09/18
- Re: Progress on merging Eglot?, Dmitry Gutov, 2022/09/18
- Re: Progress on merging Eglot?, Philip Kaludercic, 2022/09/18
- Re: Progress on merging Eglot?, Dmitry Gutov, 2022/09/18
- Re: Progress on merging Eglot?, João Távora, 2022/09/18
- Re: Progress on merging Eglot?, João Távora, 2022/09/18
- Re: Progress on merging Eglot?, Philip Kaludercic, 2022/09/18
- Re: Progress on merging Eglot?, João Távora, 2022/09/18
- Re: Progress on merging Eglot?, Stefan Monnier, 2022/09/18
- Re: Progress on merging Eglot?,
Payas Relekar <=
- Re: Progress on merging Eglot?, Payas Relekar, 2022/09/19
- Re: Progress on merging Eglot?, João Távora, 2022/09/19
- Re: Progress on merging Eglot?, Payas Relekar, 2022/09/19
- Re: Progress on merging Eglot?, Payas Relekar, 2022/09/19
- Re: Progress on merging Eglot?, Payas Relekar, 2022/09/19
- Re: Progress on merging Eglot?, João Távora, 2022/09/19
- Progress on merging Eglot: update, João Távora, 2022/09/23
- Re: Progress on merging Eglot: update, Payas Relekar, 2022/09/24
- Re: Progress on merging Eglot: update, Payas Relekar, 2022/09/24
- Re: Progress on merging Eglot: update, Richard Stallman, 2022/09/24