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

Crash without Baubles installed (+ Extra Utilities & Fastcraft) #2203

Closed
Cherry opened this issue Apr 1, 2015 · 27 comments
Closed

Crash without Baubles installed (+ Extra Utilities & Fastcraft) #2203

Cherry opened this issue Apr 1, 2015 · 27 comments

Comments

@Cherry
Copy link

Cherry commented Apr 1, 2015

No issues:

EnderIO WITH Baubles and no Extra Utilities.
EnderIO WITHOUT Baubles and no Extra Utilities.
EnderIO WITH Baubles and Extra Utilities.
Extra Utilities WITH Baubles installed.
Extra Utilities WITHOUT Baubles installed.
All of the above both WITH and WITHOUT FastCraft.

Crashes:

EnderIO, no Baubles, Extra Utilities and FastCraft. Crash report.

Versions utilised:

Baubles: 1.0.1.10
EnderIO: Build 365
Extra Utilities 1.2.2
Fastcraft: 1.21


x-posting this to the FastCraft thread as I believe the issue may reside there, but creating an issue here for reference also. Happy to provide further information if necessary.

@tterrag1098
Copy link
Member

So it only ever happens with Fastcraft installed? The stacktrace does not mention EnderIO or ExU or Baubles, so I don't see how it could be caused by them.

@Cherry
Copy link
Author

Cherry commented Apr 1, 2015

As far as I have been able to reproduce so far, yes. Here's another crash report from earlier testing with a few more mods installed that does mention ExU, as well as the baubles API (which I don't believe ExU implements). Fastcraft was again enabled during this test though.

@tterrag1098
Copy link
Member

I believe the crash in your original report is caused by Fastcraft shipping files it shouldn't be. This other report is more interesting.

@tterrag1098
Copy link
Member

Further still, EnderIO does not ship the Baubles API, so I fail to see how this could be our fault.

@Cherry
Copy link
Author

Cherry commented Apr 1, 2015

The crash doesn't appear to be happening with EnderIO build 363, which is why I originally created the issue here. I've x-posted to http://forum.industrial-craft.net/index.php?page=Thread&postID=183576#post183576 either way, just wanted to make you aware.

Build 364 (with ExU, no baubles, and Fastcraft) caused an instant crash in IConduitBundle (due to no Mekanism?): https://pastee.org/cxrj9
Build 363 loaded without issue (with ExU, no baubles, and Fastcraft).

@ghost
Copy link

ghost commented Apr 1, 2015

i dont use fastcraft and i still get this crash. extrautilites wont load without baubles installed

@sfPlayer1
Copy link

I'm fairly convinced this isn't related to FastCraft, which btw. doesn't ship anything outside the fastcraft package.

The error means that something caused the system class loader to load the com/google/common/collect/HashMultimap class, which shouldn't ever happen. Most likely someone is messing with class loader contexts or accesses the system class loader directly.

@tterrag1098
Copy link
Member

@sfPlayer1 I can assure you we do none of that. This really can't be EnderIO's problem as we don't even ship baubles API, closing.

@Cherry
Copy link
Author

Cherry commented Apr 2, 2015

Any idea as to why the issue doesn't happen in build 363 then? #2203 (comment)

I'm happy to investigate other causes, but as switching out EnderIO versions appears to resolve it, and another user also reported a similar issue above, I'm inclined to believe there's definitely an issue here somewhere, be it an edge-case possibly.

@ghost
Copy link

ghost commented Apr 2, 2015

yes, ender io is calling for baubles to load, if not installed then it forces a crash, end of story, so its the code ender io used to add more baubles support is the cause, it didn't do it before you added that code so whats that tell us...

@CaptainYarb
Copy link

I too, can confirm that an older build of EnderIO works. Clearly this issue is caused by the current build.

@ghost
Copy link

ghost commented Apr 2, 2015

from the way it seems ender io is trying to import baubles api and if its not found it crashes.

@KlutzLament
Copy link

I believe the problem is with Extra Utilities. I remember reading that there was an issue with it crashing if baubles wasn't installed. That also seems to be one of the few combinations you haven't tried, namely No EnderIO, No Baubles, with Extra Utilities.

