[PATCH] submodule--helper: set alternateLocation for cloned submodules

2016-12-11 Thread vi0oss
From: "Vitaly \"_Vi\" Shukela" In 31224cbdc7 (clone: recursive and reference option triggers submodule alternates, 2016-08-17) a mechanism was added to have submodules referenced. It did not address _nested_ submodules, however. This patch makes all not just the root

Re: [PATCH] submodule--helper: set alternateLocation for cloned submodules

2016-12-10 Thread vi0oss
On 12/08/2016 04:38 AM, vi0...@gmail.com wrote: Third review: missing && in test fixed. Shall something more be done about this or just wait until the patch gets reviewed and integrated?

Re: [PATCH] submodule--helper: set alternateLocation for cloned submodules

2016-12-08 Thread vi0oss
On 12/08/2016 08:46 PM, Jeff King wrote: On Wed, Dec 07, 2016 at 05:22:30PM -0800, Stefan Beller wrote: On Wed, Dec 7, 2016 at 4:39 PM, wrote: Previously test contained errorneous test_must_fail, which was masked by missing &&. I wonder if we could make

[PATCH] submodule--helper: set alternateLocation for cloned submodules

2016-12-07 Thread vi0oss
From: "Vitaly \"_Vi\" Shukela" In 31224cbdc7 (clone: recursive and reference option triggers submodule alternates, 2016-08-17) a mechanism was added to have submodules referenced. It did not address _nested_ submodules, however. This patch makes all not just the root

[PATCH] submodule--helper: set alternateLocation for cloned submodules

2016-12-07 Thread vi0oss
From: "Vitaly \"_Vi\" Shukela" In 31224cbdc7 (clone: recursive and reference option triggers submodule alternates, 2016-08-17) a mechanism was added to have submodules referenced. It did not address _nested_ submodules, however. This patch makes all not just the root

[PATCH 2/2] mailmap: Update my e-mail address

2016-12-07 Thread vi0oss
From: "Vitaly \"_Vi\" Shukela" Signed-off-by: Vitaly "_Vi" Shukela --- .mailmap | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.mailmap b/.mailmap index 9cc33e9..b7ae81a 100644 --- a/.mailmap +++ b/.mailmap @@ -246,7 +246,7 @@ Uwe

[PATCH 1/2] submodule--helper: set alternateLocation for cloned submodules

2016-12-07 Thread vi0oss
From: "Vitaly \"_Vi\" Shukela" In 31224cbdc7 (clone: recursive and reference option triggers submodule alternates, 2016-08-17) a mechanism was added to have submodules referenced. It did not address _nested_ submodules, however. This patch makes all not just the root

Re: [PATCH] submodule--helper: set alternateLocation for cloned submodules

2016-12-07 Thread vi0oss
On 12/07/2016 11:09 PM, Stefan Beller wrote: As submodule's alternate target does not end in .git/objects (rather .git/modules/qq/objects), this alternate target path restriction for in add_possible_reference_from_superproject relates from "*.git/objects" to just */objects". I wonder if

[PATCH] submodule--helper: set alternateLocation for cloned submodules

2016-12-07 Thread vi0oss
From: Vitaly _Vi Shukela Git v2.11 introduced "git clone --recursive --referece ...", but it didn't put the alternates for _nested_ submodules. This patch makes all not just the root repository, but also all submodules (recursively) have submodule.alternateLocation and

Proposal to add a shortcut for --force-with-lease

2016-08-20 Thread vi0oss
"--force" have currently two shortcuts: "-f" and "+", hence more ergonomic. But I expect it's better for users to use "--force-with-lease" by default for overriding remote branches (e.g. cleaning up a pull request), as it rarely fails in normal situations. I propose adding some shortcut for

[PATCH] git-gui: Fix semi-working shortcuts for unstage and revert

2012-08-16 Thread vi0oss
From: Vitaly _Vi Shukela vi0...@gmail.com Make Ctrl+U for unstaging and Ctrl+J for reverting selection behave more like Ctrl+T for adding. They were working only when one area was focused (diff or commit message), now they should work everywhere. Signed-off-by: Vitaly _Vi Shukela

[PATCH] git-gui: Fix semi-working shortcuts for unstage and revert

2012-08-12 Thread vi0oss
From: Vitaly _Vi Shukela vi0...@gmail.com Make Ctrl+U for unstaging and Ctrl+J for reverting selection behave more like Ctrl+T for adding. They were working only when one area was focused (diff or commit message), now they should work everywhere. Signed-off-by: Vitaly _Vi Shukela