# Draft of Git Rev News edition 63 3 messages from 2020-05-26 to 2020-05-27. Participants: Christian Couder, Damien Robert. Thread: https://gitlist.dev/t/53552 ## Christian Couder, 2020-05-26 17:10 Subject: Draft of Git Rev News edition 63 Message-ID: URL: https://gitlist.dev/e/CAP8UFD3MvaAPMUwc%3DhW-bayDbNpxSX3jtMiPQro4b2Ai17GkNg%40mail.gmail.com ``` Hi everyone! A draft of a new Git Rev News edition is available here: https://github.com/git/git.github.io/blob/master/rev_news/drafts/edition-63.md Everyone is welcome to contribute in any section either by editing the above page on GitHub and sending a pull request, or by commenting on this GitHub issue: https://github.com/git/git.github.io/issues/430 You can also reply to this email. In general all kinds of contribution, for example proofreading, suggestions for articles or links, help on the issues in GitHub, and so on, are very much appreciated. I tried to cc everyone who appears in this edition, but maybe I missed some people, sorry about that. Jakub, Markus, Kaartic and me plan to publish this edition on Thursday May 28th in the evening. Thanks, Christian. ``` ## Damien Robert, 2020-05-27 21:11 Subject: Re: Draft of Git Rev News edition 63 Message-ID: <20200527211140.qbgbbitoxr4mx7gk@feanor> URL: https://gitlist.dev/e/20200527211140.qbgbbitoxr4mx7gk%40feanor In-Reply-To: ``` From Christian Couder, Tue 26 May 2020 at 19:10:32 (+0200) : > A draft of a new Git Rev News edition is available here: > https://github.com/git/git.github.io/blob/master/rev_news/drafts/edition-63.md > # Reviews > remote.c: fix handling of push:remote_ref Wow! I don't know who was motivated enough to go through the history of this series, but I am impressed. I just sent a pull request with some suggestions. This series was a complete mess, a lesson learned is that I should stay focused on the original scope of the series, rather than try to extend it mid-way when I find other bugs in adjacent areas. And instead fix the new uncovered bugs in other series. > This led to some release confusion as Junio then had merged the 2 patches to the next branch, while Damien had sent a version 8 that contains only his patch without the triangular workflow fixes and test cases. > This last version is now merged in the pu branch, and will is likely to find its way to master, while hopefully the fixes related to triangular workflows will be re-sent separately. In "What's cooking in git", the series is marked as stalled, so I think Junio is waiting for a reroll. I haven't forgotten about this series, but with the current situation I did not have time to rework on it. Meanwhile, the version v8 in 'pu' does not detect triangular workflows (since I ejected that patch). The version v6 in 'next' does, but there it has the corner case of a branch 'foo' which has a pushRemote=foobar but no remote and 'origin' does not exists. In this case %(push:remote_ref) detects a triangular workflow but `git push` does not. So this is really a minor corner case (and I actually think its `git push` which should be fixed here), so it is not too bad that this version is in 'next', especially since there are still other bugs in the ref-filter machinery anyway (see eg https://public-inbox.org/git/20200418173651.djzriazxj5kbo6ax@doriath/ and https://public-inbox.org/git/20200416152145.wp2zeibxmuyas6y6@feanor/). Best regards, Damien ``` ## Christian Couder, 2020-05-27 21:54 Subject: Re: Draft of Git Rev News edition 63 Message-ID: URL: https://gitlist.dev/e/CAP8UFD1t8e%2B7_fMPhAFFBQSZ3G1Xxvw%2B%3D%2BmwfX4LB4uKJZek6A%40mail.gmail.com In-Reply-To: <20200527211140.qbgbbitoxr4mx7gk@feanor> ``` On Wed, May 27, 2020 at 11:11 PM Damien Robert wrote: > > From Christian Couder, Tue 26 May 2020 at 19:10:32 (+0200) : > > A draft of a new Git Rev News edition is available here: > > https://github.com/git/git.github.io/blob/master/rev_news/drafts/edition-63.md > > > # Reviews > > remote.c: fix handling of push:remote_ref > > Wow! I don't know who was motivated enough to go through the history of this > series, but I am impressed. I just sent a pull request with some > suggestions. Thanks! I merged your pull request and then reformatted the result a bit. > This series was a complete mess, a lesson learned is that I should stay > focused on the original scope of the series, rather than try to extend it > mid-way when I find other bugs in adjacent areas. And instead fix the new > uncovered bugs in other series. Yeah, I hope that this lesson can be learned by others reading the article. > In "What's cooking in git", the series is marked as stalled, so I think Junio > is waiting for a reroll. I haven't forgotten about this series, but with > the current situation I did not have time to rework on it. Yeah, no worries about that. It's ok to take your time. > Meanwhile, the version v8 in 'pu' does not detect triangular workflows (since > I ejected that patch). > > The version v6 in 'next' does, but there it has the corner case of > a branch 'foo' which has a pushRemote=foobar but no remote and 'origin' > does not exists. In this case %(push:remote_ref) detects a triangular > workflow but `git push` does not. > > So this is really a minor corner case (and I actually think its `git push` which > should be fixed here), so it is not too bad that this version is > in 'next', especially since there are still other bugs in the ref-filter > machinery anyway > (see eg > https://public-inbox.org/git/20200418173651.djzriazxj5kbo6ax@doriath/ and > https://public-inbox.org/git/20200416152145.wp2zeibxmuyas6y6@feanor/). Yeah, I think both versions v6 and v8 are already significant improvements over the current situation. Thanks, Christian. ```