[X2Go-Dev] Bug#820: Bug#820: Bug#820: Bug#820: Patch: x2goserver: x2goruncommand - set IMEXIT="true" for gnome-terminal

2015-03-19 Thread Mihai Moldovan
On 17.03.2015 05:05 PM, Mike Gabriel wrote:
> With gnome-terminal from GNOMEv2 IMEXIT needs to be false (because
> gnome-terminal (v2) does _not_ fork to background).
>
> With gnome-terminal from GNOMEv3 IMEXIT needs to be true (because
> gnome-terminal (v3) forks to background).

Stefan just "submitted" another bug report yesterday in a private
conversion about another piece of software that needs special handling
via IMEXIT -- but likewise different values dependent on the specific
version used.

I guess I should better handle that with a few new functions, to be able
to add other software and versions more flexible in the future.



Mihai

P.S.: a workaround for those issues is to use pubapps.




signature.asc
Description: OpenPGP digital signature
___
x2go-dev mailing list
x2go-dev@lists.x2go.org
http://lists.x2go.org/listinfo/x2go-dev

[X2Go-Dev] Bug#774: Bug#774: Bug#774: please set DESKTOP_SESSION and XDG_CURRENT_DESKTOP like local sessions

2015-03-19 Thread Mihai Moldovan
On 17.03.2015 08:11 PM, Ryan Tandy wrote:
> Control: tags -1 + patch
>
> Hi! Sorry it's taken me so long to get back to you about this.
>
> On Thu, Feb 05, 2015 at 11:31:05AM +, Mike Gabriel wrote:
>> On  Do 29 Jan 2015 19:17:40 CET, Ryan Tandy wrote:
>>> I can write a patch that adds all this information to x2goruncommand
>>> in the existing format. However it occurs to me that it might make
>>> more sense to parse the files in /usr/share/xsessions and use the
>>> information directly instead of duplicating it. Please let me know
>>> which you'd prefer to see.
>>
>>
>> x2goruncommand actually requires a complete rewrite (and has to be
>> based on freedesktop.org specs).
>>
>> Until that happens, let's hard-code this stuff. Please provide your
>> patch.
>
> Well, I started doing that, but it ended up being a lot less work (and
> a lot shorter code) to just take the settings from the session files. :)
>
> Please consider the attached patch. It makes x2goruncommand behave
> more like GDM/LightDM for GNOME-ish sessions.

This looks interesting, but does it work for non-Ubuntu/Debian systems,
too? Especially old(er) ones like SLE{S,D} 11, EPEL 5 etc.?



Mihai



signature.asc
Description: OpenPGP digital signature
___
x2go-dev mailing list
x2go-dev@lists.x2go.org
http://lists.x2go.org/listinfo/x2go-dev

[X2Go-Dev] Processed: X2Go issue (in src:x2goclient) has been marked as pending for release

2015-03-19 Thread X2Go Bug Tracking System
Processing commands for cont...@bugs.x2go.org:

> tag #827 pending
Bug #827 [x2goclient] hosted application doesn't support %i and %c in 
application exec param
Added tag(s) pending.
> fixed #827 4.0.4.0
Bug #827 [x2goclient] hosted application doesn't support %i and %c in 
application exec param
There is no source info for the package 'x2goclient' at version '4.0.4.0' with 
architecture ''
Unable to make a source version for version '4.0.4.0'
Marked as fixed in versions 4.0.4.0.
> thanks
Stopping processing here.

Please contact me if you need assistance.
-- 
827: http://bugs.x2go.org/cgi-bin/bugreport.cgi?bug=827
X2Go Bug Tracking System
Contact ow...@bugs.x2go.org with problems
___
x2go-dev mailing list
x2go-dev@lists.x2go.org
http://lists.x2go.org/listinfo/x2go-dev


[X2Go-Dev] Bug#827: X2Go issue (in src:x2goclient) has been marked as pending for release

2015-03-19 Thread Mihai Moldovan
tag #827 pending
fixed #827 4.0.4.0
thanks

Hello,

X2Go issue #827 (src:x2goclient) reported by you has been
fixed in X2Go Git. You can see the changelog below, and you can
check the diff of the fix at:

http://code.x2go.org/gitweb?p=x2goclient.git;a=commitdiff;h=3d57875

The issue will most likely be fixed in src:x2goclient (4.0.4.0).

light+love
X2Go Git Admin (on behalf of the sender of this mail)

---
commit 3d578752ed4de8f03673d22181952e798f4484ba
Author: Jason Alavaliant 
Date:   Fri Mar 20 05:37:38 2015 +0100

onmainwindow.cpp: handle %i and %c format flags in desktop files correctly: 
remove the %i flag and replace %c with the application name. Fixes: #827.

diff --git a/debian/changelog b/debian/changelog
index db1aefd..f14053e 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -254,6 +254,12 @@ x2goclient (4.0.4.0-0x2go1) UNRELEASED; urgency=low
   * New upstream release (4.0.4.0):
 - onmainwindow.cpp: fix desktop sharing via session broker. Fixes: #584.
 
+  [ Jason Alavaliant ]
+  * New upstream release (4.0.4.0):
+- onmainwindow.cpp: handle %i and %c format flags in desktop files
+  correctly: remove the %i flag and replace %c with the application name.
+  Fixes: #827.
+
  -- X2Go Release Manager   Thu, 19 Feb 2015 13:25:28 +0100
 
 x2goclient (4.0.3.2-0x2go1) unstable; urgency=medium
___
x2go-dev mailing list
x2go-dev@lists.x2go.org
http://lists.x2go.org/listinfo/x2go-dev


[X2Go-Dev] Bug#827: hosted application doesn't support %i and %c in application exec param

2015-03-19 Thread Jason Alavaliant

Package: x2goclient
Version: 4.0.3.2
Tags: patch

kde based apps on my machines have Exec lines in their desktop files 
along the line of


Exec=okular %U %i -caption %c

when launched via the hosted applications mode applications menu of 
x2goclient the "%i -caption %c" get passed as arguments to the command 
directly. That results in the program trying to load the file '%i' 
(and naturally failing) and setting the window caption to '%c'


The attached patch updates the x2go client to removes the %i param and 
replaces %c with the application name so the program is titled correctly.


Thanks
Jason
--- old/onmainwindow.cpp	2015-03-18 20:36:52.771965135 +1300
+++ new/onmainwindow.cpp	2015-03-18 20:40:31.199962491 +1300
@@ -6442,6 +6442,8 @@
 app.exec.replace(" %u","",Qt::CaseInsensitive);
 app.exec.replace("%f","",Qt::CaseInsensitive);
 app.exec.replace("%u","",Qt::CaseInsensitive);
+app.exec.replace("%i","",Qt::CaseInsensitive);
+app.exec.replace("%c",app.name,Qt::CaseInsensitive);
 if (app.exec==autostartApp)
 startAppFound=true;
 // x2goDebug<<"exec: "<___
x2go-dev mailing list
x2go-dev@lists.x2go.org
http://lists.x2go.org/listinfo/x2go-dev