Merge lp:~jtaylor/ubuntu/oneiric/gnome-rdp/fix-vnc-879686 into lp:ubuntu/oneiric/gnome-rdp

Proposed by Julian Taylor
Status: Approved
Approved by: Stéphane Graber
Approved revision: 15
Proposed branch: lp:~jtaylor/ubuntu/oneiric/gnome-rdp/fix-vnc-879686
Merge into: lp:ubuntu/oneiric/gnome-rdp
Diff against target: 92 lines (+73/-0)
3 files modified
debian/changelog (+9/-0)
debian/patches/fix-vnc-connections.patch (+63/-0)
debian/patches/series (+1/-0)
To merge this branch: bzr merge lp:~jtaylor/ubuntu/oneiric/gnome-rdp/fix-vnc-879686
Reviewer Review Type Date Requested Status
Luke Yelavich (community) Approve
Ubuntu branches Pending
Review via email: mp+80136@code.launchpad.net

Description of the change

SRU request in the linked bug

To post a comment you must log in.
Revision history for this message
Luke Yelavich (themuso) wrote :

Thanks for your work, uploaded with the maintainer field updated.

review: Approve
Revision history for this message
Stéphane Graber (stgraber) wrote :

Marking as approved as Luke uploaded it.

Unmerged revisions

15. By Julian Taylor

add patch to fix vnc connections (LP: #879686)
use xtightvncviewer, tight-vncviewer does not exist in debian
xtightvncviewer does not accepted -user flag and it must receive
a server to connect to.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/changelog'
--- debian/changelog 2011-06-07 17:18:42 +0000
+++ debian/changelog 2011-10-22 22:09:25 +0000
@@ -1,3 +1,12 @@
1gnome-rdp (0.3.0.9-1ubuntu0.1) oneiric-proposed; urgency=low
2
3 * add patch to fix vnc connections (LP: #879686)
4 use xtightvncviewer, tight-vncviewer does not exist in debian
5 xtightvncviewer does not accepted -user flag and it must receive
6 a server to connect to.
7
8 -- Julian Taylor <jtaylor.debian@googlemail.com> Fri, 21 Oct 2011 23:50:37 +0200
9
1gnome-rdp (0.3.0.9-1) unstable; urgency=low10gnome-rdp (0.3.0.9-1) unstable; urgency=low
211
3 * new upstream version12 * new upstream version
413
=== added file 'debian/patches/fix-vnc-connections.patch'
--- debian/patches/fix-vnc-connections.patch 1970-01-01 00:00:00 +0000
+++ debian/patches/fix-vnc-connections.patch 2011-10-22 22:09:25 +0000
@@ -0,0 +1,63 @@
1From: Julian Taylor <jtaylor.debian@googlemail.com>
2Date: Fri, 9 Sep 2011 14:45:24 +0200
3Description: fix vnc connections
4Bug-Ubuntu: https://bugs.launchpad.net/ubuntu/+source/gnome-rdp/+bug/879686
5Forwarded: james.p.michels@gmail.com
6
7use xtightvncviewer, tight-vncviewer does not exist in debian
8xtightvncviewer does not accepted -user flag and it must receive
9a server to connect to.
10---
11 Sessions/SessionCollection.cs | 4 ++--
12 Vnc/VncProfile.cs | 6 ++++--
13 2 files changed, 6 insertions(+), 4 deletions(-)
14
15diff --git a/Sessions/SessionCollection.cs b/Sessions/SessionCollection.cs
16index 2920e21..2e4a6e3 100644
17--- a/Sessions/SessionCollection.cs
18+++ b/Sessions/SessionCollection.cs
19@@ -83,7 +83,7 @@ namespace GnomeRDP.Sessions
20 string args = vncProfile.ToCommandLineArguements(session.Server, session.Identity, false);
21 string safeArgs = vncProfile.ToCommandLineArguements(session.Server, session.Identity, true);
22
23- new SessionHost("tight-vncviewer", args, safeArgs);
24+ new SessionHost("xtightvncviewer", args, safeArgs);
25 }
26
27 if (session.Profile is Ssh.SshProfile)
28@@ -96,4 +96,4 @@ namespace GnomeRDP.Sessions
29 }
30 }
31 }
32-}
33\ No newline at end of file
34+}
35diff --git a/Vnc/VncProfile.cs b/Vnc/VncProfile.cs
36index e661d75..1eb0128 100644
37--- a/Vnc/VncProfile.cs
38+++ b/Vnc/VncProfile.cs
39@@ -158,7 +158,7 @@ namespace GnomeRDP.Vnc
40 StringBuilder text = new StringBuilder();
41
42 string user = (identity == null) ? null : identity.Username;
43- if (string.IsNullOrEmpty(user) == false)
44+ if (false)
45 {
46 if (user.Contains(" "))
47 {
48@@ -181,8 +181,10 @@ namespace GnomeRDP.Vnc
49 {
50 text.AppendFormat(" -encodings {0}", encoding.ToString().ToLower());
51 }
52+
53+ text.AppendFormat(" {0}", server);
54
55 return text.ToString();
56 }
57 }
58-}
59\ No newline at end of file
60+}
61--
621.7.5.4
63
064
=== modified file 'debian/patches/series'
--- debian/patches/series 2011-06-07 17:18:42 +0000
+++ debian/patches/series 2011-10-22 22:09:25 +0000
@@ -1,2 +1,3 @@
1include_ac_expansions.patch1include_ac_expansions.patch
2use_default_compiler.patch2use_default_compiler.patch
3fix-vnc-connections.patch

Subscribers

People subscribed via source and target branches

to all changes: