Jump to content

Regarding Minecraft 1.12, And policy changes.


LexManos

Recommended Posts

 

Minecraft 1.12:

The newest version of Minecraft has been released. Forge has been updated to support this new version. However due to the way Mojang implemented the Recipe changes there are a lot of under the hood changes that we need to do. Most notably re-writing on of the largest/most intricate systems of Forge, The Registry system. This will take some time, so do not expect a recommended release quickly. However if you are a modder you can start using the current versions to build against. Some API's may change in the early days of Forge so be sure to be ready for that. I'm sorry, I usually try my best to maintain binary compatibility, but it's just what will need to happen. For users, you can use the current builds. But just be warned that things are actively being developed and we ask to please responsibly report issues on the forum.


Once I finish the rewrite and get a stable recommended build of Forge out I will make a more detailed post listing all the major changes like I always do.

 

New Policy on Coremods:

Sadly core modding is still a thing. As always we request that you guys attempt to work with us to get changes into Forge instead of core modding it yourself. However, if you MUST we have decided to put forth to the community a few 'Best Practices' for core modding. The intention is that large communities such as FTB, Technic, and Curse work with us to promote these best practices.

 

1) Coremod must be the coremod only, and no extra library/features/apis/etc.
There are far too many coremods in the community that package tons of classes that have nothing to do with the modifications they make to the game together so that people will be forced to use their coremod just because they want a utility. This is bad. So Coremods themselves should be limited to JUST the IFMLLoadingPlugin, and IClassTransformers, and as few utility methods needed to make those hooks run.

 

2) Coremod binaries must be signed.

This is a very basic thing that just verifies the person/organization we think made the coremods actually did. It also verifies that the file that was downloaded has not been modified in any way. As it sits there will NOT be any central authority on the keys used to sign things. So Self-signing will be allowed as long as you provide the community your signature. Starting in 1.13, with the loading system rewrite, users will be prompted to accept signatures of coremods. It is our intention to work with people like FTB, Curse, and others to make these signatures easy to use and manage.

 

For example a user would say they trust FTB, and wouldn't be prompted for every coremod that exists in a FTB modpack.

 

For the technical side you can read more about Jar Signing here: https://docs.oracle.com/javase/tutorial/deployment/jar/signindex.html

 

3) Coremods should be visible source.

This will be a controversial standard, but my thoughts on it is that if you're screwing with someone else's code (which is the only reason to ever write a coremod), then you should be willing to show what you are doing. It is stressed that this is VISIBLE SOURCE only. It is not a requirement that you allow others to use your code, or modify and distribute it. It's simply that we can see it.

 

The signatures and visible source are NOT designed to be security measures. As there is nothing preventing malicious code from being signed and a user accepting it. This is just the risk you run with Minecraft modding as you're running compiled code from random people on the internet. This is however designed to make the community more open and try and stem the number of coremods out there that have no reason to be coremods.

 

Major Policy change:

 I am happy to officially announce a new member of the Forge team. Everyone welcome Mezz. His official responsibilities are to be the Pull Request, and Issues manager. Basically, he's the guy you yell at if your PR/Issue is rotting on github. He's the guy who is tasked with reminding me that they exists. He will be the one responsible for filtering all the PRs/Issues before they get to me. So I don't have to deal with telling you guys to follow the standards like making a test mod, posting logs, etc..

 

 In addition, he is also the one who decides if old versions will have PRs accepted. Yes this means there will be a limited development system for older versions. How far back that means is ENTIRELY up to Mezz. However the rules are that ANY pr adding features for a old version MUST be applied and accepted for the current version FIRST. Save for features that would have no place in the new version. Example being adding a new achievements hook when the new version removed achievements.

 

 It will still be our official stance on this forum to only provide support for the Current version, and the previous version. However, if the community wishes to have a few dedicated people step forward and become our Legacy support team them I am willing to work with them and see what we can set up. The main reason we do not provide support for old versions is simply we do not have the manpower. So start helping out!

 

Response From Sponge: 

 

  • Like 20

I do Forge for free, however the servers to run it arn't free, so anything is appreciated.
Consider supporting the team on Patreon

Link to comment
Share on other sites

For future reference

Quote

[23:05:07] <@LexMobile> FML has mechanics for extracting jars from jars
[23:05:15] <@LexMobile> So packaging to the end user is still the same

If I understand this correctly, we would be allowed to put the coremod into its own jar and put that jar into the actual mod jar. While this might not be very useful for modders that use a coremod in multiple mods, it might be useful if its only a change for a single mod.

Question I have: could you point me and others to the jar extraction mechanic FML provides? I dont intend on using a coremod at all, but just in case.

Edited by xalcon
  • Like 2
Link to comment
Share on other sites

