--- Begin Message ---
Package: release.debian.org
Severity: normal
User: release.debian.org@packages.debian.org
Usertags: unblock
This is a pre-upload approval for transition of x2goclient src:package
3.99.2.1-2 to wheezy. The package has not been uploaded, yet.
The upload closes two RC bugs (grave) and closes two other important bugs.
The upload also fixes the hardening compiler/linker options in the project's
main Makefile.
As bonus an example file has been added that shows the usage of x2goplugin:
/debian/examples/x2goplugin.html.
Please let me know if an upload x2goclient 3.99.2.1-2 will get an unblock exception
for wheezy.
Thanks to all,
Mike
diff -Nru x2goclient-3.99.2.1/debian/changelog x2goclient-3.99.2.1/debian/changelog
--- x2goclient-3.99.2.1/debian/changelog 2012-06-12 00:15:20.000000000 +0200
+++ x2goclient-3.99.2.1/debian/changelog 2012-12-22 21:37:25.000000000 +0100
@@ -1,3 +1,22 @@
+x2goclient (3.99.2.1-2) unstable; urgency=low
+
+ * /debian/copyright:
+ + Set copyright format to v1.0.
+ * /debian/control:
+ + Orthographical and grammatical fixes in description texts.
+ * Fix patch: 001_hardening-x2goclient.patch, provide CPPFLAGS for
+ QMAKE_CFLAGS, as well. Provide CPPFLAGS first.
+ * Add patches: 004_allow-login-to-non-sh-users.patch and
+ 005_xinerama-for-non-sh-users.patch. Fix x2goclient for users that use
+ a non-sh shell (e.g. like tcsh) on the X2Go Server. (Closes: #682839).
+ * Add patch: 006_fix-x2goplugin.patch. Fixes host name resolving when
+ client is run in X2Go Plugin mode. Fixes session crashes directly
+ after session startup. Let the plugin tolerate whitespace at the beginning
+ of session profile configs. (Closes: #692555, #696524, #696529).
+ * Add example file: x2goplugin.html.
+
+ -- Mike Gabriel <mike.gabriel@das-netzwerkteam.de> Sat, 22 Dec 2012 21:36:55 +0100
+
x2goclient (3.99.2.1-1) unstable; urgency=low
* New upstream release.
diff -Nru x2goclient-3.99.2.1/debian/control x2goclient-3.99.2.1/debian/control
--- x2goclient-3.99.2.1/debian/control 2012-06-08 21:59:24.000000000 +0200
+++ x2goclient-3.99.2.1/debian/control 2012-12-17 23:46:02.000000000 +0100
@@ -34,9 +34,10 @@
Description: X2Go Client application (Qt4)
X2Go is a serverbased computing environment with
- session resuming
- - low bandwith support
+ - low bandwidth support
- LDAP support
- - client side mass storage mounting support
+ - client-side mass storage mounting support
+ - client-side printing support
- audio support
- authentication by smartcard and USB stick
.
@@ -58,9 +59,10 @@
Description: X2Go Client (Qt4) as browser plugin
X2Go is a serverbased computing environment with
- session resuming
- - low bandwith support
+ - low bandwidth support
- LDAP support
- - client side mass storage mounting support
+ - client-side mass storage mounting support
+ - client-side printing support
- audio support
- authentication by smartcard and USB stick
.
diff -Nru x2goclient-3.99.2.1/debian/copyright x2goclient-3.99.2.1/debian/copyright
--- x2goclient-3.99.2.1/debian/copyright 2012-06-09 14:28:48.000000000 +0200
+++ x2goclient-3.99.2.1/debian/copyright 2012-08-17 14:39:35.000000000 +0200
@@ -1,4 +1,4 @@
-Format: http://svn.debian.org/wsvn/dep/web/deps/dep5.mdwn?op=file&rev=166
+Format: http://www.debian.org/doc/packaging-manuals/copyright-format/1.0/
Upstream-Name: x2goclient
Upstream-Contact: Oleksandr Shneyder <oleksandr.shneyder@obviously-nice.de>
Source: http://wiki.x2go.org
diff -Nru x2goclient-3.99.2.1/debian/examples/x2goplugin.html x2goclient-3.99.2.1/debian/examples/x2goplugin.html
--- x2goclient-3.99.2.1/debian/examples/x2goplugin.html 1970-01-01 01:00:00.000000000 +0100
+++ x2goclient-3.99.2.1/debian/examples/x2goplugin.html 2012-12-22 14:20:14.000000000 +0100
@@ -0,0 +1,47 @@
+<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
+<html>
+ <head>
+ <title>X2Go Application Service</title>
+ </head>
+ <body onload="checkPlugin()">
+ <div id="x2goplugin">
+ <object
+ src="location"
+ type="application/x2go"
+ name="x2goplugin"
+ palette="background"
+ height="100%"
+ hspace="0"
+ vspace="0"
+ width="100%"
+ x2goconfig="
+ session=X2Go-Demo-Session
+ server=localhost
+ user=
+ sshport=22
+ command=KDE
+ rootless=false
+ sound=true
+ exportfs=true
+ speed=adsl
+ compression=16m-jpeg
+ quality=9
+ dpi=120
+ kbdlayout=
+ kbdtype=auto
+ showtoolbar=true
+ showstatusbar=true
+ usekbd=true
+ soundsystem=pulse
+ soundtunnel=true
+ defsndport=true
+ print=true
+ rootless=false
+ published=false
+ applications=WWWBROWSER, MAILCLIENT, OFFICE, TERMINAL
+ ">
+ </object>
+
+ </div>
+ </body>
+</html>
diff -Nru x2goclient-3.99.2.1/debian/patches/001_hardening-x2goclient.patch x2goclient-3.99.2.1/debian/patches/001_hardening-x2goclient.patch
--- x2goclient-3.99.2.1/debian/patches/001_hardening-x2goclient.patch 2012-06-08 21:59:24.000000000 +0200
+++ x2goclient-3.99.2.1/debian/patches/001_hardening-x2goclient.patch 2012-06-25 17:37:25.000000000 +0200
@@ -7,13 +7,13 @@
build_client:
lrelease x2goclient.pro
- mkdir -p $(CLIENT_DIR) && cd $(CLIENT_DIR) && qmake-qt4 ../x2goclient.pro
-+ mkdir -p $(CLIENT_DIR) && cd $(CLIENT_DIR) && qmake-qt4 QMAKE_CFLAGS="${CFLAGS}" QMAKE_CXXFLAGS="${CXXFLAGS} ${CPPFLAGS}" QMAKE_LFLAGS="${LDFLAGS}" ../x2goclient.pro
++ mkdir -p $(CLIENT_DIR) && cd $(CLIENT_DIR) && qmake-qt4 QMAKE_CFLAGS="${CPPFLAGS} ${CFLAGS}" QMAKE_CXXFLAGS="${CPPFLAGS} ${CXXFLAGS}" QMAKE_LFLAGS="${LDFLAGS}" ../x2goclient.pro
cd $(CLIENT_DIR) && $(MAKE)
build_plugin:
lrelease x2goclient.pro
- mkdir -p $(PLUGIN_DIR) && cd $(PLUGIN_DIR) && X2GO_CLIENT_TARGET=plugin qmake-qt4 ../x2goclient.pro
-+ mkdir -p $(PLUGIN_DIR) && cd $(PLUGIN_DIR) && X2GO_CLIENT_TARGET=plugin qmake-qt4 QMAKE_CFLAGS="${CFLAGS}" QMAKE_CXXFLAGS="${CXXFLAGS} ${CPPFLAGS}" QMAKE_LFLAGS="${LDFLAGS}" ../x2goclient.pro
++ mkdir -p $(PLUGIN_DIR) && cd $(PLUGIN_DIR) && X2GO_CLIENT_TARGET=plugin qmake-qt4 QMAKE_CFLAGS="${CPPFLAGS} ${CFLAGS}" QMAKE_CXXFLAGS="${CPPFLAGS} ${CXXFLAGS}" QMAKE_LFLAGS="${LDFLAGS}" ../x2goclient.pro
cd $(PLUGIN_DIR) && $(MAKE)
build_man:
diff -Nru x2goclient-3.99.2.1/debian/patches/004_allow-login-to-non-sh-users.patch x2goclient-3.99.2.1/debian/patches/004_allow-login-to-non-sh-users.patch
--- x2goclient-3.99.2.1/debian/patches/004_allow-login-to-non-sh-users.patch 1970-01-01 01:00:00.000000000 +0100
+++ x2goclient-3.99.2.1/debian/patches/004_allow-login-to-non-sh-users.patch 2012-12-16 12:13:58.000000000 +0100
@@ -0,0 +1,71 @@
+Description: Allow x2goclient to connect to user accounts that have other shells than /bin/sh...
+Author: Mike Gabriel <mike.gabriel@das-netzwerkteam.de>
+Date: Thu, 26 Jul 2012 09:06:53 +0000 (+0200)
+Source: http://code.x2go.org/gitweb?p=x2goclient.git;a=commitdiff_plain;h=5e5eb0cfffc514ee419a116a78f25ff98728c746
+Abstract:
+ Allow x2goclient to connect to user accounts that have other shells than
+ /bin/sh and alike configured as default shell. Also: removal of bashisms in
+ shell execution commands.
+
+--- a/onmainwindow.cpp
++++ b/onmainwindow.cpp
+@@ -5505,7 +5505,7 @@
+ resumingSession.agentPid + " " +
+ resumingSession.sessionId+" "+
+ resumingSession.sndPort+ " "+ command+" nosnd "+
+- sessionType +">& /dev/null & exit";
++ sessionType +" 1> /dev/null 2>/dev/null & exit";
+ if ( startSessSndSystem ==PULSE )
+ {
+ cmd="PULSE_CLIENTCONFIG=~/.x2go/C-"+
+@@ -5524,7 +5524,7 @@
+ resumingSession.sessionId+" "+
+ resumingSession.sndPort+ " "+
+ command+" esd "+
+- sessionType +">& /dev/null & exit";
++ sessionType +" 1> /dev/null 2>/dev/null & exit";
+ break;
+ case ARTS:
+ cmd="setsid x2goruncommand "+
+@@ -5533,7 +5533,7 @@
+ resumingSession.sessionId+" "+
+ resumingSession.sndPort+ " "+
+ command+" arts "+
+- sessionType +">& /dev/null & exit";
++ sessionType +" 1> /dev/null 2>/dev/null & exit";
+ break;
+
+ }
+@@ -5563,7 +5563,7 @@
+ proc->startNormal ("PULSE_CLIENTCONFIG=~/.x2go/C-"+
+ resumingSession.sessionId+"/.pulse-client.conf DISPLAY=:"+
+ resumingSession.display+
+- " setsid "+exec+">& /dev/null & exit");
++ " setsid "+exec+" 1> /dev/null 2>/dev/null & exit");
+ }
+
+ void ONMainWindow::slotRetRunCommand ( bool result, QString output,
+@@ -10265,7 +10265,7 @@
+ QString scmd="PULSE_CLIENTCONFIG=~/.x2go/C-"+
+ resumingSession.sessionId+
+ "/.pulse-client.conf "+
+- "parec > /dev/null &sleep 1 && kill %1";
++ "parec 1> /dev/null & sleep 1 && kill %1";
+
+ paProc=new SshProcess ( sshConnection, this );
+ paProc->startNormal ( scmd );
+--- a/sshprocess.cpp
++++ b/sshprocess.cpp
+@@ -130,7 +130,11 @@
+
+ void SshProcess::startNormal(const QString& cmd)
+ {
+- masterCon->addChannelConnection(this, cmd);
++ QString shcmd = "sh -c \""+cmd+"\"";
++// #ifdef DEBUG
++ x2goDebug<<"executing remote command: "<<shcmd<<endl;
++// #endif
++ masterCon->addChannelConnection(this, shcmd);
+ connect(masterCon,SIGNAL(stdOut(SshProcess*,QByteArray)),this,SLOT(slotStdOut(SshProcess*,QByteArray)));
+ connect(masterCon,SIGNAL(channelClosed(SshProcess*)), this,SLOT(slotChannelClosed(SshProcess*)));
+ }
diff -Nru x2goclient-3.99.2.1/debian/patches/005_xinerama-for-non-sh-users.patch x2goclient-3.99.2.1/debian/patches/005_xinerama-for-non-sh-users.patch
--- x2goclient-3.99.2.1/debian/patches/005_xinerama-for-non-sh-users.patch 1970-01-01 01:00:00.000000000 +0100
+++ x2goclient-3.99.2.1/debian/patches/005_xinerama-for-non-sh-users.patch 2012-12-16 12:20:39.000000000 +0100
@@ -0,0 +1,23 @@
+Description: Use ,,printf'' instead of ,,echo -e'' (Bashism). Fixes faulty creation of xinerama.conf files.
+Author: Mike Gabriel <mike.gabriel@das-netzwerkteam.de>
+Date: Thu, 11 Oct 2012 13:10:48 +0000 (+0200)
+Source: http://code.x2go.org/gitweb?p=x2goclient.git;a=commitdiff_plain;h=cae4f4d7dbacc720ee1e064441af82667b4f040f
+Abstract:
+ Use ,,printf'' instead of ,,echo -e'' (Bashism) when creating xinerama.conf
+ files on X2Go Server. Fixes occurences of faulty xinerama.conf files. Without
+ this patch, the xinerama.conf files do contain \n instead of real new lines
+ at EOL and are misinterpreted by x2goagent.
+ .
+ This patch is a follow-up fix for 004_allow-login-to-non-sh-users.patch.
+
+--- a/onmainwindow.cpp
++++ b/onmainwindow.cpp
+@@ -9118,7 +9118,7 @@
+ foreach (QRect disp, xineramaScreens)
+ screens<<QString::number(disp.x())+" "+QString::number(disp.y())+" "+QString::number(disp.width())+
+ " "+QString::number(disp.height());
+- QString cmd="export DISPLAY=:"+resumingSession.display+";echo -e "+screens.join("\\\\n")+" > ~/.x2go/C-"+
++ QString cmd="export DISPLAY=:"+resumingSession.display+";printf "+screens.join("\\\\n")+" > ~/.x2go/C-"+
+ resumingSession.sessionId+"/xinerama.conf";
+
+ x2goDebug<<cmd<<endl;
diff -Nru x2goclient-3.99.2.1/debian/patches/006_fix-x2goplugin.patch x2goclient-3.99.2.1/debian/patches/006_fix-x2goplugin.patch
--- x2goclient-3.99.2.1/debian/patches/006_fix-x2goplugin.patch 1970-01-01 01:00:00.000000000 +0100
+++ x2goclient-3.99.2.1/debian/patches/006_fix-x2goplugin.patch 2012-12-22 11:59:02.000000000 +0100
@@ -0,0 +1,51 @@
+Description: Fix issues in x2goplugin that make the plugin unusable.
+Author: Oleksandr Shneyder <o.shneyder@phoca-gmbh.de>
+Date: Wed, 7 Nov 2012 09:36:40 +0000 (+0100)
+Source:
+ http://code.x2go.org/gitweb?p=x2goclient.git;a=commitdiff_plain;h=f1fb104bc7ce1a3d93792b3594566a8870d139a3
+ http://code.x2go.org/gitweb?p=x2goclient.git;a=commitdiff;h=d4d0fe38610d3e514dc2e4e259ce8f103e4ac535
+ http://code.x2go.org/gitweb?p=x2goclient.git;a=commitdiff;h=9aa8514c8a39b4c144fede0e8369962a618f9ddf
+Abstract:
+ - Fix error "Failed to resolve hostname" when X2Go Client runs in browser
+ plugin mode.
+ - Fix crash of the plugin container at session startup.
+ - Tolerate whitespaces at the beginning of session configs provided via
+ html pages.
+
+--- a/onmainwindow.cpp
++++ b/onmainwindow.cpp
+@@ -3030,8 +3030,11 @@
+ X2goSettings st ( "sessions" );
+
+ passForm->setEnabled ( false );
+- host=st.setting()->value ( sid+"/host",
+- ( QVariant ) QString::null ).toString();
++ if(!embedMode)
++ host=st.setting()->value ( sid+"/host",
++ ( QVariant ) QString::null ).toString();
++ else
++ host=config.server;
+ QString cmd=st.setting()->value ( sid+"/command",
+ ( QVariant ) QString::null ).toString();
+ autologin=st.setting()->value ( sid+"/autologin",
+@@ -8944,6 +8947,8 @@
+
+ void ONMainWindow::setProxyWinTitle()
+ {
++ if(embedMode)
++ return;
+
+ QString title;
+
+@@ -9364,6 +9369,11 @@
+ for ( int i=0;i<lines.count();++i )
+ {
+ QString line = lines[i];
++
++ // strip left/right whitespaces, important for plugin settings via x2goconfig
++ line.remove(QRegExp("^\\s+"));
++ line.remove(QRegExp("\\s+$"));
++
+ if ( ( line=="-----BEGIN DSA PRIVATE KEY-----" ) ||
+ ( line=="-----BEGIN RSA PRIVATE KEY-----" ) )
+ {
diff -Nru x2goclient-3.99.2.1/debian/patches/series x2goclient-3.99.2.1/debian/patches/series
--- x2goclient-3.99.2.1/debian/patches/series 2012-06-08 21:59:24.000000000 +0200
+++ x2goclient-3.99.2.1/debian/patches/series 2012-12-22 12:02:48.000000000 +0100
@@ -1,3 +1,6 @@
001_hardening-x2goclient.patch
002_remove-encoding-from-desktop-entry.patch
003_fix-spelling-error-authentication.patch
+004_allow-login-to-non-sh-users.patch
+005_xinerama-for-non-sh-users.patch
+006_fix-x2goplugin.patch
diff -Nru x2goclient-3.99.2.1/debian/x2goplugin.examples x2goclient-3.99.2.1/debian/x2goplugin.examples
--- x2goclient-3.99.2.1/debian/x2goplugin.examples 1970-01-01 01:00:00.000000000 +0100
+++ x2goclient-3.99.2.1/debian/x2goplugin.examples 2012-12-22 14:20:33.000000000 +0100
@@ -0,0 +1 @@
+debian/examples/*
-- System Information:
Debian Release: 6.0.6
APT prefers stable-updates
APT policy: (500, 'stable-updates'), (500, 'proposed-updates'), (500, 'stable')
Architecture: amd64 (x86_64)
Kernel: Linux 2.6.32-5-amd64 (SMP w/4 CPU cores)
Locale: LANG=de_DE.UTF-8, LC_CTYPE=de_DE.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
--- End Message ---