Package: release.debian.org
Severity: normal
User: release.debian....@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.gabr...@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.shney...@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.gabr...@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.gabr...@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.shney...@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


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to