-
-
Notifications
You must be signed in to change notification settings - Fork 9
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
2.0.7 up thru the 2.2.0-alpha.4 #71
Comments
I think its because one of my mods i selected for the client mods has funny characters in them. DefaultOptions_1.16.5-12.2.1.jar, IKnowWhatImDoing-3.2.0-forge.jar, ModifierKeyFix-1.0.jar, Neat 1.7-27.jar, ReAuth-1.16-Forge-3.9.3.jar, Toast-Control-1.16.4-4.3.1.jar, [1.16.5][Forge] RPG-HUD-3.8.4.jar, desiredservers-1.16.4-1.0.1.jar, fancymenu_2.2.2-2_MC_1.16.2-1.16.5.jar, journeymap-1.16.5-5.7.3.jar, justenoughkeys-forge-0.2.2.jar, shutupexperimentalsettings-1.0.3.jar, torchoptimizer-1.16.5-2.4.8.jar such as "[]" and also " ". My guess is that the list on that line needs to be a quote delineated list. ie each element is quoted. |
using that information, i quoted the two I thought were suspect and both had a space in their names. it got past it, I went pack to the alpha and it got past it too. DefaultOptions_1.16.5-12.2.1.jar, IKnowWhatImDoing-3.2.0-forge.jar, ModifierKeyFix-1.0.jar, "Neat 1.7-27.jar", ReAuth-1.16-Forge-3.9.3.jar, Toast-Control-1.16.4-4.3.1.jar, "[1.16.5][Forge] RPG-HUD-3.8.4.jar", desiredservers-1.16.4-1.0.1.jar, fancymenu_2.2.2-2_MC_1.16.2-1.16.5.jar, journeymap-1.16.5-5.7.3.jar, justenoughkeys-forge-0.2.2.jar, shutupexperimentalsettings-1.0.3.jar, torchoptimizer-1.16.5-2.4.8.jar |
Yeah, if a mod has either [ or ] in them, they'll cause problems. Still have to think of a way of reliably escaping them should a user enter mods with those characters in 'em. |
For future issues / bug reports, please use the templates. They exist for a reason. 🙂 |
In the meantime, until I figure out how to reliably autoescape array-indicators like [ and ], try |
It's the space that is the issue, actually, I assumed it was the brackets.
Don't auto-escape, take the lazy way out, force quotes on each filename in
the list.
…On Wed, Aug 25, 2021 at 6:38 AM Griefed ***@***.***> wrote:
In the meantime, until I figure out how to reliably autoescape
array-indicators like [ and ], try RPG-HUD-3.8.4.jar instead of "[1.16.5][Forge]
RPG-HUD-3.8.4.jar
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#71 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AA5TJCE2FXKAJJBWZDSY5CDT6TI27ANCNFSM5CXXUTXA>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email>
.
|
Will be available in the next alpha release. 🙂 |
It won't make the server pack for me because of this issue everytime that only appears if I run it via console.
The text was updated successfully, but these errors were encountered: