git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: Update to Git 2.34.0 breaks application

From
PWPhillip Wood <phillip.wood123@gmail.com>
Date
Nov 22, 2021, 21:43 UTC
Message-ID
<04ab7301-ea34-476c-eae4-4044fef74b91@gmail.com>
In-Reply-To
<ee302c98-da27-da43-e684-c7ec8b225360@gmx.net>
Hi Alex
Thanks for the report
On 22/11/2021 08:42, Alexander Veit wrote:
Show 10 quoted lines
> After an update from Git 2.25.1 to 2.34.0 the Java application that uses 
> the Eclipse JGit library freezes.
> 
> Strace suggests that the JVM receives a SIGTTOU from the child process 
> "git config --system --edit" created with java.lang.ProcessBuilder.
> 
> Upstream[1] has identified 3d411afa[2] as the possible commit that 
> introduced the problem.
> 
> The problem does not occur on all Linux systems.

I think the problem is possibly that git is calling tcsetattr() from a background process group[1]. A possible fix would be to call tcgetpgrp() after opening /dev/tty to see if git is in the foreground process group.

Best Wishes
Phillip
[1] https://www.man7.org/linux/man-pages/man3/tcsetattr.3p.html
Show 14 quoted lines
> -Alex
> 
> Environment:
> OS: Linux Mint 20
> Kernel: 5.4.0-90-generic
> Git 2.34.0 from http://ppa.launchpad.net/git-core/ppa/ubuntu
> Java: OpenJDK 64-Bit Server VM AdoptOpenJDK (build 11.0.10+9, mixed mode)
> 
> 
> Ref.:
> [1] https://bugs.eclipse.org/bugs/show_bug.cgi?id=577358
> [2] 
> https://github.com/git/git/commit/3d411afabc9a96f41d47c07d6af6edda3d29ec92#diff-01b59b6a71e42b9c1251ffbf76a1119b965be087a78538e80e01f9239c8e5880 
> 
Previous: Alexander VeitNext: Carlo Marcelo Arenas Belón
Message 2 of 25 in “Update to Git 2.34.0 breaks application”
  1. Alexander VeitNov 22, 2021
  2. Phillip WoodNov 22, 2021
  3. editor: only save (and restore) the terminal if using a ttyCarlo Marcelo Arenas Belón, Nov 22, 2021
  4. Junio C HamanoNov 22, 2021
  5. Junio C HamanoNov 22, 2021
  6. Junio C HamanoNov 22, 2021
  7. Alexander VeitNov 23, 2021
  8. Carlo ArenasNov 23, 2021
  9. Carlo ArenasNov 22, 2021
  10. Junio C HamanoNov 23, 2021
  11. Johannes SchindelinNov 24, 2021
  12. Carlo ArenasNov 24, 2021
  13. Junio C HamanoNov 24, 2021
  14. Carlo ArenasNov 24, 2021
  15. Junio C HamanoNov 24, 2021
  16. Johannes SchindelinNov 29, 2021
  17. Phillip WoodNov 23, 2021
  18. Junio C HamanoNov 23, 2021
  19. Carlo ArenasNov 23, 2021
  20. Phillip WoodNov 30, 2021
  21. Chris TorekDec 1, 2021
  22. Junio C HamanoDec 1, 2021
  23. Junio C HamanoDec 2, 2021
  24. Carlo ArenasDec 2, 2021
  25. Johannes SchindelinDec 2, 2021

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.