Skip to content

NeoForge support#356

Draft
stewi1014 wants to merge 6 commits intopcal43:mainfrom
stewi1014:neoforge
Draft

NeoForge support#356
stewi1014 wants to merge 6 commits intopcal43:mainfrom
stewi1014:neoforge

Conversation

@stewi1014
Copy link

@stewi1014 stewi1014 commented Nov 20, 2024

This adds a NeoForge build of fastback

It depends on #355

ssh does not function in the NeoForge development environment, but works in the shadow jar; architectury/architectury-loom#248. I figure that might take a while to get a resolution to, and I'd vouch for not waiting on it given the bug only happens in development.

#260 and #261 are addressed in this PR.

I haven't looked into #313 and #264, and I'll mark this as ready once I do.

I've organized commits in this PR so they're easy to follow and understand, but it's probably worth squash-merging this.

Tunfisch96 and others added 5 commits October 24, 2024 08:09
This allows changes made to neoforge to be seen more clearly.
This configures the neoforge build, but does not address critical bugs that prevent the neoforge build from functioning.
…rt in fasback-forge'

At-least I believe it does. Uploading was failing with a jgit error aboud failing to negotiate the algorithm.
Uploading to gitea and github now work after this change, which indicates to me that this issue is resolved.

However, org.apache.sshd goes missing in development environments, and the SshHacks fix does not work anymore because of the library change.
I've made an issue in architectury-loom for this: architectury/architectury-loom#248

In addition, the error thrown due to org.apache.sshd going missing is now being logged, but will not prevent fastback from loading.
Allowing the development environment to still be used without support for ssh.
…osaves) is not currently supported on Forge'

Mixins are used in much the same way they are in the fabric version.
@stewi1014 stewi1014 marked this pull request as draft November 20, 2024 10:10
ModLoadingContext can be supplied to the Mod's constructor, so using it.
filesMatching("META-INF/neoforge.mods.toml") {
expand "version": project.version
}
}
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

processResources is duplicated

@eehunter
Copy link

How usable is this neoforge port? If I were to build it and install it on my home server with default settings, would it work?

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

Successfully merging this pull request may close these issues.

3 participants