Minecraft Forge Server

[BrutalBirdie]

@atrilahiji I will test this

.toString().split(/\[\d+\:\d+\:\d+\s(INFO|WARN|ERROR)\]\:\s/)[2]);

Tested it with jsfiddle and regex101

text = "Feb 08 17:14:35 [16:14:35 INFO]: [Vault] No new version available";
console.log(text.toString().split(/\\[\d+\:\d+\:\d+\s(INFO|WARN|ERROR)\\]\:\s/)[2]);

[BrutalBirdie]

Well ... silly me did a copy-paste mistake and forgot the \ at the first [
also why not this RegEx \\[.*\\]\:\s a lot simpler.

[A Former User]

@brutalbirdie Lol true that should work

[A Former User]

@nebulon FYI I submitted a PR to the repo https://git.cloudron.io/cloudron/minecraft-forge-app/-/merge_requests/1

This should clean it up a bit more. Let me know if theres anything else you need from my end on getting this up and running as a proper app. Again, I am perfectly willing to be in charge of submitting MRs for any updates to take a bit of the load off of @staff

[nebulon]

yes thanks, saw that. I still have to play this through on my own to see what this business with the jar files is

[A Former User]

@nebulon For sure, thanks! Yeah if you have any ideas about that let me know. I'm a bit lost with trying to run a jar headless if it relies on options to be selected in the GUI and there is no documentation that indicates that it can be run headless

[girish]

Iirc, a jar is just a zip file. So maybe the files are inside it as resources.

[A Former User]

@girish Found this when unpacking the installer jar file...

I have basically 0 knowledge about Java. Does this mean anything to anyone?

[girish]

@atrilahiji Ah, good catch. I guess we can put in a link to https://www.patreon.com/LexManos in the DESCRIPTION/Doc link and maybe also setup some patreon from our side if we decide to publish it. I am not sure how popular forge is, but we can scale that amount based on usage.

[A Former User]

@girish Yeah thats going to be troublesome. I suppose from your end its a huge issue if the maintainer feels ripped off here. I suppose it really doesn't matter if it remains as an unlisted app for the moment (unless you think its ok... I definitely prefer having it listed but I get it if thats not possible). Either way I'll add proper attribution and patreon links.

Would you like me to reach out to the developer?

[girish]

@atrilahiji Yes, agreed. I think since there is a special comment about automating the installation, we should check regardless of the license. Please do reach out to them and ask how he/she feels about it.

[A Former User]

@girish Ok sent him a message. Waiting for a reply. I'll let you know what comes of this.

[A Former User]

@girish

I asked my brother about this since he’s more experienced with forge. Looks like this works...

Still waiting to hear back from the developer

[A Former User]

Pushed an MR for 0.4.0 with proper automation. That message from my brother saved the day

Made a new one... no conflicts this time.

[A Former User]

@girish So this developer is not awfully responsive. I tried a DM via his community discord and a message on patreon (threw him $7 since I automated it for my server at least). Not sure how to proceed. I guess I'm on the fence about this because on one hand I think this would be an excellent app to have but on the other hand, if usage picks up here he may actually have a tangible loss in ad revenue?

[girish]

@atrilahiji Maybe we can publish, and then based on usage, we (as in, Cloudron the company) is happy to setup some monthly patreon based on usage/installs.

But also, let's put the patreon link in the description and also in the app's home page. I haven't looked into the app but is this using the same UI as the other minecraft apps? If so, let's change the HTML of the landing page there to have a link to their patreon.

[A Former User]

@girish Oh sure, that makes sense. I messaged one of their other devs and he confirmed the main dev often doesnt reply or see messages. And that as far as he's concerned he doesn't have an issue with it. And yeah I can throw that up onto the repo in another MR if you want. I am going to support this guy so I can clear my conscience lol.

And to answer your question yeah the repo is honestly mostly the same as the current MC apps. I'll keep an eye on releases so I can update this. Also, to clarify I am currently using a beta release of forge in the repo to match the minecraft version we use in the other apps.

[A Former User]

@girish New MR here: https://git.cloudron.io/cloudron/minecraft-forge-app/-/merge_requests/4

Screenshot of the frontend change:

[murgero]

@atrilahiji Did you fix the issue where the original app (Minecraft Java Server) has memory-issues in browser causing the browser to crash/freeze?

[A Former User]

@murgero Huh I forked the java edition repo but never explicitly fixed something like that. Never really noticed that as an issue. Was this ever fixed in the Java edition repo?