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

Use upstream org.reflections #132

Open
BenjaminAmos opened this issue Mar 1, 2022 · 1 comment
Open

Use upstream org.reflections #132

BenjaminAmos opened this issue Mar 1, 2022 · 1 comment

Comments

@BenjaminAmos
Copy link
Contributor

Currently gestalt depends on a custom snapshot of what was org.reflections:0.9.12, re-named to org.terasology:reflections:0.9.12-MB. I do not know the exact reasons for the fork but MovingBlocks/reflections only contains a single unique commit with minor code changes. The README claims that the project is unsupported but things appear to have changed since then, with several upstream org.reflections releases being made, up to the most recent org.reflections:0.10.2. It might be worth upgrading to the upstream source to benefit from any bugfixes that might have occurred since then.

This is mostly an obsolete issue with gestalt 8 (develop branch), since it doesn't rely on reflections anymore. Terasology and Destination Sol could still be relying on gestalt 7 for some time to come though.

@keturn
Copy link
Member

keturn commented May 19, 2022

I'm not finding all my notes, but here's what I remember of the last time I looked at this:

At the time, I think there had been some newer reflections release, at least 0.9.12. But looking over the bug reports, it was clear there were some problems, and trading my then-problems for different problems didn't seem worth it.

If that was sometime around MovingBlocks/Terasology#4564 , then it was before the upstream 0.10.x releases came out. Maybe those are better?

I think by then (October '21), it was long enough after the v8 work had started that I hoped v8 would be released before I had to worry about Reflections versions again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants