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

Improve Compatibility with 1.12.2 #17

Open
wants to merge 5 commits into
base: 1.12.2
Choose a base branch
from

Conversation

CDAGaming
Copy link

@CDAGaming CDAGaming commented Jan 11, 2018

This PR Upgrades Dependencies to be more compatible with 1.12.2 as well as replaces most deprecation issues.

Edit: Renamed PR to Improved, as this PR isn't a requisite for 1.12.2 compatibility and merely improves upon it.

@liach
Copy link

liach commented Jan 11, 2018

you need to fix star imports and revert java back to 6

4 similar comments
@liach
Copy link

liach commented Jan 11, 2018

you need to fix star imports and revert java back to 6

@liach
Copy link

liach commented Jan 11, 2018

you need to fix star imports and revert java back to 6

@liach
Copy link

liach commented Jan 11, 2018

you need to fix star imports and revert java back to 6

@liach
Copy link

liach commented Jan 11, 2018

you need to fix star imports and revert java back to 6

@liach
Copy link

liach commented Jan 11, 2018

hmm, apparently fasthub client is seriously bugged.

@CDAGaming
Copy link
Author

@liach I'll Fix Star Imports in a little while, but unfortunately I will not be reverting Java. The reason of which is that Java 6 isn't even supported under 1.12.X. It was changed to prevent anyone who happened to use Java 6 with 1.12.2 (May make this same change to 1.12.0 and .1)

@liach
Copy link

liach commented Jan 11, 2018

actually you should bump not because of minecraft but because of gradle (requires j7)

@Mumfrey
Copy link
Owner

Mumfrey commented Jan 11, 2018

Please fix the following issues:

  • Changing english spelling to american spelling. Why? I'm english and spell things in my native tongue.
  • Removing public abstract from interface methods, my code style requires this, please put it back.
  • Gradle wrapper, I really despise gradle wrapper, don't include it.

@CDAGaming
Copy link
Author

Alright. I'll do so in a little while (Not near a PC ATM to do this until a few hours from now)

This Commit Upgrades Dependencies to be more compatible with 1.12.2 as well as replaces any deprecation issues.
@CDAGaming
Copy link
Author

CDAGaming commented Jan 11, 2018

Feel Free to Check this again now, as all of @Mumfrey s Points plus Starred Imports are now fixed.

Copy link

@LeafHacker LeafHacker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Couple minor gradle changes

build.gradle Show resolved Hide resolved
build.gradle Outdated Show resolved Hide resolved
@darthvader45
Copy link

Are there still issues with this pull? If not, seems like things have been fixed. Also, I'm creating a new issue regarding Pam's Harvestcraft and this, as it seems that liteloader has broken the HC right-click to harvest crops functionality. This is not favorable, as I prefer not to break my crops when harvesting.

@Mumfrey
Copy link
Owner

Mumfrey commented Jun 17, 2018

I need to have a look into it. I'm really not sure how anything in the installer has anything to do with "compatibility with 1.12.2" because I see no changes here that have anything to do with compatibility. It just seems like some dependencies were changed for no particular reason.

@CDAGaming CDAGaming changed the title Full Compatibility with 1.12.2 Improve Compatibility with 1.12.2 Jun 17, 2018
build.gradle Outdated Show resolved Hide resolved
[Fix] Revert to Java v1.6 for Installer
[Fix] Add dependsOn to createExe (It does work)
[Fix] Fixed a GString Reference in Installer FileDescription
[Change] createExe#jar now is a GString
[Change] project.verRelease is now 00, not -00 (Fixed Format in version to have the dash)
(Installer EXE is still outputted at build/launch4j now launch/createExe)
@CDAGaming
Copy link
Author

All Requested Changes are now Done!

Sorry it has taken so long, but this PR should now be good to go.

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.

5 participants