[Crash Report] Incompatibility with Arc Lib
tooopt opened this issue ยท 4 comments
Minecraft and Mod version ๐
1.20.1 Forge
5.4.1 EasyNPC version
Description of the crash ๐ฅ
If EasyNPC and arc lib are installed, when you spawn an npc, the game crashes
Steps to reproduce the crash โ๏ธ
- Install easynpc
- Install ArcLib with dependencies (Arclib, Architecturyapi, UI Lib, SuperMartijn core lib and SuperMartijn config lib)
- Launch the game, create a new world and spawn an npc
Expected behavior before the crash โ๏ธ
Expected behavior would be to spawn an npc
Logs ๐
debug log
- https://pastebin.com/yTZtG5v4
crash.log
- https://pastebin.com/xtCvhDgC
Additional context ๐
Side note that the mod is incompatible only with arclib in this situation, If you delete arclib but leave the dependencies, everything works normally
Thank you for the report.
The issue lies with ArcLib itself and its use of the mixin definition.
https://github.com/DAQEM/ArcLib/blob/f602714beb0b5b6a4c941fd4fb892e353fc0d3ef/common/src/main/java/com/daqem/arc/mixin/MixinLivingEntity.java#L24
In this mixin, they define a getLivingEntity
function, which overrides the getLivingEntity
method of NPC entities and other entities that implement this function.
Since the NPC entities are not part of their library, they should avoid making such changes in the mixin.
Instead, they should use their own namespace rather than a general term like getLivingEntity
.
Alternatively, they could remove this function entirely to improve compatibility with other mods and implementations that they do not own.
As mentioned earlier, this will likely cause issues with other mods that use getLivingEntity
for their entities.
Please report this issue to ArcLib, and feel free to use this information as a reference. Thank you.
This issue is stale because it has been open 7 days with no activity. Remove stale label or comment or this will be closed in 3 days.