On Tue, Feb 17, 2015 at 02:16:30PM -0800, Junio C Hamano wrote:
Do you want to resurrect that @{publish} stuff? I think it had
sensible semantics, and I do not think we mind keeping the
push_default configuration to be read from the default_config
codepath.
I'll take a look at it and see if
On Wed, Feb 18, 2015 at 11:08:34AM -0800, Junio C Hamano wrote:
Jeff King p...@peff.net writes:
+ if (!strcmp(k, push.followtags)) {
+ if (git_config_bool(k, v))
+ *flags |= TRANSPORT_PUSH_FOLLOW_TAGS;
+ else
+ *flags =
Jeff King p...@peff.net writes:
On Wed, Feb 18, 2015 at 11:08:34AM -0800, Junio C Hamano wrote:
...
Not very strongly either way. Seeing the above does not bother me
too much, but I do not know how I would feel when I start seeing
val = git_config_book(k, v);
flip_bool(val,
On Wed, Feb 18, 2015 at 11:50:20AM -0800, Junio C Hamano wrote:
Jeff King p...@peff.net writes:
On Wed, Feb 18, 2015 at 11:08:34AM -0800, Junio C Hamano wrote:
...
Not very strongly either way. Seeing the above does not bother me
too much, but I do not know how I would feel when I
Jeff King p...@peff.net writes:
If we wanted to implement @{push} (or @{publish}) to mean the
tracking ref of the remote ref you would push to if you ran git-push,
then this is a step in the wrong direction.
Is that because push_default variable needs to be looked at from
sha1_name.c when
On Tue, Feb 17, 2015 at 09:45:07AM -0800, Junio C Hamano wrote:
Jeff King p...@peff.net writes:
If we wanted to implement @{push} (or @{publish}) to mean the
tracking ref of the remote ref you would push to if you ran git-push,
then this is a step in the wrong direction.
Is that
Jeff King p...@peff.net writes:
So we probably would do something similar to @{push} side, which
would mean that push_default variable and the logic needs to be
visible to remote.c if we want to have the helper that is similar to
set_merge() that is used from branch_get() to support
On Mon, Feb 16, 2015 at 12:47:54AM -0500, Jeff King wrote:
When the push_default flag was originally added, it was
made globally visible to all code. This might have been
useful if other commands or library calls ended up depending
on it, but as it turns out, only builtin/push.c cares.
On Sun, Feb 15, 2015 at 9:47 PM, Jeff King p...@peff.net wrote:
When the push_default flag was originally added, it was
made globally visible to all code. This might have been
useful if other commands or library calls ended up depending
on it, but as it turns out, only builtin/push.c cares.
When the push_default flag was originally added, it was
made globally visible to all code. This might have been
useful if other commands or library calls ended up depending
on it, but as it turns out, only builtin/push.c cares.
Let's make it a static variable in builtin/push.c. Since it
is no
10 matches
Mail list logo