Re: [PATCH] Documentation/gitmodules: Only 'update' and 'url' are required

2014-01-04 Thread Heiko Voigt
On Fri, Jan 03, 2014 at 10:31:22AM -0800, W. Trevor King wrote:
 From: W. Trevor King wk...@tremily.us
 
 Before this commit, all the settings fell under the initial Each
 submodule section also contains the following required keys:.  The
 example shows sections with just 'update' and 'url' entries, but we

You mean:

...with just 'path' and 'url' entries...

?

Other than that, the patch looks good to me.

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


Re: [PATCH] Documentation/gitmodules: Only 'update' and 'url' are required

2014-01-04 Thread W. Trevor King
On Sat, Jan 04, 2014 at 11:17:54PM +0100, Heiko Voigt wrote:
 On Fri, Jan 03, 2014 at 10:31:22AM -0800, W. Trevor King wrote:
  Before this commit, all the settings fell under the initial Each
  submodule section also contains the following required keys:.  The
  example shows sections with just 'update' and 'url' entries, but we
 
 You mean:
 
   ...with just 'path' and 'url' entries...

Oops, yes.  And in the summary/subject as well.  I can submit a v2 if
folks want, or just deputize Junio to fix it if/when this is applied.

Cheers,
Trevor

-- 
This email may be signed or encrypted with GnuPG (http://www.gnupg.org).
For more information, see http://en.wikipedia.org/wiki/Pretty_Good_Privacy


signature.asc
Description: OpenPGP digital signature


[PATCH] Documentation/gitmodules: Only 'update' and 'url' are required

2014-01-03 Thread W. Trevor King
From: W. Trevor King wk...@tremily.us

Before this commit, all the settings fell under the initial Each
submodule section also contains the following required keys:.  The
example shows sections with just 'update' and 'url' entries, but we
should still make the required/optional distinction explicit for the
descriptions.

Signed-off-by: W. Trevor King wk...@tremily.us
---
 Documentation/gitmodules.txt | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/Documentation/gitmodules.txt b/Documentation/gitmodules.txt
index f7be93f..347a9f7 100644
--- a/Documentation/gitmodules.txt
+++ b/Documentation/gitmodules.txt
@@ -35,6 +35,8 @@ submodule.name.url::
linkgit:git-clone[1] or (if it begins with ./ or ../) a location
relative to the superproject's origin repository.
 
+In addition, there are a number of optional keys:
+
 submodule.name.update::
Defines what to do when the submodule is updated by the superproject.
If 'checkout' (the default), the new commit specified in the
-- 
1.8.5.2.gaa5d535.dirty

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