push.default???

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi all, I just upgraded to git 1.6.3 and found this new little gem called push.default...

Now, having gone through an attempt of proposing a different semantics for "git push", this stroke me as total nonsense, because now we have two totally incompatible ways to specify push refspecs.

A sensible implementation would have been something like this:

1) Also in 1.6.3, invent a special refspec for "tracking", something like "HEAD>" (of course this is not a special case; "refs/heads/*>" would also work, yadda yadda)

2) Also in 1.6.3, add a "--push={current,tracking,matching,mirror}" option to "git remote add" that would set up a push refspec without the need to actually know refspec syntax. (--mirror would become just a synonym for --push=mirror).

3) Possibly, in 1.6.3 make "git clone" add a "push = :" line for the origin branch. This was actually suggested in a patch by myself.

4) in 1.6.4 or 1.7.0, make "git push" fail outright if there is no push line, with text suggesting

  For remotes that you will create in the future, please use the
  `--push' argument to `git remote add'.  For existing remotes,
  you can use the following command to obtain the same behavior as
  git 1.6.3:

    git config --add remote.origin.push :

  For alternative configurations, please look at the release notes
  for git 1.6.4.

so that it's a quick cut'n'paste into the shell to fix this (though once per repository).


I know it's my fault that I did not follow the development of git last March, but I could not help ranting that it is extremely wrong to specify what to push without a refspec (in the configuration, not in the command line -- the cmdlines can always have more "porcelain" attached to them).

(1) and (2) in particular can still be straightened, and (4) too maybe. I can work on the implementation if we agree on the details.

Paolo

--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]