Skip to content

Conversation

@stellanera98
Copy link
Contributor

not sure it was necessary in the end since at least part of the issue was not having it reobfuscate the jarJar file, but:

  • made MixinLivingEntity extend Entity
  • removed MixinEntity
    this is because A) the shadowed method is already public and B) https://wiki.fabricmc.net/tutorial:mixin_tips suggests extending the parent class of the target in such cases
  • suffix for default (non jarJar) jar is now -slim
  • jarJar task now also gets the manifest thingy that jar had. To be clear, I do not know where this data is going or if its needed
  • jarJar file is now included in the "publish" gradle task
  • trying to remove the "slim" jar made gradle unhappy because of implicit dependencies, dont know why and dont want to deep dive into forgegradle or the publish plugin to find out, it may stay and sit in a dusty corner where it'll be ignored

Based on #2129 with permission of the author

@stellanera98
Copy link
Contributor Author

@dphaldes

Comment on lines +17 to +21
public MixinLivingEntity(EntityType<?> p_19870_, Level p_19871_) {
super(p_19870_, p_19871_);
throw new IllegalStateException("Instantiated MixinLivingEntity");
}

Copy link
Contributor

Choose a reason for hiding this comment

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

Why is this needed ? It's an abstract class and won't be instantiated

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants