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

[PATCH v2 0/8] fix push --progress over file://, git://, etc.

From
Tay Ray Chuan <rctay89@gmail.com>
Date
Oct 16, 2010, 18:36 UTC
Message-ID
<1287254223-4496-1-git-send-email-rctay89@gmail.com>
In-Reply-To
<AANLkTimo=Bd_XGvX=TPzVsds3xQGy9126+7Qg+zvk=d2@mail.gmail.com>

This patch series addresses the issue of git push not displaying progress messages to non-tty stderr, even if --progress is used. As suggested by the subject, this issue afflicts the "builtin smart transports" - file://, git://, ssh://. (All of them use git_transport_push() and thus git-pack-objects.)

The last patch contains the actual fix; most of the other patches improve tests that depend on a tty.

Contents: [PATCH v2 0/8] fix push --progress over file://, git://, etc. [PATCH v2 1/8] tests: factor out terminal handling from t7006 [PATCH v2 2/8] tests: test terminal output to both stdout and stderr [PATCH v2 3/8] test-lib: allow test code to check the list of declared prerequisites [PATCH v2 4/8] test_terminal: catch use without TTY prerequisite [PATCH v2 5/8] test_terminal: give priority to test-terminal.perl usage [PATCH v2 6/8] t5523-push-upstream: add function to ensure fresh upstream repo [PATCH v2 7/8] t5523-push-upstream: test progress messages [PATCH v2 8/8] push: pass --progress down to git-pack-objects

Jeff King (3):
  tests: factor out terminal handling from t7006
  tests: test terminal output to both stdout and stderr
  push: pass --progress down to git-pack-objects
Jonathan Nieder (3):
  test-lib: allow test code to check the list of declared prerequisites
  test_terminal: catch use without TTY prerequisite
  test_terminal: give priority to test-terminal.perl usage
Tay Ray Chuan (2):
  t5523-push-upstream: add function to ensure fresh upstream repo
  t5523-push-upstream: test progress messages
 builtin/send-pack.c        |    3 ++
 send-pack.h                |    1 +
 t/lib-terminal.sh          |   39 +++++++++++++++++++++++
 t/t5523-push-upstream.sh   |   44 +++++++++++++++++++++++++-
 t/t7006-pager.sh           |   38 +---------------------
 t/t7006/test-terminal.perl |   58 ----------------------------------
 t/test-lib.sh              |   26 +++++++++++----
 t/test-terminal.perl       |   75 ++++++++++++++++++++++++++++++++++++++++++++
 transport.c                |    1 +
 9 files changed, 183 insertions(+), 102 deletions(-)
 create mode 100644 t/lib-terminal.sh
 delete mode 100755 t/t7006/test-terminal.perl
 create mode 100755 t/test-terminal.perl
-- 
1.7.2.2.513.ge1ef3
Previous: Tay Ray ChuanNext: Tay Ray Chuan
Message 14 of 41 in “Fix to push --progress. The --progress flag was not being passed into tranport.c from send-pack.h, making the --progress flag unusable”
  1. Fix to push --progress. The --progress flag was not being passed into tranport.c from send-pack.h, making the --progress flag unusableChase Brammer, Oct 12, 2010
  2. Jonathan NiederOct 12, 2010
  3. Junio C HamanoOct 13, 2010
  4. Jeff KingOct 13, 2010
  5. Tay Ray ChuanOct 13, 2010
  6. 0/3 fix push --progress over file://, git://, etc.Tay Ray Chuan, Oct 13, 2010
  7. 1/3 t5523-push-upstream: add function to ensure fresh upstream repoTay Ray Chuan, Oct 13, 2010
  8. Jonathan NiederOct 13, 2010
  9. 2/3 t5523-push-upstream: test progress messagesTay Ray Chuan, Oct 13, 2010
  10. 3/3 push: pass --progress down to git-pack-objectsTay Ray Chuan, Oct 13, 2010
  11. Tay Ray ChuanOct 14, 2010
  12. Jeff KingOct 14, 2010
  13. Tay Ray ChuanOct 13, 2010
  14. 0/8 fix push --progress over file://, git://, etc.Tay Ray Chuan, Oct 16, 2010
  15. 1/8 tests: factor out terminal handling from t7006Tay Ray Chuan, Oct 16, 2010
  16. 2/8 tests: test terminal output to both stdout and stderrTay Ray Chuan, Oct 16, 2010
  17. 3/8 test-lib: allow test code to check the list of declared prerequisitesTay Ray Chuan, Oct 16, 2010
  18. 4/8 test_terminal: catch use without TTY prerequisiteTay Ray Chuan, Oct 16, 2010
  19. 5/8 test_terminal: give priority to test-terminal.perl usageTay Ray Chuan, Oct 16, 2010
  20. 6/8 t5523-push-upstream: add function to ensure fresh upstream repoTay Ray Chuan, Oct 16, 2010
  21. 7/8 t5523-push-upstream: test progress messagesTay Ray Chuan, Oct 16, 2010
  22. 8/8 push: pass --progress down to git-pack-objectsTay Ray Chuan, Oct 16, 2010
  23. Jonathan NiederOct 17, 2010
  24. Jonathan NiederOct 17, 2010
  25. Jeff KingOct 22, 2010
  26. Jonathan NiederOct 17, 2010
  27. 0/3 more push progress testsJeff King, Oct 14, 2010
  28. 1/3 tests: factor out terminal handling from t7006Jeff King, Oct 14, 2010
  29. Jonathan NiederOct 14, 2010
  30. 2/3 tests: test terminal output to both stdout and stderrJeff King, Oct 14, 2010
  31. Jonathan NiederOct 14, 2010
  32. 3/3 t5523: test push progress output to ttyJeff King, Oct 14, 2010
  33. Jonathan NiederOct 14, 2010
  34. Jeff KingOct 14, 2010
  35. 0/2 test_terminal: check that TTY prerequisite is declaredJonathan Nieder, Oct 14, 2010
  36. 1/2 test-lib: allow test code to check the list of declared prerequisitesJonathan Nieder, Oct 14, 2010
  37. Ævar Arnfjörð BjarmasonOct 15, 2010
  38. Jonathan NiederOct 15, 2010
  39. 2/2 test_terminal: catch use without TTY prerequisiteJonathan Nieder, Oct 14, 2010
  40. Jeff KingOct 15, 2010
  41. Tay Ray ChuanOct 15, 2010

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.