git.github.io icon indicating copy to clipboard operation
git.github.io copied to clipboard

Any comment about upcoming Git Rev News edition 115

Open chriscool opened this issue 1 year ago • 16 comments

A currently mostly empty draft is there:

https://github.com/git/git.github.io/blob/master/rev_news/drafts/edition-115.md

Feel free to comment in this issue, suggest topics, suggest persons to interview, or use the edit button (that looks like a pen) to edit and create a pull request with the changes you would like.

Let's try to publish this edition around the end of September 2024!

Thanks!

cc @jnareb @mjaix @sivaraam @gitster @stepnem

chriscool avatar Sep 01 '24 20:09 chriscool

Some things we can talk about:

  • [x] results of GSoC 2024
  • [x] Git participating in the next Outreachy round
  • [X] Git Merge 2024

chriscool avatar Sep 28 '24 10:09 chriscool

My links have landed in 78f48f9 (with little addition in f699ee9).

jnareb avatar Oct 01 '24 00:10 jnareb

@jnareb thanks for the great links!

About your little addition, I think it's a good idea to talk about it, but I left a comment: https://github.com/git/git.github.io/commit/f699ee92997b3aeccb2019b66fc6c741ee1c3a5d#commitcomment-147419315

chriscool avatar Oct 01 '24 08:10 chriscool

@mjaix thanks for your great fixes in ef17404d623a3f271bf2c5002c907be1b191905d and 704771623717ee4eb8d02cb9607a5d7d67826ac9!

However, about "In summary, the root problem can be understood as a portability one." that you added, I am not sure it's very clear why it's only a portability issue, as I am not sure Git would use the pipefail option even if it was available everywhere. I think that Git shell scripts just use no (or perhaps very few) shell options for a number of reasons, not just portability ones.

(For example, one of the benefits of using very few options is that it's easier for readers to understand what a snippet of code or a patch hunk does without having to look around everywhere for which options might or might not be set in the context of the snippet or hunk.)

It might be a good idea to add links to "the pipefail option" and "named pipes", like perhaps this one https://tldp.org/LDP/abs/html/options.html#OPTIONSREF (for the pipefail option) let me know if you want me to do it.

chriscool avatar Oct 01 '24 09:10 chriscool

It might be a good idea to add links to "the pipefail option" and "named pipes", like perhaps this one https://tldp.org/LDP/abs/html/options.html#OPTIONSREF (for the pipefail option) let me know if you want me to do it.

If talking about portability it might make more sense to link to https://pubs.opengroup.org/onlinepubs/9799919799/utilities/V3_chap02.html#tag_19_09_02 instead, as 'pipefail' has recently been added to POSIX (issue 8). (The TLDP link seems to be about Bash.)

stepnem avatar Oct 01 '24 09:10 stepnem

@stepnem yeah, nice link, thanks!

chriscool avatar Oct 01 '24 10:10 chriscool

I've submitted #731 with a few nits I've found, but I also don't understand what "xs-related tools" in the Stacked Diffs paragraph refer to. Searching for "xs" in the current draft or the mentioned edition 111 didn't help. Am I missing something?

stepnem avatar Oct 01 '24 11:10 stepnem

I've submitted #731 with a few nits I've found, but I also don't understand what "xs-related tools" in the Stacked Diffs paragraph refer to. Searching for "xs" in the current draft or the mentioned edition 111 didn't help. Am I missing something?

Good find. It was a typo (Ctrl+X Ctrl+S is save file in GNU Emacs, and if Ctrl does not take...). Fixed in fd76e5b

jnareb avatar Oct 01 '24 12:10 jnareb

Good find. It was a typo (Ctrl+X Ctrl+S is save file in GNU Emacs, and if Ctrl does not take...). Fixed in fd76e5b

Thanks, I've fixed up the resulting merge conflict and re-pushed my PR branch.

stepnem avatar Oct 01 '24 12:10 stepnem

Thanks @stepnem , your PR is merged!

I would have hoped to have a reply from @mjaix but I am going to publish the edition in a few hours anyway.

chriscool avatar Oct 01 '24 16:10 chriscool

Sorry, Access from $work I difficult.

mjaix avatar Oct 01 '24 17:10 mjaix

Sorry, Access from $work I difficult, but now established .. Feel free to leave out my hallucination regarding portability, or even the complete part in brackets.

mjaix avatar Oct 01 '24 18:10 mjaix

The first link for "Host your own Radicle seed node" is actually linking to the previous bullet-point's GitVersion article.

steadmon avatar Oct 01 '24 19:10 steadmon

@mjaix I think portability is a part of the issue but not the only one. I will change the part in brackets a bit. Thanks for your reply!

@steadmon thanks for the report! I will try to change the link.

chriscool avatar Oct 01 '24 19:10 chriscool

@steadmon link changed in 57bdc63564bb2309df6c60055e619cd2f505149e

@mjaix changes made in 3516914526dca0af47d645cafe87ea448711f822

chriscool avatar Oct 01 '24 19:10 chriscool

Published and announce email sent: https://lore.kernel.org/git/CAP8UFD1pRLHAxVQGQ-oxzKTDSp+EuspR8BtGhWNch8MRbcfBzw@mail.gmail.com/

Draft email was: https://lore.kernel.org/git/CAP8UFD350-iO7o0xAw9qg5f5uX8tN86so2TYf6AvD5Y1C=WyEA@mail.gmail.com/

chriscool avatar Oct 01 '24 20:10 chriscool