AgriCraft

AgriCraft

30M Downloads

Replace direct referencing of other mods with reflection

InfinityRaider opened this issue ยท 6 comments

commented

These mods' classes are still directly being referenced on initialization, has to be replaced with reflection:

  • Botania: Configuration class
  • Forge Multipart
  • Pam's Wee Flowers: seed classes
  • Pam's Mob Drop Crops: seed classes
  • Tinker's construct: cache pointers to certain tool classes
  • Witchery: Item registry class
commented

Well done. I looked into getting gradle to pull the deps automatically using ivy and curse forge, but that took forever, and still had some major issues.

commented

Something you could look at is placing the mods to test with in a libs file on each git branch, so different branches could test different mods. It might make it easier to update the versions too, and quicker for travis...

It might be frowned upon, but whatever...

commented

I was talking about migrating the libs out of dropbox to the repo.

commented

I like to have all libs there, and I just toggle provided/compile to what I want to test.

commented

That's too bad...

commented

Some mods don't have open distributing permissions, and the installdeps.sh kinda 'hides' them.