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

Re: push.default: current vs upstream

From
Dmitry Potapov <dpotapov@gmail.com>
Date
Apr 6, 2012, 11:38 UTC
Message-ID
<CAHkcotjrVqvYnAV5U7gPngbW0saghAv8vZB3jh=dOKLPmYdJrQ@mail.gmail.com>
In-Reply-To
<vpqwr5uceis.fsf@bauges.imag.fr>

On Thu, Apr 5, 2012 at 8:46 PM, Matthieu Moy <Matthieu.Moy@grenoble-inp.fr> wrote:

>
> I can hardly imagine someone knowing what "git pull" does, and
> _surprised_ to see that "git push" sends commits to the same place.

It seems you assume that people use in a _centralized_ workflow. In this case, 'upstream' does largely the right thing, so no one will be surprised.

However, many of those who got used to a distributed workflow will find that surprising, because when they created a new branch that meant it to push as a _new_ branch. So other people could take a look, or they may need the maintainer ACK to push anything to 'master'. Also they may have a policy nothing should be fast-forwarded to master, but only to be merged with a merge commit.

And then it is more natural for people to think in terms of names that are immediately obvious to anyone, while 'upstream' behavior depends on the state that is not immediately obvious. You may start a new topic branch based on 'master' or some the latest release to have a more stable base. And 'upstream' will work differently in this case. If you know about 'tracking' then it may be obvious to you, but it is not so obvious to those who only start to use git for short time...

>
> And I still have my concern with real beginners: what advice would you
> give to a user whose "git push" is denied because of non-fast forward. I
> raised this concern already:
Don't use a central workflow, because it sucks :)

Seriously, why do you care about beginners who use a centralized workflow and not beginners who have to use with existing projects that use more or less distributed workflow, where pushing to 'master' is more likely to be the wrong thing to do than otherwise... And when push is denied, they may ask someone whether they are doing something wrong. In case when master is fast-forwarded silently, they are not likely to notice that they did something wrong, and the fact that happens only sometimes (depending on some "tracking" feature which they have not heard) is not very helpful.

