Submodules mirrors or .gitmodules change #37593

New Issue

Dalai Felinto · 2013-11-23T16:32:41+01:00

Dalai Felinto commented

2013-11-23 16:32:41 +01:00

How about the submodules, are they to be mirrored as well?
The way things are setup now (see .gitmodules) one using github needs to clone not only blender.git but all the other submodules (see my bf-committers email )

Alternatively we could update .gitmodules to use absolute paths instead of relative ones. The problem with absolute paths is that the protocol (git/http/https) get saved as well. Another option is to include .gitmodules in .gitignore, but that sounds as a bad practise.

The current workaround is to run the following commands before running git submodule update --init --recursive --remote:

git config submodule.scons.url git://git.blender.org/scons.git
git config submodule.release/scripts/addons_contrib.url git://git.blender.org/blender-addons-contrib.git
git config submodule.release/scripts/addons.url git://git.blender.org/blender-addons.git
git config submodule.release/datafiles/locale.url git://git.blender.org/blender-translations.git

How about the submodules, are they to be mirrored as well? The way things are setup now (see .gitmodules) one using github needs to clone not only blender.git but all the other submodules (see my [bf-committers email ](http://lists.blender.org/pipermail/bf-committers/2013-November/042253.html)) Alternatively we could update .gitmodules to use absolute paths instead of relative ones. The problem with absolute paths is that the protocol (git/http/https) get saved as well. Another option is to include .gitmodules in .gitignore, but that sounds as a bad practise. The current workaround is to run the following commands before running **git submodule update --init --recursive --remote**: ``` git config submodule.scons.url git://git.blender.org/scons.git git config submodule.release/scripts/addons_contrib.url git://git.blender.org/blender-addons-contrib.git git config submodule.release/scripts/addons.url git://git.blender.org/blender-addons.git git config submodule.release/datafiles/locale.url git://git.blender.org/blender-translations.git ```

Dalai Felinto commented

2013-11-23 16:32:41 +01:00

Changed status to: 'Open'

Dalai Felinto commented

2013-11-23 16:32:41 +01:00

Added subscribers: @brecht, @ideasman42, @Sergey, @dfelinto

Julien Rivaud commented

2013-11-24 16:19:43 +01:00

Added subscriber: @FrnchFrgg

Julien Rivaud commented

2013-11-24 16:19:43 +01:00

The fact that the protocol is saved with the full path is not a problem: you can use the url.xxxxx.insteadOf mechanism to replace the protocols. At the simplest level:
git config url.git@git.blender.org:.insteadOf git://git.blender.org
git config url.git@git.blender.org:.insteadOf http://git.blender.org

will make all fetches (pulls) or pushes go through the ssh+git protocol instead of the read-only ones, for git.blender.org.

Also, urls in .gitmodules are only used to populate .git/config at submodule --init time, you can change the latter values to whatever you want without any problem (and that avoids the need to somehow ignore changes in the .gitmodules files)

The fact that the protocol is saved with the full path is not a problem: you can use the url.xxxxx.insteadOf mechanism to replace the protocols. At the simplest level: git config url.git@git.blender.org:.insteadOf git://git.blender.org git config url.git@git.blender.org:.insteadOf http://git.blender.org will make all fetches (pulls) or pushes go through the ssh+git protocol instead of the read-only ones, for git.blender.org. Also, urls in .gitmodules are only used to populate .git/config at submodule --init time, you can change the latter values to whatever you want without any problem (and that avoids the need to somehow ignore changes in the .gitmodules files)

Dalai Felinto commented

2013-11-25 14:15:13 +01:00

@FrnchFrgg what is your suggestion to the official fix/solution?

I updated the workaround in the description as per your suggestion. If that's an acceptable workaround I'm fine. I just think that official mirrors should 'just work', or at least we should move those instructions to the wiki

@FrnchFrgg what is your suggestion to the official fix/solution? I updated the workaround in the description as per your suggestion. If that's an acceptable workaround I'm fine. I just think that official mirrors should 'just work', or at least we should move those instructions to the wiki

Sergey Sharybin was assigned by Aaron Carlisle

2016-09-04 20:03:00 +02:00

Aaron Carlisle commented

2016-09-04 20:25:50 +02:00

Added subscriber: @Blendify

Aaron Carlisle commented

2016-09-04 20:25:50 +02:00

@brecht I guess this can be closed too?

Brecht Van Lommel commented

2016-09-04 20:26:17 +02:00

Changed status from 'Open' to: 'Archived'

Brecht Van Lommel closed this issue

2016-09-04 20:26:17 +02:00

Sign in to join this conversation.

No Milestone

No project

No Assignees

4 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: infrastructure/blender-org#37593

Download

What's New

Blender Studio

Manual

Developers Blog

Documentation

Benchmark

Blender Conference

Development Fund

One-time Donations

Submodules mirrors or .gitmodules change #37593