Subject: [PATCH for submodule ideas wiki] `git commit` to warn about broken submodule states configuration

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

 



---

  Hi Jens,
 
  I cannot find the edit button in the wiki[1] any more.
  Please apply this patch documenting another idea.
  
  Thanks,
  Stefan
  
  [1] https://github.com/jlehmann/git-submod-enhancements/wiki

 Home.textile | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/Home.textile b/Home.textile
index 152ad04..3b00496 100644
--- a/Home.textile
+++ b/Home.textile
@@ -47,6 +47,17 @@ h2. Submodule related bugs to fix
 * E.g. @git log <submodule>/@ and @git diff <rev> <submodule>/@ do not produce any output while dropping the '/' at the end makes them work as expected.
 * @git add@ does not honour the submodule.&lt;name&gt;.ignore settings
 * @git submodule add@ messes up paths from nested submodules, see ("thread on mailing list":http://thread.gmane.org/gmane.comp.version-control.git/288130)
+* @git commit@ can be cautious about submodule related commits. That is if a commit contains a change to the .gitmodules file, make sure that the corresponding gitlink is present (either already in the tree or newly added). IIUC Jonathan correctly a submodule which is configured in .gitmodules, but not present as a gitlink is considered broken, which we may want to avoid in the first place. When implementing this, we need to take care of historical mistakes being handled correctly as well as a flag to commit, which overwrites the potential warning. The gitmodules file unlike other in-tree configuration files (.gitignore, .gitattributes, .mailmap) have a 1:1 mapping so the existence of the gitlink is strongly expected. (Counter example with .gitignore: Ignoring *.exe doesn't expect any .exe file to be there). I imagine this to roughly work as:
+
+bc.     $ git add .gitmodules
+    $ git commit -m "add submodule to .gitmodules, but not as gitlink"
+      Warning: Not committing as this would break submodule consistency. Use
+      --force-submodules to commit the broken submodule state.
+    $ echo $?
+      1 # fail
+    $ git commit --force-submodules -m "add submodule to .gitmodules, but not as gitlink"
+    $ echo $?
+      0 # Success
 
 *[[Ideas]]*
 
-- 
2.7.0.rc0.42.g8e9204f.dirty

--
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]