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

Updated gradle to 3.4.1 and fixed some bugs #3

Merged
merged 2 commits into from
Mar 14, 2017
Merged

Updated gradle to 3.4.1 and fixed some bugs #3

merged 2 commits into from
Mar 14, 2017

Conversation

ImMorpheus
Copy link
Contributor

@ImMorpheus ImMorpheus commented Mar 10, 2017

This issue (SpongeStart not generating IntelliJ task) was found while trying to fix a problem with spongeforge download

In the log I found
java.nio.file.AccessDeniedException: .idea\runConfigurations\StartForgeServer.xml

Normally configurations are stored in .idea\workspace.xml (unless a directory-based project format is used and the run/debug configs are shared).

Finally gradle was updated, hoping to fix #1 (it didn't work).

Edit:

SpongeStart ignoring forge version is now fixed.

@ImMorpheus ImMorpheus changed the title Updated gradle to 3.4.1 and fixed IntelliJ task not generating Updated gradle to 3.4.1 and fixed some bugs Mar 10, 2017
@thomas15v
Copy link
Contributor

thomas15v commented Mar 13, 2017

Looks neat, although I would have prefered that you made separated pull requests for each fix.

I still do not understand why the system would deny folder creation in .idea or creating the file in the first place. The reason I didn't want to modify the workspace.xml is that in case Idea updates the format, I really didn't want to corrupt their project.

So if I am correct the changeset for this PR is ?:

  • Fix forge version being ignored
  • Change run configuration creation to editing the workspace.xml in favor of creating new files.
  • Refactored some awful java code style violations by me (sorry about that).
  • Update the gradle version to 3.4.1

I don't see a reason why you changes wouldn't work, but atm I don't really have time to test them for myself. Are you 100% sure that everything works fine?

@ImMorpheus
Copy link
Contributor Author

ImMorpheus commented Mar 13, 2017

Correct. Sorry about not separating the fixes.

Refactored some awful java code style violations by me (sorry about that).

It was not intentional. Technically I planned to refactor the plugin with the next pull

Yes they work. Obviously I tested them before opening a pull request

@thomas15v thomas15v merged commit 0ba924b into Qixalite:master Mar 14, 2017
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.

Running StartVanillaServer in IntelliJ IDEA fails with gradle 3.2.1 and gradle 3.3.
2 participants