From owner-svn-ports-all@FreeBSD.ORG Sat Feb 21 21:08:26 2015 Return-Path: Delivered-To: svn-ports-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id A9CEDF36; Sat, 21 Feb 2015 21:08:26 +0000 (UTC) Received: from svn.freebsd.org (svn.freebsd.org [IPv6:2001:1900:2254:2068::e6a:0]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 8A628833; Sat, 21 Feb 2015 21:08:26 +0000 (UTC) Received: from svn.freebsd.org ([127.0.1.70]) by svn.freebsd.org (8.14.9/8.14.9) with ESMTP id t1LL8QtK071382; Sat, 21 Feb 2015 21:08:26 GMT (envelope-from makc@FreeBSD.org) Received: (from makc@localhost) by svn.freebsd.org (8.14.9/8.14.9/Submit) id t1LL8Qkf071380; Sat, 21 Feb 2015 21:08:26 GMT (envelope-from makc@FreeBSD.org) Message-Id: <201502212108.t1LL8Qkf071380@svn.freebsd.org> X-Authentication-Warning: svn.freebsd.org: makc set sender to makc@FreeBSD.org using -f From: Max Brazhnikov Date: Sat, 21 Feb 2015 21:08:26 +0000 (UTC) To: ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: svn commit: r379551 - in head/net/krdc: . files X-SVN-Group: ports-head MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-BeenThere: svn-ports-all@freebsd.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: SVN commit messages for the ports tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 21 Feb 2015 21:08:26 -0000 Author: makc Date: Sat Feb 21 21:08:25 2015 New Revision: 379551 URL: https://svnweb.freebsd.org/changeset/ports/379551 QAT: https://qat.redports.org/buildarchive/r379551/ Log: net/krdc: - Add patch from upstream for FreeRDP 1.1+ support PR: 197374 Reported by: amvandemore@gmail.com Added: head/net/krdc/files/patch-git8436cc62 (contents, props changed) Modified: head/net/krdc/Makefile Modified: head/net/krdc/Makefile ============================================================================== --- head/net/krdc/Makefile Sat Feb 21 20:54:04 2015 (r379550) +++ head/net/krdc/Makefile Sat Feb 21 21:08:25 2015 (r379551) @@ -2,6 +2,7 @@ PORTNAME= krdc PORTVERSION= ${KDE4_VERSION} +PORTREVISION= 1 CATEGORIES= net kde MASTER_SITES= KDE/${KDE4_BRANCH}/${PORTVERSION}/src DIST_SUBDIR= KDE/${PORTVERSION} Added: head/net/krdc/files/patch-git8436cc62 ============================================================================== --- /dev/null 00:00:00 1970 (empty, because file is newly added) +++ head/net/krdc/files/patch-git8436cc62 Sat Feb 21 21:08:25 2015 (r379551) @@ -0,0 +1,317 @@ +--- rdp/rdpview.cpp ++++ rdp/rdpview.cpp +@@ -146,102 +146,205 @@ bool RdpView::start() + } + } + ++ // Check the version of FreeRDP so we can use pre-1.1 switches if needed ++ QProcess *xfreeRDPVersionCheck = new QProcess(this); ++ xfreeRDPVersionCheck->start("xfreerdp", QStringList("--version")); ++ xfreeRDPVersionCheck->waitForFinished(); ++ QString versionOutput = xfreeRDPVersionCheck->readAllStandardOutput(); ++ xfreeRDPVersionCheck->deleteLater(); ++ + m_process = new QProcess(m_container); + + QStringList arguments; + +- int width, height; +- if (m_hostPreferences->width() > 0) { +- width = m_hostPreferences->width(); +- height = m_hostPreferences->height(); +- } else { +- width = this->parentWidget()->size().width(); +- height = this->parentWidget()->size().height(); +- } +- arguments << "-g" << QString::number(width) + 'x' + QString::number(height); +- +- arguments << "-k" << keymapToXfreerdp(m_hostPreferences->keyboardLayout()); ++ if (versionOutput.contains(QLatin1String(" 1.0"))) { ++ kDebug(5012) << "Use FreeRDP 1.0 compatible arguments"; + +- if (!m_url.userName().isEmpty()) { +- // if username contains a domain, it needs to be set with another parameter +- if (m_url.userName().contains('\\')) { +- const QStringList splittedName = m_url.userName().split('\\'); +- arguments << "-d" << splittedName.at(0); +- arguments << "-u" << splittedName.at(1); ++ int width, height; ++ if (m_hostPreferences->width() > 0) { ++ width = m_hostPreferences->width(); ++ height = m_hostPreferences->height(); + } else { +- arguments << "-u" << m_url.userName(); ++ width = this->parentWidget()->size().width(); ++ height = this->parentWidget()->size().height(); + } ++ arguments << "-g" << QString::number(width) + 'x' + QString::number(height); ++ ++ arguments << "-k" << keymapToXfreerdp(m_hostPreferences->keyboardLayout()); ++ ++ if (!m_url.userName().isEmpty()) { ++ // if username contains a domain, it needs to be set with another parameter ++ if (m_url.userName().contains('\\')) { ++ const QStringList splittedName = m_url.userName().split('\\'); ++ arguments << "-d" << splittedName.at(0); ++ arguments << "-u" << splittedName.at(1); ++ } else { ++ arguments << "-u" << m_url.userName(); ++ } ++ } else { ++ arguments << "-u" << ""; ++ } ++ ++ arguments << "-D"; // request the window has no decorations ++ arguments << "-X" << QString::number(m_container->winId()); ++ arguments << "-a" << QString::number((m_hostPreferences->colorDepth() + 1) * 8); ++ ++ switch (m_hostPreferences->sound()) { ++ case 1: ++ arguments << "-o"; ++ break; ++ case 0: ++ arguments << "--plugin" << "rdpsnd"; ++ break; ++ case 2: ++ default: ++ break; ++ } ++ ++ if (!m_hostPreferences->shareMedia().isEmpty()) { ++ QStringList shareMedia; ++ shareMedia << "--plugin" << "rdpdr" << "--data" << "disk:media:" + m_hostPreferences->shareMedia() << "--"; ++ arguments += shareMedia; ++ } ++ ++ QString performance; ++ switch (m_hostPreferences->performance()) { ++ case 0: ++ performance = 'm'; ++ break; ++ case 1: ++ performance = 'b'; ++ break; ++ case 2: ++ performance = 'l'; ++ break; ++ default: ++ break; ++ } ++ ++ arguments << "-x" << performance; ++ ++ if (m_hostPreferences->console()) { ++ arguments << "-0"; ++ } ++ ++ if (m_hostPreferences->remoteFX()) { ++ arguments << "--rfx"; ++ } ++ ++ if (!m_hostPreferences->extraOptions().isEmpty()) { ++ const QStringList additionalArguments = KShell::splitArgs(m_hostPreferences->extraOptions()); ++ arguments += additionalArguments; ++ } ++ ++ // krdc has no support for certificate management yet; it would not be possbile to connect to any host: ++ // "The host key for example.com has changed" ... ++ // "Add correct host key in ~/.freerdp/known_hosts to get rid of this message." ++ arguments << "--ignore-certificate"; ++ ++ // clipboard sharing is activated in KRDC; user can disable it at runtime ++ arguments << "--plugin" << "cliprdr"; ++ ++ arguments << "-t" << QString::number(m_port); ++ arguments << m_host; ++ ++ kDebug(5012) << "Starting xfreerdp with arguments: " << arguments.join(" "); ++ ++ arguments.removeLast(); // host must be last, remove and re-add it after the password ++ if (!m_url.password().isNull()) ++ arguments << "-p" << m_url.password(); ++ arguments << m_host; ++ + } else { +- arguments << "-u" << ""; +- } ++ kDebug(5012) << "Use FreeRDP 1.1+ compatible arguments"; + +- if (!m_url.password().isNull()) +- arguments << "-p" << m_url.password(); +- +- arguments << "-D"; // request the window has no decorations +- arguments << "-X" << QString::number(m_container->winId()); +- arguments << "-a" << QString::number((m_hostPreferences->colorDepth() + 1) * 8); +- +- switch (m_hostPreferences->sound()) { +- case 1: +- arguments << "-o"; +- break; +- case 0: +- arguments << "--plugin" << "rdpsnd"; +- break; +- case 2: +- default: +- break; +- } ++ int width, height; ++ if (m_hostPreferences->width() > 0) { ++ width = m_hostPreferences->width(); ++ height = m_hostPreferences->height(); ++ } else { ++ width = this->parentWidget()->size().width(); ++ height = this->parentWidget()->size().height(); ++ } ++ arguments << "-decorations"; ++ arguments << "/w:" + QString::number(width); ++ arguments << "/h:" + QString::number(height); ++ ++ arguments << "/kbd:" + keymapToXfreerdp(m_hostPreferences->keyboardLayout()); ++ ++ if (!m_url.userName().isEmpty()) { ++ // if username contains a domain, it needs to be set with another parameter ++ if (m_url.userName().contains('\\')) { ++ const QStringList splittedName = m_url.userName().split('\\'); ++ arguments << "/d:" + splittedName.at(0); ++ arguments << "/u:" + splittedName.at(1); ++ } else { ++ arguments << "/u:" + m_url.userName(); ++ } ++ } else { ++ arguments << "/u:"; ++ } + +- if (!m_hostPreferences->shareMedia().isEmpty()) { +- QStringList shareMedia; +- shareMedia << "--plugin" << "rdpdr" << "--data" << "disk:media:" + m_hostPreferences->shareMedia() << "--"; +- arguments += shareMedia; +- } ++ arguments << "/parent-window:" + QString::number(m_container->winId()); ++ arguments << "/bpp:" + QString::number((m_hostPreferences->colorDepth() + 1) * 8); ++ arguments << "/audio-mode:" + QString::number(m_hostPreferences->sound()); + +- QString performance; +- switch (m_hostPreferences->performance()) { +- case 0: +- performance = 'm'; +- break; +- case 1: +- performance = 'b'; +- break; +- case 2: +- performance = 'l'; +- break; +- default: +- break; +- } ++ if (!m_hostPreferences->shareMedia().isEmpty()) { ++ QStringList shareMedia; ++ shareMedia << "/drive:media," + m_hostPreferences->shareMedia(); ++ arguments += shareMedia; ++ } + +- arguments << "-x" << performance; ++ QString performance; ++ switch (m_hostPreferences->performance()) { ++ case 0: ++ performance = "modem"; ++ break; ++ case 1: ++ performance = "broadband"; ++ break; ++ case 2: ++ performance = "lan"; ++ break; ++ default: ++ break; ++ } + +- if (m_hostPreferences->console()) { +- arguments << "-0"; +- } ++ arguments << "/network:" + performance; + +- if (m_hostPreferences->remoteFX()) { +- arguments << "--rfx"; +- } ++ if (m_hostPreferences->console()) { ++ arguments << "/admin"; ++ } + +- if (!m_hostPreferences->extraOptions().isEmpty()) { +- const QStringList additionalArguments = KShell::splitArgs(m_hostPreferences->extraOptions()); +- arguments += additionalArguments; +- } ++ if (m_hostPreferences->remoteFX()) { ++ arguments << "/rfx"; ++ } ++ ++ if (!m_hostPreferences->extraOptions().isEmpty()) { ++ const QStringList additionalArguments = KShell::splitArgs(m_hostPreferences->extraOptions()); ++ arguments += additionalArguments; ++ } ++ ++ // krdc has no support for certificate management yet; it would not be possbile to connect to any host: ++ // "The host key for example.com has changed" ... ++ // "Add correct host key in ~/.freerdp/known_hosts to get rid of this message." ++ arguments << "/cert-ignore"; ++ ++ // clipboard sharing is activated in KRDC; user can disable it at runtime ++ arguments << "+clipboard"; + +- // krdc has no support for certificate management yet; it would not be possbile to connect to any host: +- // "The host key for example.com has changed" ... +- // "Add correct host key in ~/.freerdp/known_hosts to get rid of this message." +- arguments << "--ignore-certificate"; ++ arguments << "/port:" + QString::number(m_port); ++ arguments << "/v:" + m_host; + +- // clipboard sharing is activated in KRDC; user can disable it at runtime +- arguments << "--plugin" << "cliprdr"; ++ kDebug(5012) << "Starting xfreerdp with arguments: " << arguments.join(" "); + +- arguments << "-t" << QString::number(m_port); +- arguments << m_host; ++ //avoid printing the password in debug ++ if (!m_url.password().isNull()) { ++ arguments << "/p:" + m_url.password(); ++ } ++ kDebug(5012) << "Starting xfreerdp with arguments: " << arguments.join(" "); + +- kDebug(5012) << "Starting xfreerdp with arguments:" << arguments; ++ } + + setStatus(Connecting); + +@@ -302,7 +405,7 @@ void RdpView::connectionError() + + void RdpView::processError(QProcess::ProcessError error) + { +- kDebug(5012) << "processError:" << error; ++ kDebug(5012) << error; + if (m_quitFlag) // do not try to show error messages while quitting (prevent crashes) + return; + +@@ -319,11 +422,11 @@ void RdpView::processError(QProcess::ProcessError error) + void RdpView::receivedStandardError() + { + const QString output(m_process->readAllStandardError()); +- kDebug(5012) << "receivedStandardError:" << output; ++ kDebug(5012) << output; + QString line; + int i = 0; + while (!(line = output.section('\n', i, i)).isEmpty()) { +- ++ + // the following error is issued by freerdp because of a bug in freerdp 1.0.1 and below; + // see: https://github.com/FreeRDP/FreeRDP/pull/576 + //"X Error of failed request: BadWindow (invalid Window parameter) +@@ -345,7 +448,7 @@ void RdpView::receivedStandardError() + void RdpView::receivedStandardOutput() + { + const QString output(m_process->readAllStandardOutput()); +- kDebug(5012) << "receivedStandardOutput:" << output; ++ kDebug(5012) << output; + QString line; + int i = 0; + while (!(line = output.section('\n', i, i)).isEmpty()) {