All of lore.kernel.org
 help / color / mirror / Atom feed
* Branch metainformation
@ 2006-04-20  7:39 Jakub Narebski
  2006-04-20  9:37 ` Johannes Schindelin
  0 siblings, 1 reply; 4+ messages in thread
From: Jakub Narebski @ 2006-04-20  7:39 UTC (permalink / raw)
  To: git

It seems that ever so often there is question posted on this list on which
the answer is to add '+' in 'pu' branch 'Pull:' line. I think that it is
not a good policy to have puller to remember which branches can be
fast-forwarded, and which needs to be merged. It should be an information
set and stored by person managing the branch (pullee), as some kind of
metainformation for the branch (like description and optional signature is
for tags).

Perhaps then there would be a place for short, one-line description of the
branch...

-- 
Jakub Narebski
Warsaw, Poland

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

* Re: Branch metainformation
  2006-04-20  7:39 Branch metainformation Jakub Narebski
@ 2006-04-20  9:37 ` Johannes Schindelin
  2006-04-20 10:18   ` Junio C Hamano
  0 siblings, 1 reply; 4+ messages in thread
From: Johannes Schindelin @ 2006-04-20  9:37 UTC (permalink / raw)
  To: Jakub Narebski; +Cc: git

Hi,

On Thu, 20 Apr 2006, Jakub Narebski wrote:

> It seems that ever so often there is question posted on this list on which
> the answer is to add '+' in 'pu' branch 'Pull:' line. I think that it is
> not a good policy to have puller to remember which branches can be
> fast-forwarded, and which needs to be merged. It should be an information
> set and stored by person managing the branch (pullee), as some kind of
> metainformation for the branch (like description and optional signature is
> for tags).
> 
> Perhaps then there would be a place for short, one-line description of the
> branch...

I have an alternative in my private repository: When git-pull/git-fetch 
are called with "-S <nick>", then the current URL is set as URL for that 
shorthand, and the rest is *added* as Pull line.

However, this all depends on my (rejected) patch to move the remotes 
information into the config file.

I have not backported the "-S" feature to the non-config remotes, 
since I 1) do not need it myself, and 2) am wary that it is not that easy 
to ensure data integrity by shell-scripting.

Ciao,
Dscho

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

* Re: Branch metainformation
  2006-04-20  9:37 ` Johannes Schindelin
@ 2006-04-20 10:18   ` Junio C Hamano
  2006-04-20 11:41     ` Johannes Schindelin
  0 siblings, 1 reply; 4+ messages in thread
From: Junio C Hamano @ 2006-04-20 10:18 UTC (permalink / raw)
  To: Johannes Schindelin; +Cc: git

Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:

> However, this all depends on my (rejected) patch to move the remotes 
> information into the config file.

You seem to keep saying rejected, but IIRC you did not finish it
IOW the patch was not ready to be merged.  I do not recall, for
example, reading the part that touches git-parse-remote or
git-fetch to read from the config, but it was a long time ago
and my recollection is vague.

While we are talking about per branch property, some issues
raised on the list (and #git) recently would be helped by a
convention (and perhaps some core side support) for per-branch
property.  Here is a short list.

 + When I am on branch X, I would want "git pull" to pull
   (i.e. fetch and merge) from repository Y, not always "origin".

 + When I am on branch X, I would want "git push" to push to
   repository Y (we do not even use "origin" as the default for
   push).

 + This branch is not to be rebased (you could do this using
   custom pre-rebase hook but having a standard "branch property"
   would make it easy for such a hook to decide.

 - Do not merge and base your work on this branch -- this is
   "view only" and unstable (e.g. "pu" in git.git).

If we were to do a remote to config reorganization (for that we
need a migration plan and a period that we support both), the
per-branch configuration should be designed to support at least
the commonly asked ones.

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

* Re: Branch metainformation
  2006-04-20 10:18   ` Junio C Hamano
@ 2006-04-20 11:41     ` Johannes Schindelin
  0 siblings, 0 replies; 4+ messages in thread
From: Johannes Schindelin @ 2006-04-20 11:41 UTC (permalink / raw)
  To: Junio C Hamano; +Cc: git

Hi,

On Thu, 20 Apr 2006, Junio C Hamano wrote:

> Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:
> 
> > However, this all depends on my (rejected) patch to move the remotes 
> > information into the config file.
> 
> You seem to keep saying rejected, but IIRC you did not finish it

I had another impression: people seemed less than enthusiastic about it. 
But maybe I misunderstood, and only the automatic rewriting part was 
disliked. I'll try to find time to clean the patch and resend it...

> While we are talking about per branch property, some issues
> raised on the list (and #git) recently would be helped by a
> convention (and perhaps some core side support) for per-branch
> property.  Here is a short list.
> 
>  + When I am on branch X, I would want "git pull" to pull
>    (i.e. fetch and merge) from repository Y, not always "origin".
> 
>  + When I am on branch X, I would want "git push" to push to
>    repository Y (we do not even use "origin" as the default for
>    push).
> 
>  + This branch is not to be rebased (you could do this using
>    custom pre-rebase hook but having a standard "branch property"
>    would make it easy for such a hook to decide.
> 
>  - Do not merge and base your work on this branch -- this is
>    "view only" and unstable (e.g. "pu" in git.git).
> 
> If we were to do a remote to config reorganization (for that we
> need a migration plan and a period that we support both), the
> per-branch configuration should be designed to support at least
> the commonly asked ones.

Hmm. ATM the syntax I use is

	[remote.company]
		url = company.com:gits/daproject
		pull = origin
		push = master

Some bits of your wishes could be written as

	[branch.pu]
		rebase = no
		pull = company

Ciao,
Dscho

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

end of thread, other threads:[~2006-04-20 11:41 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-04-20  7:39 Branch metainformation Jakub Narebski
2006-04-20  9:37 ` Johannes Schindelin
2006-04-20 10:18   ` Junio C Hamano
2006-04-20 11:41     ` Johannes Schindelin

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.