You certainly have my support in regards to the changes regarding coremods. For too long have mod authors, written coremods that prevent Forge doing its thing: promoting mod compatibility. It's inevitable that this new policy won't eradicate coremods, but perhaps it will make their authors think more carefully about the repercussions of making changes.

 

And in closing, I would like to congratulate Lex on reaching 7777 posts :D

  • Like 2
Link to comment
Share on other sites

While I do support changes regarding coremod as the idea (and completely agree with points 2 and 3 as-written), I'm a bit confused as to what the first part says about coremods that just can't have the mod and coremod part separated without the whole thing blowing up when any part is used without the other. While I know it's probably nto common, I want to know what the first policy says about my mod. With cubic chunks I can't for example have my any part of the mod working without coremod, and the coremod part can't work without other parts of the mod. Well, I actually use mixin which uses a tweaker, but this is outside of my control (and I could stop using mixin but it would be huge amount of work and has more potential for me breaking things).

 

While in principle I could make the "coremod" part work by itself, it's not going to be of any use. If this really still applies in my case, I really hope it never becomes anything close to a requirement.

  • Like 1
Link to comment
Share on other sites

Feel free to explode if a coremod is installed without its handler mod.

We already have a mechanic for depending on other mods, so you can do normal dependency resolution.

Tweakers are considered coremods for this policy, Its all the same thing. As for Mixins I've already spoken to Mumfey and he's on board with this. He just needs to sign the Mixin library and he'll be good. However, Mixins is a special case in regards to #1. Its entire point is to be a library for other coremods so there isn't anything to 'separate' out. The intention is to stop coremods being required by mods who dont NEED them because they use a central/popular library.

  • Like 1

I do Forge for free, however the servers to run it arn't free, so anything is appreciated.
Consider supporting the team on Patreon

Link to comment
Share on other sites

@vpontin: Note that this isn't a requirement for open source coremods, they have to merely be visible source. Like Lex outlines in the post, this does not mean that the code is open to be edited by the community or that it would have to be licensed under an open source licence. The only requirement is that people can look at the code in order to reason about it and understand it more easily.

Link to comment
Share on other sites

7 minutes ago, LexManos said:

Feel free to explode if a coremod is installed without its handler mod.

We already have a mechanic for depending on other mods, so you can do normal dependency resolution.

Tweakers are considered coremods for this policy, Its all the same thing. As for Mixins I've already spoken to Mumfey and he's on board with this. He just needs to sign the Mixin library and he'll be good. However, Mixins is a special case in regards to #1. Its entire point is to be a library for other coremods so there isn't anything to 'separate' out. The intention is to stop coremods being required by mods who dont NEED them because they use a central/popular library.

