LambdAurora / LambDynamicLights

A dynamic lights mod for Minecraft on Fabric Loader.
https://modrinth.com/mod/lambdynamiclights
Other
397 stars 98 forks source link

LambDynamicLights broken on macOS using Minecraft 1.19.4 #151

Closed aellul27 closed 1 year ago

aellul27 commented 1 year ago

Describe the bug LambDynamicLights crashes when opening it in modmenu or opening a world macOS on the 1.19.4 build.

To Reproduce Steps to reproduce the behavior:

  1. Install forge and lambdynamiclights
  2. Run installation
  3. Open any world
  4. See error

Expected behavior The expected behavior is that the world renders in allowing the player to play in that server instance. Instead, lambdynamic lights crashes.

Desktop (please complete the following information):

Additional context The error that seems to be causing it is: [23/04/2023 18:15:29 pm] Caused by: java.nio.file.NoSuchFileException: config/lambdynlights.toml [23/04/2023 18:15:29 pm] at java.base/sun.nio.fs.UnixException.translateToIOException(UnixException.java:92) [23/04/2023 18:15:29 pm] at java.base/sun.nio.fs.UnixException.rethrowAsIOException(UnixException.java:106) [23/04/2023 18:15:29 pm] at java.base/sun.nio.fs.UnixException.rethrowAsIOException(UnixException.java:111)

1.19.4 Log

Fnige commented 1 year ago

Caused by: java.nio.file.NoSuchFileException: config/lambdynlights.toml

If you have a ldl config in your config folder, try deleting it

aellul27 commented 1 year ago

FnigePython no effect by deleting config/lambdynlights.toml

CuddlyZombie commented 1 year ago

+1, I'm having the exact same issue, my exception is instead

dev.lambdaurora.lambdynlights.shadow.nightconfig.core.io.WritingException: Error while saving the FileConfig to config/lambdynlights.toml Deleting the file did not resolve it.

phiberber commented 1 year ago

I'm actually having both errors:

[07/05/2023 15:38:37 PM] dev.lambdaurora.lambdynlights.shadow.nightconfig.core.io.WritingException: Error while saving the FileConfig to config/lambdynlights.toml
[07/05/2023 15:38:37 PM]    at dev.lambdaurora.lambdynlights.shadow.nightconfig.core.file.WriteAsyncFileConfig$WriteCompletedHandler.failed(WriteAsyncFileConfig.java:166)

And

[07/05/2023 15:38:37 PM] Caused by: java.nio.file.NoSuchFileException: config/lambdynlights.toml

I would assume that the path is wrong? Lambdynlights.toml does exist in the config folder, Java would not prompt a NoSuchFileException with an existing file, and I would doubt the mod is deleting the config file at anytime.

After learning a little about Fabric and how other mods make config implementations, I'm inclined to say that the problem is in the line https://github.com/LambdAurora/LambDynamicLights/blob/a2da510bda2bf17ca638eb900f648116319635d8/src/main/java/dev/lambdaurora/lambdynlights/DynamicLightsConfig.java#L41

Having by example BorderlessMining

    import net.fabricmc.loader.api.FabricLoader;

        [...]
    private static final transient Path configFile = FabricLoader.getInstance().getConfigDir().resolve("borderlessmining.json");
aellul27 commented 1 year ago

Opened a pull request to fix this issue: #152

phiberber commented 1 year ago

What's the reason behind the re-opening? Did the issue persist?

aellul27 commented 1 year ago

What's the reason behind the re-opening? Did the issue persist?

Pull request #152 has not been merged with the repositiory. I have re-opened the issue so people who are encountering the issue can find the required changes and to bring attention to the issue.