* push docs: inaccurate description of omitted destination?
@ 2019-05-13 3:47 Kyle Meyer
2019-05-13 9:18 ` Junio C Hamano
0 siblings, 1 reply; 2+ messages in thread
From: Kyle Meyer @ 2019-05-13 3:47 UTC (permalink / raw)
To: git
It seems to me that push's manpage contains a couple of inaccurate (or
at least confusingly incomplete) statements about omitting the
destination part of the refspec.
First, the refspec section of the manpage has this to say:
If `git push [<repository>]` without any `<refspec>` argument is set to
update some ref at the destination with `<src>` with
`remote.<repository>.push` configuration variable, `:<dst>` part can
be omitted--such a push will update a ref that `<src>` normally updates
without any `<refspec>` on the command line. Otherwise, missing
`:<dst>` means to update the same ref as the `<src>`.
Reading that, I'd think that, if I haven't configured
remote.<repository>.push in a way that involves <src>, omitting <dst>
will update the remote ref with the same name. But push.default is also
consulted before falling back entirely to using the remote ref with the
same name:
$ git rev-parse --symbolic-full-name HEAD
refs/heads/topic
$ git rev-parse --symbolic-full-name @{upstream}
refs/remotes/origin/master
$ git -c push.default=upstream push -n origin topic ;# unmentioned case
To ../rem
09cc638..58f7f72 topic -> master
$ git branch --unset-upstream topic
$ git -c push.default=upstream push -n origin topic ;# fallback case
To ../rem
* [new branch] topic -> topic
Second, the example section says
`git push origin master`::
Find a ref that matches `master` in the source repository
(most likely, it would find `refs/heads/master`), and update
the same ref (e.g. `refs/heads/master`) in `origin` repository
with it. If `master` did not exist remotely, it would be
created.
Perhaps I'm misreading that, but I'd interpret that as saying the remote
ref with the same name will always be updated, but that doesn't match
the refspec description of an omitted destination shown above or the
push.default-dependent behavior demonstrated in the example above.
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: push docs: inaccurate description of omitted destination?
2019-05-13 3:47 push docs: inaccurate description of omitted destination? Kyle Meyer
@ 2019-05-13 9:18 ` Junio C Hamano
0 siblings, 0 replies; 2+ messages in thread
From: Junio C Hamano @ 2019-05-13 9:18 UTC (permalink / raw)
To: Kyle Meyer; +Cc: git
Kyle Meyer <kyle@kyleam.com> writes:
> It seems to me that push's manpage contains a couple of inaccurate (or
> at least confusingly incomplete) statements about omitting the
> destination part of the refspec.
>
> First, the refspec section of the manpage has this to say:
>
> If `git push [<repository>]` without any `<refspec>` argument is set to
> update some ref at the destination with `<src>` with
> `remote.<repository>.push` configuration variable, `:<dst>` part can
> be omitted--such a push will update a ref that `<src>` normally updates
> without any `<refspec>` on the command line. Otherwise, missing
> `:<dst>` means to update the same ref as the `<src>`.
>
> Reading that, I'd think that, if I haven't configured
> remote.<repository>.push in a way that involves <src>, omitting <dst>
> will update the remote ref with the same name.
... if such a push would update the ref at the remote with the same
name, yes. Otherwise no. And that is determined by what push mode
is being used (e.g. are we using the matching mode? The current
mode? Something else?).
> But push.default is also
> consulted before falling back entirely to using the remote ref with the
> same name:
Yes. The push.default configuration participates in the selection
of the ref at the remote that gets updated, given a refspec element
with a missing :<dst>. And you do not have to write :<dst> to name
the ref that a refspec with missing :<dst> would update anyway.
> `git push origin master`::
> Find a ref that matches `master` in the source repository
> (most likely, it would find `refs/heads/master`), and update
> the same ref (e.g. `refs/heads/master`) in `origin` repository
> with it. If `master` did not exist remotely, it would be
> created.
>
> Perhaps I'm misreading that, but I'd interpret that as saying the remote
> ref with the same name will always be updated,...
That part is random listing of examples with vanilla configuration,
so depending on what setting you personally have, the behaviour may
be different.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2019-05-13 9:18 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-05-13 3:47 push docs: inaccurate description of omitted destination? Kyle Meyer
2019-05-13 9:18 ` Junio C Hamano
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).