@Cherry
Copy link
Author

Cherry commented Apr 2, 2015

Extra Utilities WITH Baubles installed.
Extra Utilities WITHOUT Baubles installed.

^ both work without fine without EnderIO, as in the original post, even with the new ExU 1.2.3 released today.

EnderIO is the key variable here. Removing it (or reverting to build 363 or lower) resolves the issue.

@tterrag1098
Copy link
Member

EnderIO works by itself, but not once you install ExU, and the stack trace starts in ExU code. How does this point to us again?

@Cherry
Copy link
Author

Cherry commented Apr 2, 2015

ExU also works by itself, and the two have worked fine together until build 365 of EnderIO.

Granted, the stack trace does start in ExU code, but the crash goes away when I use build 363 of EnderIO.

I'm absolutely happy to provide any other tests or information here, I've just exhausted everything I can think to provide. If there's anything further I can do to hasten the resolution of this, please let me know.

@Cherry
Copy link
Author

Cherry commented Apr 3, 2015

Build 365 seems to add "more baubles support" as per e8feb57 so I don't feel that it's a huge jump to consider that may be affecting this issue.

My test environment generally doesn't contain Mekanism, so I haven't tested build 364 up to this point, but I have just done so, and build 364 also loads without any issue, using the same set of tests as originally dictated (after adding Mekanism into the mix to prevent the IConduitBundle crash).

Build 365 is the only build that this issue occurs with.

@Cherry
Copy link
Author

Cherry commented Apr 3, 2015

I just compiled the latest source in Git, reverting solely commit e8feb57 and this issue no longer occurs.

I hate to submit a pull request that removes features, but it completely resolves this issue.

@tterrag1098
Copy link
Member

Our code is fine, I'm not removing a feature because it somehow causes another mod to crash.

@sfPlayer1
Copy link

While said commit seems to trigger the problem, it by no means verifies it being the cause. It's probably just affecting the load order or something similar.

Maybe extract all your mods and then search (grep) for getSystemClassLoader or HashMultimap.

Running java with -verbose:class may be suitable to log the loading of HashMultimap.

@CaptainYarb
Copy link

@tterrag1098 Wow, that's a pretty messed up mentality to take on an issue, especially in a mod compatibility issue with dozens of modpacks that use your modpack. You've done nothing but try to brush us off while we are only trying to provide you with tests that show the issue with the most recent build.

@ghost
Copy link

ghost commented Apr 3, 2015

Our code is fine my ass, this has got to be one of the worst mod authors there is, he blames everything but whats the problem, his code, removing the code fixes the issues, its not just one mod that its having problems with, this is what you get when someone else takes over someones mod, if crazypants were here im sure the issue would be fixed. and i agree 100% with @Blazedd you keep brushing us off, closing things, and not providing any real support.

@tterrag1098
Copy link
Member

Brushing you off would be ceasing to respond to an issue which I've already determined is not my fault. EnderIO does not ship any Baubles classes, yet ExU crashes with a NoClassDefFoundError? That doesn't make sense.

@tterrag1098
Copy link
Member

If you point to me the part of our code that could possibly be causing this issue I will fix it, but until then I can see no such code.

@tterrag1098
Copy link
Member

Here, try this build, I've read through all baubles related code and cleaned up anything I thought suspicious. I severely doubt it will affect anything but it's worth a shot. http://ci.tterrag.com/job/EnderIO/368/

@Cherry
Copy link
Author

Cherry commented Apr 3, 2015

No need to bash anyone guys, we can all be amicable here :)

I very much appreciate the cleanup @tterrag1098, though as you expected, it hasn't changed anything. I've opened an ExU issue regarding this bug also, rwtema/extrautilities_old_issues#554 so I guess we'll see if @rwtema has any insight.

@MatthiasMann
Copy link
Contributor

As far as I can tell from the code - EIO only interacts with Baubles if FML tells us that baubles is installed. And I run EIO in my test world without Baubles without issues.

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 a pull request may close this issue.

6 participants