While I completely understand the intention and I agree with it (cubic chunks itself is being a "victim" of depending on coremod while the coremod part isn't needed, specifcially I optionally depend on MalisisCore for GUI). What you say is still a bit confusing and seemingly self-contradictory to me, but I will wait and see. Spong uses mixin too, so I will definitely see what they do. I'm in very similar situation here.

Edited by Barteks2x
Link to comment
Share on other sites

If you USE mixins, things should be baked down into metadata that Mixins{Mumfrey's library} loads. That data {Usually the compiled classes and whatever other metadata Mixins create} should be separate from your mod and signed. And the source code that generates that data should be visible in some way.

 

If you require a coremod in your main mod, Just add 'required: mymod_coremod' to your @Mod.

If you require a mod from your coremod {such as the handlers for the mixins you do} then you can add the mod as a dependency to the coremod.

  • Like 1

I do Forge for free, however the servers to run it arn't free, so anything is appreciated.
Consider supporting the team on Patreon

Link to comment
Share on other sites

So I should have the mixin classes (what you called metadata) separate from the other part. Does forge support circular dependencies (coremod depending on mod and mod depending on that coremod)? If it does, then I can do that but it seems needlessly artificial and unnecessary separation to me. And the main reason I asked the question in the first place.

 

My whole mod is completely open source so I'm absolutely fine with source visibility here. And I was interested in signing my jar anyway, now I just have a good reason to do it.

Link to comment
Share on other sites

Yes, Forge supports mutual dependencies. As long as you don't define both requiring to load AFTER each other. It should just detect that ModA wants ModB in the list, and ModB wants ModA in the list. The intention of #1 is to allow those who care about coremods to go in and figure out what is relevant to the coremod, the things that edit bytecode. It's to help prevent people from hiding things amongst 500 different classes that don't touch bytecode. On top of that it is also a technical thing. If the coremod code is separate from the mod code we can better manage the classpath and classloaders. We have had many issues over the years of people including APIs in coremod. Which screws over our ability to sort and verison APIs so that the one available is the one people want.

 

I kinda expect that once people see start using the Jar-in-Jar stuff, {which I admit needs fleshing out and documentation} that people will stop fat-jaring APIs but instead include them as a internal jar. And let Forge pull out the ones that best suit the environment based on what mods say they need.

  • Like 1

I do Forge for free, however the servers to run it arn't free, so anything is appreciated.
Consider supporting the team on Patreon

Link to comment
Share on other sites

I don't speak for the whole community but since I am a leader of Sponge, I'll throw my 2 cents in by first saying that yes, SpongeForge will be adjusted to comply. We find these changes to be a good-faith effort to make some wholesome changes to the ecosystem as an attempt at dealing with the elephant in the room: coremods. SpongeForge falls under a coremod which will not function at all without its core changes but, as Lex has mentioned above, we can simply make the core portion not function at all should the whole thing not be here and throw up a descriptive error to say why. Expect to see some changes land in the next couple of days (when someone gets some time) on Sponge's end to make this change.

Edited by Zidane
  • Like 4
Link to comment
Share on other sites

Chiming in as one of the other leaders of Sponge, I've always believed that making heavy modifications to the game (as coremods sometimes do) would benefit from being viewed source, since often times debugging odd interactions between SpongeForge (and SpongeCommon as the larger part) have been often times difficult to say the least to work out where the issue lies. That being said, I strongly support the idea around coremods being viewed source for the sake of inter-coremod compatibility overall, as the goal with SpongeForge has always been about remaining compatible with as many mods as possible out of the box (even if the technological attempts at maintaining that compatibility is not as evident or thought up very fast). As @Zidane has mentioned, SpongeForge will be making the necessary changes in the very near future to abide by these requirements. As per usual, my goal with inter-mod compatibility is always to provide as much descriptive console spam error logs to help Sponge and/or the mod itself get a better fix and make the users happier overall.

  • Like 3
Link to comment
Share on other sites

On 6/17/2017 at 11:47 PM, LexManos said:

Coremods should be visible source.

Don't you think that this will also cause an increase of coremods, because more people will learn how to use ASM? 

I would add another recommendation for coremod makers - just provide configs for coremods with options to disable them or some of their features.

Link to comment
Share on other sites

2 hours ago, Alexiy said:

Don't you think that this will also cause an increase of coremods, because more people will learn how to use ASM? 

I would add another recommendation for coremod makers - just provide configs for coremods with options to disable them or some of their features.

That may be a concern, there will always be copy pasta. But Honestly that is easy enough to do these days and SUPER easy to do wrong. So I think itd probably just increase the quality of the copy pasta.

As for config stuff, that is to specific, we're not saying ANYTHING about functionality. Just guidelines on packaging.

I do Forge for free, however the servers to run it arn't free, so anything is appreciated.
Consider supporting the team on Patreon

Link to comment
Share on other sites

14 hours ago, Alexiy said:

Don't you think that this will also cause an increase of coremods, because more people will learn how to use ASM? 

I would add another recommendation for coremod makers - just provide configs for coremods with options to disable them or some of their features.

I don't think so. Modifying code via bytecode manipulation requires extensive knowledge of the JVM stack and instruction set. You can't really "copy-paste" an injection and expect it to work on a new class. It might work, it might not. 

 

To Lex:

I support these changes. I will continue to advocate that things should be done through forge and will likely not help people learn how to use ASM, at least not in a public setting, but I would instruct (to the best of my ability) how to coremod safely. I did a lot of it in 1.7.10 (and I'm not necessarily proud of it) and for the folks that merit knowing should be informed. Maintaining compatibility is important.

 

The one thing we will want is an easily accessible guide on signing jars and how to share the singing details correctly. It definitely isn't something i know how to do off the top of my head and I'm not confident that I'd be able to search out a solution either. (Can I search, sure, but I read between the lines that there might be a step that has to be done a certain way and don't know if I would be able to find details).

 

I'm also glad to see the appointment of someone to handle lingering PR requests and filtering, I know I had some that took a while to be approved. Even if in some cases it was my own fault for not following the guidelines, but again, best practices aren't something most people can pick up in an afternoon. Its easy to make mistakes (like knowing that X mistake was made in several places) and I know you don't have much time to impart that knowledge yourself. Don't hesitate to find a few helpers for Mezz, as well. Don't want to see him getting bogged down either. 

  • Like 1

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

Link to comment
Share on other sites

  • 1 month later...
On 18/06/2017 at 6:17 AM, LexManos said:

As always we request that you guys attempt to work with us to get changes into Forge instead of core modding it yourself.

How would I go about doing this?

 

I have been working on a coremod that causes the game to render the world 6 times per frame. Is this something that should have forge hooks for it, or is it better to use a coremod?

  • Like 1
Link to comment
Share on other sites

  • 8 months later...

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Unfortunately, your content contains terms that we do not allow. Please edit your content to remove the highlighted words below.
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Announcements



×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use.