Dmitry
Previous: Christopher TiwaldNext: demerphq
Message 141 of 152 in “[ANNOUNCE] Git 1.7.10-rc3”
  1. Junio C HamanoMar 28, 2012
  2. Jeff KingMar 29, 2012
  3. Junio C HamanoMar 29, 2012
  4. Jeff KingMar 29, 2012
  5. Junio C HamanoMar 30, 2012
  6. push.default: current vs upstreamJeff King, Mar 30, 2012
  7. Junio C HamanoMar 30, 2012
  8. Jeff KingMar 30, 2012
  9. Junio C HamanoMar 30, 2012
  10. Jeff KingMar 30, 2012
  11. Junio C HamanoMar 30, 2012
  12. Jeff KingMar 30, 2012
  13. Junio C HamanoMar 30, 2012
  14. Nathan GrayMar 31, 2012
  15. Seth RobertsonMar 31, 2012
  16. Junio C HamanoApr 1, 2012
  17. Nathan GrayApr 1, 2012
  18. Matthieu MoyApr 2, 2012
  19. Junio C HamanoApr 2, 2012
  20. Matthieu MoyApr 2, 2012
  21. Junio C HamanoApr 2, 2012
  22. Matthieu MoyApr 2, 2012
  23. Junio C HamanoApr 2, 2012
  24. Matthieu MoyApr 2, 2012
  25. Junio C HamanoApr 2, 2012
  26. demerphqApr 2, 2012
  27. Matthieu MoyApr 2, 2012
  28. demerphqApr 4, 2012
  29. Jeff KingApr 5, 2012
  30. Matthieu MoyApr 5, 2012
  31. Jeff KingApr 6, 2012
  32. Matthieu MoyApr 6, 2012
  33. Jeff KingApr 6, 2012
  34. Junio C HamanoApr 6, 2012
  35. Jehan BingApr 6, 2012
  36. Michael HaggertyApr 7, 2012
  37. Jeff KingApr 7, 2012
  38. Andrew SayersApr 7, 2012
  39. Jeff KingApr 12, 2012
  40. Junio C HamanoApr 12, 2012
  41. Andrew SayersApr 12, 2012
  42. Junio C HamanoApr 12, 2012
  43. Jeff KingApr 12, 2012
  44. Philip OakleyApr 12, 2012
  45. Junio C HamanoApr 13, 2012
  46. Andrew SayersApr 17, 2012
  47. Junio C HamanoApr 8, 2012
  48. Matthieu MoyApr 11, 2012
  49. Junio C HamanoApr 11, 2012
  50. Jeff KingApr 12, 2012
  51. Matthieu MoyApr 12, 2012
  52. Jeff KingApr 12, 2012
  53. Matthieu MoyApr 12, 2012
  54. Junio C HamanoApr 12, 2012
  55. Junio C HamanoApr 19, 2012
  56. Matthieu MoyApr 19, 2012
  57. Junio C HamanoApr 19, 2012
  58. 0/3 push.default upcomming changeMatthieu Moy, Apr 19, 2012
  59. 1/3 push: introduce new push.default mode "simple"Matthieu Moy, Apr 19, 2012
  60. Jeff KingApr 19, 2012
  61. Matthieu MoyApr 20, 2012
  62. 0/4 push.default upcomming changeMatthieu Moy, Apr 20, 2012
  63. 1/4 Documentation: explain push.default option a bit moreMatthieu Moy, Apr 20, 2012
  64. Jeff KingApr 20, 2012
  65. Junio C HamanoApr 20, 2012
  66. Michael HaggertyApr 21, 2012
  67. Junio C HamanoApr 21, 2012
  68. Michael HaggertyApr 21, 2012
  69. Junio C HamanoApr 21, 2012
  70. 2/4 push: introduce new push.default mode "simple"Matthieu Moy, Apr 20, 2012
  71. Jeff KingApr 20, 2012
  72. Zbigniew Jędrzejewski-SzmekApr 22, 2012
  73. Junio C HamanoApr 20, 2012
  74. Matthieu MoyApr 23, 2012
  75. 3/4 t5570: use explicit push refspecMatthieu Moy, Apr 20, 2012
  76. 4/4 push: start warning upcoming default change for push.defaultMatthieu Moy, Apr 20, 2012
  77. Jeff KingApr 20, 2012
  78. Matthieu MoyApr 22, 2012
  79. Junio C HamanoApr 23, 2012
  80. 0/7 push.default upcomming changeMatthieu Moy, Apr 23, 2012
  81. 1/7 Documentation: explain push.default option a bit moreMatthieu Moy, Apr 23, 2012
  82. Junio C HamanoApr 23, 2012
  83. Philip OakleyApr 23, 2012
  84. Junio C HamanoApr 23, 2012
  85. Philip OakleyApr 23, 2012
  86. 2/7 Undocument deprecated alias 'push.default=tracking'Matthieu Moy, Apr 23, 2012
  87. Junio C HamanoApr 23, 2012
  88. Ævar Arnfjörð BjarmasonJan 31, 2013
  89. Junio C HamanoJan 31, 2013
  90. Jonathan NiederJan 31, 2013
  91. Jonathan NiederJan 31, 2013
  92. Junio C HamanoJan 31, 2013
  93. Junio C HamanoJan 31, 2013
  94. Jonathan NiederJan 31, 2013
  95. Junio C HamanoJan 31, 2013
  96. Jonathan NiederJan 31, 2013
  97. Junio C HamanoJan 31, 2013
  98. Junio C HamanoJan 31, 2013
  99. Jonathan NiederJan 31, 2013
  100. Matthieu MoyJan 31, 2013
  101. Junio C HamanoJan 31, 2013
  102. Junio C HamanoJan 31, 2013
  103. Jonathan NiederJan 31, 2013
  104. Junio C HamanoFeb 1, 2013
  105. Jonathan NiederJan 31, 2013
  106. Philip OakleyJan 31, 2013
  107. 3/7 t5528-push-default.sh: add helper functionsMatthieu Moy, Apr 23, 2012
  108. Junio C HamanoApr 23, 2012
  109. Matthieu MoyApr 23, 2012
  110. Junio C HamanoApr 23, 2012
  111. Matthieu MoyApr 23, 2012
  112. Matthieu MoyApr 23, 2012
  113. Junio C HamanoApr 23, 2012
  114. Junio C HamanoApr 23, 2012
  115. 2/3 fixup! t5528-push-default.sh: add helper functionsJunio C Hamano, Apr 23, 2012
  116. 3/3 push: suggested updates to push configuration documentationJunio C Hamano, Apr 23, 2012
  117. 4/7 push: introduce new push.default mode "simple"Matthieu Moy, Apr 23, 2012
  118. Michael HaggertyApr 23, 2012
  119. Matthieu MoyApr 23, 2012
  120. Junio C HamanoApr 23, 2012
  121. Matthieu MoyApr 23, 2012
  122. 5/7 t5570: use explicit push refspecMatthieu Moy, Apr 23, 2012
  123. 6/7 push: document the future default change for push.default (matching -> simple)Matthieu Moy, Apr 23, 2012
  124. 7/7 push: start warning upcoming default change for push.defaultMatthieu Moy, Apr 23, 2012
  125. 0/7 push.default upcomming changeMatthieu Moy, Apr 24, 2012
  126. 1/7 Documentation: explain push.default option a bit moreMatthieu Moy, Apr 24, 2012
  127. 2/7 Undocument deprecated alias 'push.default=tracking'Matthieu Moy, Apr 24, 2012
  128. 3/7 t5528-push-default.sh: add helper functionsMatthieu Moy, Apr 24, 2012
  129. 4/7 push: introduce new push.default mode "simple"Matthieu Moy, Apr 24, 2012
  130. Junio C HamanoApr 25, 2012
  131. Matthieu MoyApr 25, 2012
  132. 5/7 t5570: use explicit push refspecMatthieu Moy, Apr 24, 2012
  133. 6/7 push: document the future default change for push.default (matching -> simple)Matthieu Moy, Apr 24, 2012
  134. 7/7 push: start warning upcoming default change for push.defaultMatthieu Moy, Apr 24, 2012
  135. Junio C HamanoApr 24, 2012
  136. 2/3 t5570: use explicit push refspecMatthieu Moy, Apr 19, 2012
  137. 3/3 push: start warning upcoming default change for push.defaultMatthieu Moy, Apr 19, 2012
  138. t5541: warning message is given even with --quietJunio C Hamano, Apr 26, 2012
  139. Matthieu MoyApr 26, 2012
  140. Give better 'pull' advice when pushing non-ff updates to current branchChristopher Tiwald, Apr 11, 2012
  141. Dmitry PotapovApr 6, 2012
  142. demerphqApr 6, 2012
  143. Dmitry PotapovApr 6, 2012
  144. demerphqApr 6, 2012
  145. Dmitry PotapovApr 6, 2012
  146. Junio C HamanoMar 30, 2012
  147. Jeff KingApr 3, 2012
  148. Jeff KingApr 3, 2012
  149. Junio C HamanoApr 3, 2012
  150. Junio C HamanoApr 3, 2012
  151. Jeff KingApr 5, 2012
  152. Felipe ContrerasApr 8, 2012

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.