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

Forge log4j configuration is not applied correctly on Windows #200

Open
gingershaped opened this issue Mar 13, 2024 · 3 comments
Open

Forge log4j configuration is not applied correctly on Windows #200

gingershaped opened this issue Mar 13, 2024 · 3 comments
Labels
bug Something isn't working Forge This issue is specific to the Forge platform.

Comments

@gingershaped
Copy link

The launch.cfg file generated by the generateDLIConfig task sets the property log4j.configurationFile to the raw path of the log4j.xml located in .gradle/loom-cache. Log4j subsequently attempts to parse this path as a URI. This works on Linux, but fails on Windows because Windows paths are not valid URIs.

@Juuxel Juuxel added bug Something isn't working Forge This issue is specific to the Forge platform. labels Mar 14, 2024
@Juuxel
Copy link
Member

Juuxel commented Mar 14, 2024

I can confirm this, actually, but I have no clue why it only works on Fabric... I think we can switch to setting the property to an URI if that fixes the bug.

@rlnt
Copy link

rlnt commented Aug 26, 2024

Can confirm this on NeoForge 1.21.1

WARN StatusConsoleListener Error parsing URI C:\Users\rlnt\repos\unified\multi-1.21\NeoForge\.gradle\loom-cache\log4j.xml
WARN StatusConsoleListener Advanced terminal features are not available in this environment
Architect Plugin: 3.4.159
Architectury Loom: 1.7.412

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Forge This issue is specific to the Forge platform.
Projects
None yet
Development

No branches or pull requests

4 participants