ForgeEssentials / ForgeEssentials

Repository for ForgeEssentials 1.x
https://www.curseforge.com/minecraft/mc-mods/forge-essentials-74735
GNU General Public License v3.0
282 stars 144 forks source link

[1.8] Update to ForgeGradle 2 #1788

Closed liach closed 8 years ago

liach commented 8 years ago

On 1.8/develop build.gradle L22: classpath "net.minecraftforge.gradle:ForgeGradle:1.2-SNAPSHOT" to classpath "net.minecraftforge.gradle:ForgeGradle:2.0-SNAPSHOT"; L41: apply plugin: "forge" to apply plugin: "net.minecraftforge.gradle.forge" L69: version = "1.8-11.14.3.1450" to version = "1.8-11.14.3.1516" or some other later versions. L256 task sourceJar(type: Jar) { to sourceJar { maybe?

Between L293-L306 There were some issues because some function of old reobf task was removed. And the new task name is reobfJar. I have no way to resolve this issue for now.

L301,L302 Change addExtraSrgFile to addSecondarySrgFile.

I do not know a lot about gradle so I could not make a pr. Sorry.

liach commented 8 years ago

Btw you might pull 1.7 into 1.8 to update gradle wrapper.

yuuka-miya commented 8 years ago

Not going to update to FG2 in the short term - there may be some issue in deps.

Also, I'm waiting on 1.8.8, which depends on when the forge people fix fernflower.

liach commented 8 years ago

@luacs1998 1.8.8 has came out. Will you make a branch for it?

olee commented 8 years ago

I think we rather update FE 1.8 to 1.8.8 - I don't think we need to have one version at 1.8. At least the current 1.8 version is ok and can be used by people until they upgrade to 1.8.

yuuka-miya commented 8 years ago

That's what I plan to do - kill 1.8.0 and move to 1.8.8.

RlonRyan commented 8 years ago

While you are at it, why not:

yuuka-miya commented 8 years ago

1) no, it makes things difficult - I'd have to do a lot of fucking up, especially with the common repository.

2) not when sponge compat hits the mainstream builds, which won't be before the new year

3) PR

4) automation stuff? You mean the maven things? Those have to be in the buildscript

5) PR (change wrapperVersion in the buildfile, make sure you have local gradle installed, run gradle wrapper and commit the files it spits out in addition to the changed buildfile)

RlonRyan commented 8 years ago

1) Yeah, it is feat to setup. 2) I thought I saw something that said Mojang was bundling the latest java versions (i.e. 1.8) 3) Yup 4) I meant the auto-release stuff. Or I am not sure what I meant. 5) The wrapper task might as well be removed from buildscript. It's built in to Gradle, to the point that no updating of the buildscript is needed when updating the Gradle wrapper version.

RlonRyan commented 8 years ago

I guess what I am advocating for is redoing the buildscript from scratch... aren't I?

yuuka-miya commented 8 years ago

2) that's for clients only

4) I have no recollection of any of the "auto-release" stuff you speak of.

5) If you remove the wrapper task, you'll need to add it back the next time you want to run gradle wrapper. Better to just leave it.

yuuka-miya commented 8 years ago

Continue in #1141