Re: [PATCH 0/6] Reroll of rr/triangular-push-fix

2013-06-24 Thread Ramkumar Ramachandra
Junio C Hamano wrote: > 'simple' is supposed to be an easy and safe default to help new > people. Your original patch to change its semantics for the central > workflow from the current 'make sure upstream is set and set to the > same name' to 'anything goes' is making the mode more dangerous than

Re: [PATCH 0/6] Reroll of rr/triangular-push-fix

2013-06-24 Thread Junio C Hamano
Ramkumar Ramachandra writes: > I hope (perhaps foolishly) to persuade you nevertheless. I fear that > if this series solidifies before I get there, we'll be stuck with this > stupid erroring-out behavior forever. I do not think it is "stupid" at all. 'simple' is supposed to be an easy and safe

Re: [PATCH 0/6] Reroll of rr/triangular-push-fix

2013-06-24 Thread Ramkumar Ramachandra
Junio C Hamano wrote: > [PATCH 3/6] push: change `simple` to accommodate triangular workflows > > Sqaushed in the fix to keep the semantics of "simple" when used in > the centralized workflow the same as before. Yeah, I'm worried about this as I pointed out earlier. I don't like erroring ou

[PATCH 0/6] Reroll of rr/triangular-push-fix

2013-06-23 Thread Junio C Hamano
This is mostly Ram's triangular-push-fix series, but the fix-up commits I queued on top of it when the series was queued to 'pu' have been squashed in. [PATCH 1/6] t/t5528-push-default: remove redundant test_config lines No changes, as posted by Ram. [PATCH 2/6] config doc: rewrite push.de