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
Report from testing the Beta Program version 3.5.0.1_B48 #3226
Comments
Beta versions do automatically enable I2P - disable it in the plugin settings (don't uninstall it, it will re-install) "Unable to make field private static transient java.net.InetAddress$NameService" is a known error, the latest unix launch script has the required "add opens" command line parameters to remove it "A terminally deprecated method in java.lang.System has been called" etc is a known issue. I very much doubt that there is actually 1Mib of all "0 bytes" |
OK. Did another experiment, split the redirects of stdout and stderr:
... it continues on and on. The message seems to be repeated (there are some newlines at first) but the array gradually grows to larger and larger sizes. |
There is, you can check my test procedure:
No output produced. |
Also the whole old log file compresses (gzip default opts) to just 848KB. and what I posted is the complete |
I don't see any zero bytes so no idea. I'll reduce that logging |
Looks like startup script (the On my system
...so the Also, may I suggest you start using the excellent shellcheck tool? Shell scripting is quite error prone and I already see a few warnings about unused variables etc. |
nah, I don't do linux |
Shellcheck is multiplatform, there are integrations into vsCode etc. and you can also check your script online by just pasting it into "https://www.shellcheck.net/".
to
... would do the trick? Let me read the whole script and report back. |
wtf is java --version outputting that first line (Picked up _JAVA_OPTIONS: -Dawt.useSystemAAFontSettings=on -Dswing.aatext=true) for? Any changes have to deal with all the JRE/JDKs out there... |
Probably have to ask RedHat about that
Any official listing of all of those? ... as you might have noticed, I'm not really a Java person |
Yeah. I think the issue might be the merge of stdout and stderr there. The |
The third answer there warns about this and uses grep instead. |
so just pick up stdout and ignore stderr and use the existing code? |
I suspect the redirect is there for a reason for some specific JDK versions that just outputs the version to stderr.
should do the trick (the sed command bellow assumes the presence of the version keyword anyway so it shouldn't break anything), but I have a question: |
Yup, fail -> run it anways |
OK. Less immediate breakage. Would probably like to output a warning / error message to stderr anyway so user can report if it breaks for some future JDK version again. |
OK. So maybe like this: check_version()
{
JAVA_ISGCJ=$($JAVABIN -version 2>&1 | grep "gcj")
if [ ! "$JAVA_ISGCJ x" = " x" ] ; then
echo "$MSG_ISGCJ"
return 1
fi
JAVA_HEADER=$("$JAVABIN" -version 2>&1 | grep -i version | head -n 1)
if [ "$JAVA_HEADER" = "" ] ; then
echo "ERROR: Failed to detect JAVA version" >&2
fi
JAVA_IMPL=$(echo "${JAVA_HEADER}" | cut -f1 -d' ')
if [ "$JAVA_IMPL" = "#" ] ; then
echo "$MSG_JAVABORK"
$JAVABIN -version 2>&1
exit 1
else
JAVA_VERSION=$(echo "${JAVA_HEADER}" | sed "s/^.* version \"\(.*\)\".*$/\1/")
if echo "$JAVA_VERSION" | grep "^1.[0-7]" ; then
echo "$MSG_JAVA_TOO_OLD" "[$JAVABIN = ${JAVA_VERSION}]" ; echo "$MSG_JAVA_BELOW_MIN"
return 1
else
echo "${MSG5} [$JAVABIN = ${JAVA_IMPL} ${JAVA_VERSION}]"
return 0
fi
fi
} You can move the new error message into a variable up top maintain the style |
Running on this adjusted script now, successfully getting the '--add-opens' option. Still getting a huge stdout, but at least now I run with options for my particular JDK. The "Unable to make field private static transient " exception is gone. |
So to summarize: |
B49 has reduced debug and fixed startup script |
Few issues I noticed so far while testing the latest Beta release:
strings dump from the log file (what was left of it).:
It didn't crash. I shut it down manually.
Aside:
The text was updated successfully, but these errors were encountered: