Cursed-Legacy-API icon indicating copy to clipboard operation
Cursed-Legacy-API copied to clipboard

No refmap loaded

Open halotroop2288 opened this issue 4 years ago • 6 comments

Compatibility is broken with other mappings because mixins fail due to using mapped injection points. Fix this by including a refmap.

halotroop2288 avatar Aug 30 '21 18:08 halotroop2288

wait it doesn't have a generated refmap? 😳

Probably a problem with mixin compiling

valoeghese avatar Oct 29 '21 19:10 valoeghese

Calimamsy seems to indicate the problem was probably not on our end? CLAPI has a refmap, so "including a refmap" can't be the problem.

valoeghese avatar Oct 29 '21 21:10 valoeghese

I'll close this for now as we do have refmaps, so your comment on how to fix is invalid. And your workspace was modified in other ways, so there is a possibility it could have been that.

If you have reason to believe the problem IS on our end and that the normal Cursed Legacy workspace does not work with other mapping sets, please feel free to reopen the issue with attached evidence and please give an indication of which mixins are not remapping on compile correctly.

valoeghese avatar Oct 29 '21 21:10 valoeghese

Calimamsy seems to indicate the problem was probably not on our end?

What does Calimamsy know? How to start a feud over nothing is all I've seen. I seeked their guidance on making these APIs more compatible with one another and they only responded with hostility.

CLAPI has a refmap, so "including a refmap" can't be the problem.

I'd really like to see some evidence on that. My logs say otherwise.

And your workspace was modified in other ways, so there is a possibility it could have been that.

Yeah, this is programming. We all modify our workspaces to suit our needs. You're gonna have to be more specific oh what you think I modified that would cause this.

which mixins are not remapping on compile correctly

These Mixins were not remapping via Loom on import as a dependency. This caused startup to fail while applying them. You can try this yourself by making a new mod and applying BTU's mappings, or even BIN-Mappings.

reopen the issue with attached evidence

There is no chance of me returning, unless the whole of the pre-1.3 Fabric modding scene sees major changes. Using this toolchain is like pulling teeth.

I'm looking at Chocohead for maintaining an outdated version of Loom with unnecessary unique features as our only way of creating Fabric mods for pre-1.3 versions, and ModificationStation for making an API that's incompatible with the more official one, and letting it become widely adopted.

Yes. I realize this is nearly impossible to fix this now. And that's exactly why I won't be returning. It is a toxic environment which I no longer have any problem responding to with my own toxicity.

halotroop2288 avatar Oct 29 '21 22:10 halotroop2288

I'd really like to see some evidence on that. My logs say otherwise.

I did look into the problem you presented, and investigated the cursed legacy api 1.1.0 jar file. Here are some examples of the refmaps.

image image

I would prefer to not turn this into a toxic argument, as that does no good to either side. I shall reopen this issue, as it is a pertinent issue that, as you have mentioned, was likely prematurely closed. I shall test at a later date. The issue at hand is likely not the absence of a refmap, but a problem generating the refmap. Loom has had such problems in other projects in the past and even now, for example, Lint currently doesn't work in prod due to a refmap not being generated properly.

valoeghese avatar Oct 29 '21 22:10 valoeghese

Speaking of making the APIs more compatible, I already plan on working on that in the near future.

valoeghese avatar Oct 29 '21 22:10 valoeghese