Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dependencies #2318

Merged
merged 5 commits into from Apr 3, 2014

Conversation

Projects
None yet
3 participants
@tomspilman
Copy link
Member

commented Apr 2, 2014

This pull request replaces the old dependencies repo kungfubanana/MonoGame-Dependencies with a new Mono-Game/MonoGame.Dependencies repo. This does a few things:

  • Moves the dependencies to a repo the team controls.
  • Makes the git download smaller as we threw away unneeded old history.
  • Renames the folder from Libs to Dependencies which makes more sense.

Downstream forks will only need to do a normal submodule init/update to get the new folder. To remove the old files you will need to manually delete the ThirdParty\Libs folder. Unfortunately git doesn't make this any easier.

@mgbot

This comment has been minimized.

Copy link
Member

commented Apr 2, 2014

Test FAILed.
Refer to this link for build results: http://build.monogame.net/job/PullRequestTester/669/

@tomspilman

This comment has been minimized.

Copy link
Member Author

commented Apr 2, 2014

@mgbot test

@mgbot

This comment has been minimized.

Copy link
Member

commented Apr 2, 2014

Test FAILed.
Refer to this link for build results: http://build.monogame.net/job/PullRequestTester/670/

@tomspilman

This comment has been minimized.

Copy link
Member Author

commented Apr 2, 2014

Gah... I have to cleanup the mac build slave.

@tomspilman

This comment has been minimized.

Copy link
Member Author

commented Apr 3, 2014

@mgbot test

@mgbot

This comment has been minimized.

Copy link
Member

commented Apr 3, 2014

Test PASSed.
Refer to this link for build results: http://build.monogame.net/job/PullRequestTester/671/

@tomspilman

This comment has been minimized.

Copy link
Member Author

commented Apr 3, 2014

Finally!

@KonajuGames

This comment has been minimized.

Copy link
Contributor

commented Apr 3, 2014

Merging.

KonajuGames added a commit that referenced this pull request Apr 3, 2014

@KonajuGames KonajuGames merged commit 2c8a098 into develop Apr 3, 2014

1 check passed

default Merged build finished.
Details

@tomspilman tomspilman deleted the Dependencies branch Apr 3, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.