Skip to content

Conversation

nickelsen
Copy link
Collaborator

This will be updated before merge with refs to master-branches when related PRs are merged in sc2reader, ggpyjobs, esdb and ggtracker components.

@nickelsen nickelsen changed the title Use real repos WIP: Use real repos Apr 13, 2016
@dsjoerg
Copy link
Member

dsjoerg commented Apr 15, 2016

Hi @nickelsen I believe this is ready to go, as I have taken in all the relevant code.

@nickelsen nickelsen changed the title WIP: Use real repos Use real repos Apr 16, 2016
@nickelsen
Copy link
Collaborator Author

Yes, this it ready to go - I have updated it to point to official repos everywhere!

@dsjoerg dsjoerg merged commit 1238aeb into master Apr 16, 2016
@dsjoerg
Copy link
Member

dsjoerg commented Apr 16, 2016

I burned >half an hour last night trying to figure out how to update submodule references in my own checkout. I found so many conflicting pieces of advice about how to do, I even stooped to reading man pages, nothing worked.

https://git-scm.com/docs/git-submodule
http://stackoverflow.com/questions/14404704/how-do-i-replace-a-git-submodule-with-another-repo
http://stackoverflow.com/questions/2155887/git-submodule-head-reference-is-not-a-tree-error

The struggle is real

@nickelsen nickelsen deleted the use-real-repos branch April 17, 2016 09:15
@nickelsen
Copy link
Collaborator Author

I know - it's crazy trying to wrap ones mind around submodules, branches and multiple remotes. I never worked with submodules before and this has blown my mind several times on so many levels.

But everything considered, I think it's pretty good now and I've at least learned something along the way. Ideally we'd set up something that monitors the master branch of the four repos in the stack and pushes an update of the references in this repo whenever they change. I'll have a look if we can get something free for open source somewhere.

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

Successfully merging this pull request may close these issues.

2 participants