GitPlugin: Support for submodules

[Peter Suter]

Created as part of the move of GitPlugin. Tickets originally reported for ​th:GitPlugin: ​th:#8880, ​th:#3842

Quoting ​mresnick:

Is anyone working on submodule support in the source browser ?

From the git_fs.py file:

pass # FIXME: this is a workaround for missing git submodule support in the plugin

Particularly for TRAC 0.12 which supports multiple repositories it should be possible to switch context to display submodule files.

Quoting ​anonymous:

Error[1] apears when change contains only submodule version update. Same error can be reproduced when change contains information about merge branch into master.

[1] TypeError: unsupported operand type(s) for +: 'NoneType' and 'NoneType'

[anonymous]

Is this going to get implemented at some point?

[Christian Boos]

Well, we can't do much until someone contributes a repository icon ;-)

[Peter Suter]

Quoting ​anonymous:

Error[1] apears when change contains only submodule version update. Same error can be reproduced when change contains information about merge branch into master.

[1] TypeError: unsupported operand type(s) for +: 'NoneType' and 'NoneType'

This part is a duplicate of #10162.

[Peter Suter]

But as comment:3:ticket:10162 suggests, maybe we can for now treat subrepositories as directories. Changing GitRepository.get_changes()​ and GitChangeset.get_changes()​ from everything except directories are files to the reverse perhaps?

kind = Node.DIRECTORY
if mode2.startswith('10') or mode1.startswith('10'):
    kind = Node.FILE

(untested)

[Jun Omae]

#11183 was closed as duplicate.

[Lele Gaifax]

​here is another possible workaround to the problem.

[Jun Omae]

Related to ​th:#11732, ​th:r13908 for ​th:TracPygit2Plugin and #6474.

[Ryan J Ollos]

Moved ticket assigned to next-stable-1.0.x since maintenance of 1.0.x is coming to a close. Please move the ticket back if it's critical to fix on 1.0.x.

[Jun Omae]

I got TypeErrors under production environment. At least, I think we should prevent the TypeErrors even if #10162 is not resolved.

[Jun Omae]

Proposed changes in [5148072ee/jomae.git] (jomae.git@t10603).

[Jun Omae]

Reconsidering, submodule'd entry should be a directory. In proposed changes, that would be a file.

$ git submodule add -b 1.0-stable --name trac git@github.com:edgewall/trac.git trac
Cloning into 'trac'...
remote: Counting objects: 96491, done.
remote: Compressing objects: 100% (44/44), done.
remote: Total 96491 (delta 18), reused 17 (delta 8), pack-reused 96439
Receiving objects: 100% (96491/96491), 35.90 MiB | 54.00 KiB/s, done.
Resolving deltas: 100% (76657/76657), done.
Checking connectivity... done.
$ git commit -a -m 'add submodule'
[master (root-commit) a35122f40] add submodule
 2 files changed, 5 insertions(+)
 create mode 100644 .gitmodules
 create mode 160000 trac
$ git archive --format=tar --prefix=./ HEAD | tar tvf -
drwxrwxr-x root/root         0 2017-12-11 12:05 ./
-rw-rw-r-- root/root        93 2017-12-11 12:05 ./.gitmodules
drwxrwxr-x root/root         0 2017-12-11 12:05 ./trac/           # <== directory

I'll re-propose changes.

[Jun Omae]

Proposed changes in [ab9e4aaa8/jomae.git] (jomae.git@t10603.1), again.

In the changes, GitNode.kind for submodule'd entry is Node.DIRECTORY and referenced sha in the submodule is stored in commit of GitNode.properties().

>>> from trac.env import Environment
>>> from trac.versioncontrol.api import RepositoryManager
>>> env = Environment('/dev/shm/tracenv')
>>> rm = RepositoryManager(env)
>>> repos = rm.get_repository('')
>>> node = repos.get_node('/sub')
>>> node.kind
'dir'
>>> node.get_properties()
{'commit': 'c392cf2e6a537dc3efe797576efad365a0fbeade', 'mode': '160000'}
>>> node.get_content() is None
True
>>> node.get_content_length() is None
True

[Jun Omae]

Replying to Jun Omae:

Proposed changes in [ab9e4aaa8/jomae.git] (jomae.git@t10603.1), again.

Revised the branch. Fixed GitNode.get_entries() returns a list included itself.

[Jun Omae]

Committed in [16409] and merged in [16410-16411].