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

Absolute paths for Minecraft 1.12.2 and earlier. #1459

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

AHilyard
Copy link

@AHilyard AHilyard commented Oct 3, 2022

Purpose

Forge for Minecraft 1.12.2 and earlier do not fully support relative paths, causing some instances to fail. Here are some easy steps to reproduce the problem:

  1. Create a Forge 1.12.2 instance.
  2. Add the mod "Equipment Compare" to the instance.
  3. Launch the instance.

It can be seen from the latest.log file that Forge has tried to search the same directory twice, once with the relative path and once with a non-normalized absolute path.
Other launchers appear use absolute paths to specify game directory and such, which bypasses this issue.

Approach

Add a check in launchInstance to bypass the relative path string replacement when the Minecraft version is 1.12.2 or lower.

@CLAassistant
Copy link

CLAassistant commented Oct 3, 2022

CLA assistant check
All committers have signed the CLA.

@Eskaan
Copy link
Collaborator

Eskaan commented Jan 20, 2023

May this bug be solved by selecting the symlink option in the settings? (I think that is part of the reason we have it)

@Eskaan Eskaan added Status: Awaiting Feedback This will be closed whitout feedback. and removed Status: Ready for Review labels Jan 20, 2023
@EndangeredNayla EndangeredNayla mentioned this pull request Jan 22, 2023
3 tasks
@MrEAlderson
Copy link

Can this be merged already?

@Eskaan
Copy link
Collaborator

Eskaan commented Feb 13, 2023

Awaiting Feedback

@AHilyard
Copy link
Author

May this bug be solved by selecting the symlink option in the settings? (I think that is part of the reason we have it)

I have just tested this and changing startup method to symlink mode does work. I guess this and the other identical issues point to another problem--why is symlink mode not the default if it fixes this problem, is there some downside?

@blarfoon
Copy link
Member

The relative path is used to avoid the "command line too long" issue. Doing this in general could cause it again even with the relative paths. A possible fix would be to only use the absolute path for the game/assets paths?

@AHilyard
Copy link
Author

AHilyard commented Feb 13, 2023

The relative path is used to avoid the "command line too long" issue. Doing this in general could cause it again even with the relative paths. A possible fix would be to only use the absolute path for the game/assets paths?

I thought that issue was fixed in #1384? The path that needs to be specified as an absolute path is the path to the instance folder. I'm guessing all the other paths can be specified relatively, but I don't know for sure.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Awaiting Feedback This will be closed whitout feedback. Type: Bug Non fatal problems who can be ignored/worked around
Development

Successfully merging this pull request may close these issues.

None yet

5 participants