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

Regression: PolyMC does not properly respect some org.lwjgl3.json's after 09e85e948cdb361c306a1cccbc3557a464366a21 #1654

Open
1 task done
Richard-Rogalski opened this issue May 10, 2024 · 3 comments
Labels
bug Something isn't working

Comments

@Richard-Rogalski
Copy link
Contributor

Richard-Rogalski commented May 10, 2024

Operating System

Linux, PPC64le architecture, using custom natives

Version of PolyMC

Tested up to latest git master, still seems present

Version of Qt

5.15.13
6.7.0
I believe this old commit only supported Qt5

Description of bug

The instances listed here only work at or before commit 09e85e948cdb361c306a1cccbc3557a464366a21. For example-- this instance here. Before this regression, it downloaded the natives specified in org.lwjgl3.json, currently, it doesn't and just errors out. (will show log of what happens currently, but will have to wait to rebuild poly yet again).

Note that my git bisect knocked down the troublesome commit to either 09e85e948cdb361c306a1cccbc3557a464366a21 or the one immediately after. I could not test the one after due to an unrelated build failure with the code (if you want me to test this commit, I can try again and patch the code to fix each build failure one by one).

Also note that I do not own a valid copy of minecraft (anymore, did not migrate), so I have to rely on an offline account and cannot test with a valid online MC account. (using the good 'ol echo '{"accounts": [{"entitlement": {"canPlayMinecraft": true,"ownsMinecraft": true},"type": "Offline"}],"formatVersion": 3}' > ~/.local/share/PolyMC/accounts.json). could be related(?)

Steps to reproduce

I think that it should still be testable on a non ppc machine (testing whether it respects the json files), but if any Poly dev wishes to test on real ppc hardware they can reply here or on discord with an ssh pubkey. rrogalski#0 on discord

Suspected cause

No response

This issue is unique

  • I have searched the issue tracker and did not find an issue describing my bug.
@Richard-Rogalski Richard-Rogalski added the bug Something isn't working label May 10, 2024
@Richard-Rogalski
Copy link
Contributor Author

Richard-Rogalski commented May 11, 2024

before regression. it works!

after regression + after2.txt , git master build of poly. you can see it is no longer using the natives

modern prism + this is as far as it gets, unrelated issue just thought I'd share heh

@LennyMcLennington
Copy link
Member

LennyMcLennington commented May 11, 2024

@Richard-Rogalski thanks for the detailed reporting, but I need some clarification. You said that the instance only works "on or before" 09e85e9 (implying that the commit is good) but then you said the troublesome commit was either 09e85e9 (implying that the commit is bad) or the commit immediately after. Which commits were actually tested? Do you have a bisect log that you can share?

@Richard-Rogalski
Copy link
Contributor Author

Sorry. 09e85e948cdb361c306a1cccbc3557a464366a21 is good. I was unsure if the commit immediately after introduced the regression, or if it was good and unaffected, as I had build failures on that commit.

I've looked more closely at the issue. It seems that the 'linux' in "linux": "natives-linux" was aliased to linux-x86{_64}, so after the 'regression' poly will only look at linux if on x86. After this change, poly would be explicitly looking for "linux-ppc64le": "natives-linux". So I'm not sure if the current behavior is intended or not intended, maybe if linux-ppc64le isn't specified it tries just linux?

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

No branches or pull requests

2 participants