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

Remove gdx-ai from core repo #2312

Merged
merged 1 commit into from Sep 21, 2014
Merged

Remove gdx-ai from core repo #2312

merged 1 commit into from Sep 21, 2014

Conversation

@Tom-Ski
Copy link
Member

Tom-Ski commented Sep 8, 2014

  • Remove from parent pom.
  • Remove from build
  • Removed tests
  • Removed code
davebaol added a commit that referenced this pull request Sep 21, 2014
Remove gdx-ai from core repo
@davebaol davebaol merged commit a7d55f2 into libgdx:master Sep 21, 2014
@badlogic
Copy link
Member

badlogic commented Sep 21, 2014

Hi guys, could you not remove modules without telling me about it? This influences the build among other things. Thanks.

@MobiDevelop
Copy link
Member

MobiDevelop commented Sep 21, 2014

In his defense, I told him he could merge if his new repo was ready because
people are/were having problems finding the right snapshots.

Blame me if it messed something up.

@NathanSweet
Copy link
Member

NathanSweet commented Sep 21, 2014

I blame Nex.

On Sun, Sep 21, 2014 at 7:45 PM, Justin Shapcott notifications@github.com
wrote:

In his defense, I told him he could merge if his new repo was ready because
people are/were having problems finding the right snapshots.

Blame me if it messed something up.


Reply to this email directly or view it on GitHub
#2312 (comment).

@badlogic
Copy link
Member

badlogic commented Sep 21, 2014

We had a plan to push out 1.3.2 with the old gdx-ai, then switch over to the new repo.

@badlogic
Copy link
Member

badlogic commented Sep 21, 2014

The reason there'sno release yet is related to RoboVM which has issues on iOS 8. Once that's fixed i can do a release.

@MobiDevelop
Copy link
Member

MobiDevelop commented Sep 21, 2014

Ah, I guess that plan was not shared with everyone as I had no clue that
was the plan. For what it's worth, gdx-ai was never in the setup so I guess
the impact is somewhat low.

@Tom-Ski
Copy link
Member Author

Tom-Ski commented Sep 21, 2014

Shouldn't our next release be 1.4.0 due to API breakages? Given the circumstances, the least painful route could be to release 1.4.0 without ai in the core repo, bump the ai repo to 1.4.0 and release ai. From then on the release cycles can be completely independent.

@badlogic
Copy link
Member

badlogic commented Sep 21, 2014

Right, communication. We do what Tomski does as soon as the RoboVM issues
are resolved.
On Sep 21, 2014 8:42 PM, "Tomski" notifications@github.com wrote:

Shouldn't our next release be 1.4.0 due to API breakages? Given the
circumstances, the least painful route could be to release 1.4.0 without ai
in the core repo, bump the ai repo to 1.4.0 and release ai. From then on
the release cycles can be completely independent.


Reply to this email directly or view it on GitHub
#2312 (comment).

@davebaol
Copy link
Member

davebaol commented Sep 21, 2014

We had a plan to push out 1.3.2 with the old gdx-ai, then switch over to the new repo.

Sorry for the mess, I think I got confused by your comments below where you wrote that the next libgdx release will not include steering:

My bad :(

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

Successfully merging this pull request may close these issues.

None yet

5 participants
You can’t perform that action at this time.