[education/rkward] rkward/rbackend: First testing with the new handshake workaround looks promising. Clean up some.

Thomas Friedrichsmeier null at kde.org
Tue May 17 14:42:46 BST 2022


Git commit 6dbdc1fc2fceea3e0a28357ff769448c666735e1 by Thomas Friedrichsmeier.
Committed on 17/05/2022 at 13:42.
Pushed by tfry into branch 'master'.

First testing with the new handshake workaround looks promising. Clean up some.

M  +2    -4    rkward/rbackend/rkfrontendtransmitter.cpp

https://invent.kde.org/education/rkward/commit/6dbdc1fc2fceea3e0a28357ff769448c666735e1

diff --git a/rkward/rbackend/rkfrontendtransmitter.cpp b/rkward/rbackend/rkfrontendtransmitter.cpp
index 32a52ff2..daadc64c 100644
--- a/rkward/rbackend/rkfrontendtransmitter.cpp
+++ b/rkward/rbackend/rkfrontendtransmitter.cpp
@@ -136,12 +136,10 @@ void RKFrontendTransmitter::run () {
 	// Could it help to start a dummy process, before that? And, if doing so, will we be able to read its output?
 	QProcess dummy;
 	QStringList dummyargs = args;
-	dummyargs.removeAt(3); // the --server-name. With this empty, the backend will exit
-	qDebug("%s", qPrintable(dummyargs.join("\n")));
+	dummyargs.removeAt(dummyargs.size()-4); // the --server-name. With this empty, the backend will exit
 	dummy.start(RKSessionVars::RBinary(), dummyargs, QIODevice::ReadOnly);
 	dummy.waitForFinished();
-	QString r_home = QString::fromLocal8Bit(dummy.readAllStandardOutput());
-	RK_DEBUG(RBACKEND, DL_WARNING, "This debug message is not meant to stay. String: %s", qPrintable(r_home));
+	dummy.readAllStandardOutput();
 #endif
 	backend->start(RKSessionVars::RBinary(), args, QIODevice::ReadOnly);
 


More information about the rkward-tracker mailing list