Skip to content
This repository has been archived by the owner on Nov 4, 2023. It is now read-only.

Modman and git submodules #21

Open
devolonter opened this issue May 16, 2013 · 2 comments
Open

Modman and git submodules #21

devolonter opened this issue May 16, 2013 · 2 comments

Comments

@devolonter
Copy link
Contributor

Yesterday I tested new module manager. It seems, everything works perfectly with all modules and repositories. But I’ve noticed that manager runs git clone without --recursive option. So, if module uses git submodules, directory with submodule will be empty after installing.

I’ve tried to just add the --recursive option, but it didn’t work. The installation was not completed, and I had to kill the git process manually... Although it works fine if I run the same command directly from the command line

I would like to know if there any chance that the git submodules will be also supported. Or I should refuse from having them in my repositories?

Thanks!

@blitz-research
Copy link
Owner

Will have a look at this eventually - do you have any immediate need for it?

Git does seem to start sub-processes up in a 'weird' way, enough to mean I couldn't cheat and just use shell commands with redirected stdout to drive everything. Which was probably for the best as it meant I had to write brl.process - which is probably where the issue is.

@devolonter
Copy link
Contributor Author

No, it’s not urgent. At the moment I use submodules in flixel repository only for examples and 3rd party plugins. So, the module works fine without them.

Also, you can try to solve the issue with another command for getting submodules:

git submodule update --init after git clone

This command works not recursively.

P.S. Just for the information. Using submodules also require changes in the update method. You need to add run git submodule update after git merge

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants