git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* When to run git-update-server-info
@ 2007-02-13  4:49 Pavel Roskin
  2007-02-13  5:20 ` Junio C Hamano
  0 siblings, 1 reply; 2+ messages in thread
From: Pavel Roskin @ 2007-02-13  4:49 UTC (permalink / raw)
  To: GIT list

Hello!

It looks like that either the documentation is misleading or git is not
working as promised.

Documentation/repository-layout.txt says:

info/refs::
        This file is to help dumb transports to discover what
        refs are available in this repository.  Whenever you
        create/delete a new branch or a new tag, `git
        update-server-info` should be run to keep this file
        up-to-date if the repository is published for dumb
        transports.  The `git-receive-pack` command, which is
        run on a remote repository when you `git push` into it,
        runs `hooks/update` hook to help you achieve this.

My understanding of the above text is that git-update-server-info only
needs to be run if a new branch or tag is created, not on every
git-push.  Just because hooks/update is run on every push, it doesn't
mean that it needs to be done every time.

What happens in reality is that git-pull refuses to pull new revisions
from the repository if git-update-server-info is not run after a simple
git-push (from another repository, of course) that didn't create any
branch or tag.  However, git-clone would load all revisions.  I'm
accessing the repository over http.

The problem it, there is more than one way to get the branch revisions,
one is from info/refs and the other is from individual files under
refs/heads and refs/tags.  Using database terms, the git repository in
not normalized, so we get an update anomaly.

I understand that accessing one file could be faster over http, so I
tend to think that it's the documentation needs to be fixed.  If that's
the case, maybe hooks/update should be created executable by default?

Alternatively, git-update-server-info functionality could be embedded
into git-receive-pack to simplify server setup.

-- 
Regards,
Pavel Roskin

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: When to run git-update-server-info
  2007-02-13  4:49 When to run git-update-server-info Pavel Roskin
@ 2007-02-13  5:20 ` Junio C Hamano
  0 siblings, 0 replies; 2+ messages in thread
From: Junio C Hamano @ 2007-02-13  5:20 UTC (permalink / raw)
  To: Pavel Roskin; +Cc: GIT list

Pavel Roskin <proski@gnu.org> writes:

> Hello!
>
> It looks like that either the documentation is misleading or git is not
> working as promised.
>
> Documentation/repository-layout.txt says:
>
> info/refs::
>         This file is to help dumb transports to discover what
>         refs are available in this repository.  Whenever you
>         create/delete a new branch or a new tag, `git
>         update-server-info` should be run to keep this file
>         up-to-date if the repository is published for dumb
>         transports.  The `git-receive-pack` command, which is
>         run on a remote repository when you `git push` into it,
>         runs `hooks/update` hook to help you achieve this.
>
> My understanding of the above text is that git-update-server-info only
> needs to be run if a new branch or tag is created, not on every
> git-push.  Just because hooks/update is run on every push, it doesn't
> mean that it needs to be done every time.

I agree the above can be interpreted as such, and documentation
updates is needed.  The information in info/refs are used in
full, not just the list of ref names on the far right of each
line but also the object name at the beginning of each line.

> I understand that accessing one file could be faster over http, so I
> tend to think that it's the documentation needs to be fixed.  If that's
> the case, maybe hooks/update should be created executable by default?

The repository that serve things over http are _minorities_, so
I do not think that is a good idea.

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2007-02-13  5:20 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-02-13  4:49 When to run git-update-server-info Pavel Roskin
2007-02-13  5:20 ` Junio C Hamano

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 NNTP newsgroup(s).