Merge lp:~u78qir8a9-deactivatedaccount/do-plugins/screen into lp:do-plugins

[Alex Launi]

[X (u78qir8a9-deactivatedaccount)]

  New Plugin: GNU Screen

  This is a plugin for Gnome Do to work with GNU Screen sessions. GNU Screen
  (command 'screen') is a command line application for managing multi-windowed
  shell sessions that can be detached and reattached for persistency, and offers
  much configurability.

  This adds the Plugin directory ./GNUScreen, and adds the plugin to monodevelop
  as well as to the autotools setup.

  This is the Plugin code as provided in gitorious revision 2fd1c03b at
  screen-plugin-gnome-do/mainline

  http://gitorious.org/projects/screen-plugin-gnome-do

[Alex Launi (alexlauni)]

When I enabled this plugin it blocked my console and made me pick 1, 2, or 3. I had never launched screen on my laptop before and I guess this is why, but that's a case that needs considered.

[Alex Launi (alexlauni)]

You know what would make this one of the best Do plugins ever written? The ability to find ssh sessions, and index screen sessions running on remote hosts. How beautiful would that be?

[X (u78qir8a9-deactivatedaccount)]

"When I enabled this plugin it blocked my console and made me pick 1, 2, or 3. I had never launched screen on my laptop before and I guess this is why, but that's a case that needs considered."

Whoa ok that sounds bad. What does "pick 1,2, or 3" mean?

I know a plugin as this has many open issues, like

1. default location of the binary (now "screen")
2. configurable permanent arguments(?)
3. how to find the user's terminal app of choice (this hardcoded to "gnome-terminal", I was just looking at other plugins). Perhaps that's one that is simple to sove using Unix.Process..

[Alex Launi (alexlauni)]

On Mon, Feb 9, 2009 at 5:02 AM, Ulrik Sverdrup <email address hidden>wrote:

> Whoa ok that sounds bad. What does "pick 1,2, or 3" mean?
>

I have no idea what it meant! I just chose 1 to get it to unblock...

--
--Alex Launi

[Matthew Frizelle (gleebtorin)]

"You know what would make this one of the best Do plugins ever written? The ability to find ssh sessions, and index screen sessions running on remote hosts. How beautiful would that be?"

That would actually be a pretty awesome idea. As a heavy user of screen, that would be very useful to me :)

[X (u78qir8a9-deactivatedaccount)]

2009/2/17 Matthew Frizelle <email address hidden>:
> "You know what would make this one of the best Do plugins ever written? The ability to find ssh sessions, and index screen sessions running on remote hosts. How beautiful would that be?"
>
> That would actually be a pretty awesome idea. As a heavy user of screen, that would be very useful to me :)
> --
> https://code.launchpad.net/~ulrik-sverdrup/do-plugins/screen/+merge/3476
> You are subscribed to branch lp:~ulrik-sverdrup/do-plugins/screen.
>

Something to hack on when I have time! That would require finding out
which ssh hosts have key authentication, and then "simply" send "ssh
host 'screen -list'" over the network.. I wouldn't do it without the
user's explicit consent (plugin preferences) though. Anyway, that's
for a later time.

So, this thing that happened first time you used the plugin, do we
have to resolve that to merge this? I don't understand at all what
could have happened, but I can try reproducing it somehow.

ulrik

[Alex Launi (alexlauni)]

Organize using statements by length, or lexicographically, and separate them logically, all of the System namespaces together, etc.

Make sure you're following mono guidelines for method calls, I see a lot of MethodName(params) when I should be seeing only MethodName (params)

whitespace issues. separate your methods by newlines. Description and LastUpdateTime in NewScreenAction.cs are touching.

There are some weird tabbing issues too, I'm going to blame monodevelop, but it still needs fixed. Make sure oyu're using all tabs, not spaces.

In SupportsModifierItemForItems, you assign a bool, and then return it. You can save yourself a stack variable. It may not be necessary, but you may also want to check if that 'as' statement failed and left you with a null value-
return file == null ? false : Directory.Exists (fitem.Path);

I see you're hardcoding GNOME terminal, there is a GConf key for the users' preferred terminal. This should be respected, and you should launch that.

perform could use breaking up, if not into methods than just use whitespace. Perform is pretty dense, hard to read. One suggestion I've got for this is to declare your variables at the beginning, then assign them later, vs. having declarations littered throughout the method. Obviously this isn't true for small scoped variables.

These comments pretty much hold for the rest of the merge.

review: Needs Fixing (ball-buster style)

[Alex Launi (alexlauni)]

ping? This is a cool plugin, if you just fix these couple of issues we can merge!

[Alex Launi (alexlauni)]

Rejecting due to abandonment

Unmerged revisions

523. By X on 2009-02-08

New Plugin: GNU Screen

This is a plugin for Gnome Do to work with GNU Screen sessions. GNU Screen
(command 'screen') is a command line application for managing multi-windowed
shell sessions that can be detached and reattached for persistency, and offers
much configurability.

This adds the Plugin directory ./GNUScreen, and adds the plugin to monodevelop
as well as to the autotools setup.

This is the Plugin code as provided in gitorious revision 2fd1c03b at
screen-plugin-gnome-do/mainline

http://gitorious.org/projects/screen-plugin-gnome-do

=== modified file 'DoPlugins.mds'
--- DoPlugins.mds	2009-02-01 04:00:44 +0000
+++ DoPlugins.mds	2009-02-08 17:28:11 +0000
@@ -68,6 +68,7 @@
       <Entry build="True" name="xmms2" configuration="Debug" />
       <Entry build="True" name="TinyUrl" configuration="Debug" />
       <Entry build="True" name="Tracker" configuration="Debug" />
+      <Entry build="True" name="GNUScreen" configuration="Debug" />
     </Configuration>
     <Configuration name="Release" ctype="CombineConfiguration">
       <Entry build="True" name="Rhythmbox" configuration="Release" />
@@ -137,6 +138,7 @@
       <Entry build="True" name="xmms2" configuration="Release" />
       <Entry build="True" name="TinyUrl" configuration="Release" />
       <Entry build="True" name="Tracker" configuration="Release" />
+      <Entry build="True" name="GNUScreen" configuration="Release" />
     </Configuration>
   </Configurations>
   <StartMode startupentry="Rhythmbox" single="True">
@@ -207,6 +209,7 @@
     <Execute type="None" entry="xmms2" />
     <Execute type="None" entry="TinyUrl" />
     <Execute type="None" entry="Tracker" />
+    <Execute type="None" entry="GNUScreen" />
   </StartMode>
   <MonoDevelop.ChangeLogAddIn.ChangeLogInfo policy="UpdateNearestChangeLog" />
   <Entries>
@@ -277,5 +280,6 @@
     <Entry filename="Xmms2/xmms2.mdp" />
     <Entry filename="TinyUrl/TinyUrl.mdp" />
     <Entry filename="Tracker/Tracker.mdp" />
+    <Entry filename="GNUScreen/GNUScreen.mdp" />
   </Entries>
 </Combine>
\ No newline at end of file

=== added directory 'GNUScreen'
=== added file 'GNUScreen/GNUScreen.mdp'
--- GNUScreen/GNUScreen.mdp	1970-01-01 00:00:00 +0000
+++ GNUScreen/GNUScreen.mdp	2009-02-08 17:28:11 +0000
@@ -0,0 +1,30 @@
+<Project name="GNUScreen" fileversion="2.0" language="C#" clr-version="Net_2_0" ctype="DotNetProject">
+  <Configurations active="Debug">
+    <Configuration name="Debug" ctype="DotNetProjectConfiguration">
+      <Output directory="bin/Debug" assembly="GNUScreen" />
+      <Build debugmode="True" target="Library" />
+      <Execution runwithwarnings="True" consolepause="False" runtime="MsNet" clr-version="Net_2_0" />
+      <CodeGeneration compiler="Mcs" warninglevel="4" optimize="True" unsafecodeallowed="False" generateoverflowchecks="True" definesymbols="DEBUG" generatexmldocumentation="False" ctype="CSharpCompilerParameters" />
+    </Configuration>
+    <Configuration name="Release" ctype="DotNetProjectConfiguration">
+      <Output directory="bin/Release" assembly="GNUScreen" />
+      <Build debugmode="False" target="Library" />
+      <Execution runwithwarnings="True" consolepause="False" runtime="MsNet" clr-version="Net_2_0" />
+      <CodeGeneration compiler="Mcs" warninglevel="4" optimize="True" unsafecodeallowed="False" generateoverflowchecks="True" generatexmldocumentation="False" ctype="CSharpCompilerParameters" />
+    </Configuration>
+  </Configurations>
+  <Contents>
+    <File name="." subtype="Directory" buildaction="Compile" />
+    <File name="Resources/GNUScreen.addin.xml" subtype="Code" buildaction="EmbedAsResource" />
+    <File name="src/NewScreenAction.cs" subtype="Code" buildaction="Compile" />
+    <File name="src/ScreenAction.cs" subtype="Code" buildaction="Compile" />
+    <File name="src/ScreenSessions.cs" subtype="Code" buildaction="Compile" />
+  </Contents>
+  <References>
+    <ProjectReference type="Gac" localcopy="True" refto="System, Version=2.0.0.0, Culture=neutral, PublicKeyToken=b77a5c561934e089" />
+    <ProjectReference type="Gac" localcopy="True" refto="Do.Universe, Version=0.8.0.0, Culture=neutral, PublicKeyToken=null" />
+    <ProjectReference type="Gac" localcopy="True" refto="Do.Platform, Version=0.8.0.0, Culture=neutral, PublicKeyToken=null" />
+    <ProjectReference type="Gac" localcopy="True" refto="Mono.Posix, Version=2.0.0.0, Culture=neutral, PublicKeyToken=0738eb9f132ed756" />
+    <ProjectReference type="Gac" localcopy="True" refto="System.Core, Version=3.5.0.0, Culture=neutral, PublicKeyToken=b77a5c561934e089" />
+  </References>
+</Project>
\ No newline at end of file

=== added file 'GNUScreen/Makefile.am'
--- GNUScreen/Makefile.am	1970-01-01 00:00:00 +0000
+++ GNUScreen/Makefile.am	2009-02-08 17:28:11 +0000
@@ -0,0 +1,18 @@
+include $(top_srcdir)/build.rules.mk
+
+ASSEMBLY=GNUScreen
+
+FILES = \
+	src/NewScreenAction.cs \
+	src/ScreenAction.cs \
+	src/ScreenSessions.cs
+
+RESOURCES = \
+	Resources/GNUScreen.addin.xml
+
+REFERENCES = \
+	Mono.Posix \
+	System \
+	System.Core \
+	$(DO_PLATFORM_LIBS) \
+	$(DO_UNIVERSE_LIBS)

=== added directory 'GNUScreen/Resources'
=== added file 'GNUScreen/Resources/GNUScreen.addin.xml'
--- GNUScreen/Resources/GNUScreen.addin.xml	1970-01-01 00:00:00 +0000
+++ GNUScreen/Resources/GNUScreen.addin.xml	2009-02-08 17:28:11 +0000
@@ -0,0 +1,27 @@
+<Addin
+	id="GNUScreen"
+	namespace="Do"
+	version="1.0"
+	name="GNU Screen"
+	description="Work with GNU Screen sessions"
+	author="Ulrik Sverdrup"
+	category="Community"
+	>
+
+	<Runtime>
+		<Import assembly="GNUScreen.dll"/>
+	</Runtime>
+
+	<Dependencies>
+		<Addin id="Universe" version="1.0" />
+	</Dependencies>
+
+	<Extension path = "/Do/ItemSource">
+		<ItemSource type="GnomeDoScreen.ScreenSessionItemSource" />
+	</Extension>
+	<Extension path = "/Do/Action">
+		<Action type="GnomeDoScreen.AttachScreenAction" />
+		<Action type="GnomeDoScreen.NewScreenAction" />
+		<Action type="GnomeDoScreen.ScreenCommandAction" />
+	</Extension>
+</Addin>

=== added directory 'GNUScreen/src'
=== added file 'GNUScreen/src/NewScreenAction.cs'
--- GNUScreen/src/NewScreenAction.cs	1970-01-01 00:00:00 +0000
+++ GNUScreen/src/NewScreenAction.cs	2009-02-08 17:28:11 +0000
@@ -0,0 +1,110 @@
+/* NewScreenAction.cs
+ *
+ * GNOME Do is the legal property of its developers. Please refer to the
+ * COPYRIGHT file distributed with this
+ * source distribution.
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation, either version 3 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program. If not, see <http://www.gnu.org/licenses/>.
+ */
+
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.IO;
+using Do.Universe;
+using Do.Platform;
+using System.Diagnostics;
+
+using Mono.Unix;
+
+namespace GnomeDoScreen {
+	public class NewScreenAction : Act {
+		
+		public override string Name {
+			get {
+				return Catalog.GetString("New Screen session");
+			}
+		}
+		
+		public override string Description {
+			get {
+				return Catalog.GetString("Create a new GNU Screen session");
+			}
+		}
+		public DateTime LastUpdateTime { get; private set;}
+		
+		public override string Icon {
+			get {
+				return "terminal";
+			}
+		}
+		
+		public override IEnumerable<Type> SupportedItemTypes {
+			get {
+				yield return typeof(ITextItem);
+			}
+		}
+		
+		public override bool SupportsItem (Item item) {
+			return true;
+		}
+
+		/* Support directory as optional
+		 */
+        public override IEnumerable<Type> SupportedModifierItemTypes {
+            get { yield return typeof(IFileItem); }
+        }
+
+        public override bool ModifierItemsOptional {
+            get { return true; }
+        }
+		public override bool SupportsModifierItemForItems (IEnumerable<Item> items, Item modItem)
+		{   
+			IFileItem fitem = modItem as IFileItem;
+			bool exists = Directory.Exists(fitem.Path);
+			return exists;
+		}
+		
+		public override IEnumerable<Item> Perform (IEnumerable<Item> items, IEnumerable<Item> modItems) {
+			string exec = "gnome-terminal";
+
+			string sessionname = "";
+			if (items.First () is ITextItem) {
+				ITextItem textitem = items.First () as ITextItem;
+				sessionname = textitem.Text.Trim();
+			}
+			string workdir = null;
+			if (modItems.Count() > 0 && modItems.First() is IFileItem) {
+				string path = (modItems.First() as IFileItem).Path;
+				/* Check if it is an existing directory
+				 */
+				if (Directory.Exists(path))
+					workdir = path;
+			}
+			Process term = new Process ();
+			term.StartInfo.FileName = exec;
+			string sessionarg = "", titlearg = "";
+			if (sessionname != "")
+				sessionarg = "-S '" + sessionname + "'";
+			if (sessionname != "")
+				titlearg = String.Format("--title='{0}'", sessionname);
+			term.StartInfo.Arguments = String.Format("{0} -x screen {1}", titlearg, sessionarg);
+			if (workdir != null)
+				term.StartInfo.WorkingDirectory = workdir;
+			Log<NewScreenAction>.Debug(term.StartInfo.Arguments);
+			term.Start ();
+			return null;
+		}
+	}
+}

=== added file 'GNUScreen/src/ScreenAction.cs'
--- GNUScreen/src/ScreenAction.cs	1970-01-01 00:00:00 +0000
+++ GNUScreen/src/ScreenAction.cs	2009-02-08 17:28:11 +0000
@@ -0,0 +1,209 @@
+/* ScreenAction.cs
+ *
+ * GNOME Do is the legal property of its developers. Please refer to the
+ * COPYRIGHT file distributed with this
+ * source distribution.
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation, either version 3 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program. If not, see <http://www.gnu.org/licenses/>.
+ */
+
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using Do.Universe;
+using Do.Platform;
+using System.Diagnostics;
+
+using Mono.Unix;
+
+namespace GnomeDoScreen {
+	public class AbstractScreenAction : Act {
+		static ScreenSessionItemSource source_cache = null;
+		public override string Name {
+			get { return "AbstractScreenAction"; }
+		}
+		
+		public override string Description {
+			get { return "Base class"; }
+		}
+
+		public override string Icon {
+			get {
+				return "terminal";
+			}
+		}
+		
+		public override IEnumerable<Type> SupportedItemTypes {
+			get {
+				return new Type[] {
+					typeof(ITextItem),
+					typeof(ScreenSessionItem),
+				};
+			}
+		}
+
+		public static DateTime LastUpdateTime { get; private set;}
+
+		private static void UpdateCacheIfOld(int wait) {
+			/* Wait before update */
+			TimeSpan update_interval = new TimeSpan(0,0,wait);
+			if ((DateTime.UtcNow - LastUpdateTime) > update_interval ) {
+				source_cache.UpdateItems();
+				LastUpdateTime = DateTime.UtcNow;
+			}
+		}
+
+		private static bool TextMatchesItem(string text, ScreenSessionItem item) {
+			if (item == null)
+				return false;
+			string pid = item.Pid;
+			string name = item.SessionName;
+			if (name.StartsWith(text) || pid.StartsWith(text))
+				return true;
+			return false;;
+		}
+		
+		public override bool SupportsItem (Item item) {
+			if (item is ScreenSessionItem) {
+				/* If it's a SSI, it should be updated from
+				 * the source side
+				 */
+				return true;
+			} else if (item is ITextItem) {
+				ITextItem titem = item as ITextItem;
+				/* Check if the given text
+				 * matches the beginning of the 
+				 * _pid_ or _tty name_ of any running 
+				 * session
+				 */
+				/* Wait shorter time for text items */
+				if (source_cache == null)
+					source_cache = new ScreenSessionItemSource();
+				UpdateCacheIfOld(15);
+				foreach (Item it in source_cache.Items) {
+					ScreenSessionItem ssitem = it as ScreenSessionItem;
+					if (this.TextMatchesItem(titem.Text, ssitem)) {
+							return true;
+					}
+				}
+			}
+			return false;
+		}
+		public override IEnumerable<Item> Perform (IEnumerable<Item> items, IEnumerable<Item> modItems) {
+			return null;
+		}
+
+		public static void SpawnScreenSession(string name, string arguments) {
+			string term = "gnome-terminal";
+			ProcessStartInfo ps = new ProcessStartInfo (term);
+			string term_arguments = "";
+			if (name != "")
+				term_arguments += "--title='" + name + "' ";
+			term_arguments += "-x screen ";
+			if (arguments != "")
+				term_arguments += arguments;
+			ps.Arguments = term_arguments;
+			using (Process p = Process.Start (ps)) {
+				Log<AbstractScreenAction>.Debug(ps.Arguments);
+			}
+		}
+
+		public static void RunScreen(string arguments) {
+			string term = "screen";
+			ProcessStartInfo ps = new ProcessStartInfo (term);
+			ps.UseShellExecute = false;
+			ps.Arguments = arguments;
+			using (Process p = Process.Start (ps)) {
+				Log<ScreenCommandAction>.Debug(ps.Arguments);
+			}
+		}
+	}
+
+	public class AttachScreenAction : AbstractScreenAction {
+		
+		public override string Name {
+			get {
+				return Catalog.GetString("Attach");
+			}
+		}
+		
+		public override string Description {
+			get {
+				return Catalog.GetString("Attach or reattach a running GNU Screen session");
+			}
+		}
+		
+		public override IEnumerable<Item> Perform (IEnumerable<Item> items, IEnumerable<Item> modItems) {
+			string screenname = null;
+			string termname = "";
+			if (items.First () is ITextItem) {
+				ITextItem textitem = items.First () as ITextItem;
+				screenname = textitem.Text;
+			} else if (items.First () is ScreenSessionItem) {
+				ScreenSessionItem sitem = items.First () as ScreenSessionItem;
+				/* Attach with Pid, because it is always correct,
+				 * the session name can change! */
+				screenname = sitem.Pid;
+				termname = sitem.Name;
+			}
+			if (screenname != null) {
+				SpawnScreenSession(termname, "-x -R " + screenname);
+			}
+			return null;
+		}
+
+	}
+
+	public class ScreenCommandAction : AbstractScreenAction {
+		public override string Name {
+			get {
+				return Catalog.GetString("Send command");
+			}
+		}
+		
+		public override string Description {
+			get {
+				return Catalog.GetString("Send a command to a running GNU Screen session");
+			}
+		}
+		public override string Icon {
+			get { return "gtk-execute"; }
+		}
+
+		public override IEnumerable<Type> SupportedModifierItemTypes {
+			get { yield return typeof(ITextItem); }
+		}
+
+		public override IEnumerable<Item> Perform (IEnumerable<Item> items, IEnumerable<Item> modItems) {
+			string screenname = null;
+			string command = null;
+			if (items.First () is ITextItem) {
+				ITextItem textitem = items.First () as ITextItem;
+				screenname = textitem.Text.Trim();
+			} else if (items.First () is ScreenSessionItem) {
+				ScreenSessionItem sitem = items.First () as ScreenSessionItem;
+				/* Use with Pid, because it is always correct,
+				 * the session name can change! */
+				screenname = sitem.Pid;
+			}
+			if (modItems.First() is ITextItem) {
+				command = (modItems.First() as ITextItem).Text;
+			}
+			if (screenname != null && command != null) {
+				RunScreen("-S " + screenname + " -X " + command);
+			}
+			return null;
+		}
+	}
+}

=== added file 'GNUScreen/src/ScreenSessions.cs'
--- GNUScreen/src/ScreenSessions.cs	1970-01-01 00:00:00 +0000
+++ GNUScreen/src/ScreenSessions.cs	2009-02-08 17:28:11 +0000
@@ -0,0 +1,152 @@
+/* ScreenSessions.cs
+ *
+ * GNOME Do is the legal property of its developers. Please refer to the
+ * COPYRIGHT file distributed with this
+ * source distribution.
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation, either version 3 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program. If not, see <http://www.gnu.org/licenses/>.
+ */
+
+using System;
+using System.IO;
+using System.Collections.Generic;
+using System.Text.RegularExpressions;
+using System.Diagnostics;
+
+using Do.Platform;
+using Do.Universe;
+
+using Mono.Unix;
+
+
+namespace GnomeDoScreen {
+
+	public class ScreenSessionItem : Item {
+		string pid, sessionname, name, status, date;
+
+		public ScreenSessionItem (string ipid, string iname, string istatus, string idate)
+		{
+			sessionname = iname;
+			status = istatus;
+			date = idate;
+
+			/* Pid used to identify the session */
+			pid = ipid;
+			/* Display name */
+			name = String.Format("{0} ({1})", sessionname, pid);
+		}
+
+		public override string Name { get { return name; } }
+		public override string Description { 
+			get { 
+				return status + " GNU Screen session, created " + date;
+			}
+		}
+		public override string Icon { get { return "gnome-window-manager"; } }
+
+		public string Pid { get { return pid; } }
+		public string SessionName { get { return sessionname; } }
+	}
+
+	public class ScreenSessionItemSource : ItemSource {
+		List<Item> items;
+
+		public ScreenSessionItemSource ()
+		{
+			items = new List<Item> ();
+			UpdateItems ();
+		}
+
+		public override string Name { get { return Catalog.GetString("Screen sessions"); } }
+		public override string Description { get { return Catalog.GetString("Lists running GNU Screen sessions"); } }
+		public override string Icon { get { return "terminal"; } }
+
+		public override IEnumerable<Type> SupportedItemTypes {
+			get {
+				yield return typeof (ScreenSessionItem);
+			}
+		}
+
+		public override IEnumerable<Item> Items {
+			get {
+				return items;
+			}
+		}
+
+		public override IEnumerable<Item> ChildrenOfItem (Item parent)
+		{
+			yield break;
+		}
+
+		public override void UpdateItems ()
+		{
+			items.Clear ();
+			Log<ScreenSessionItemSource>.Debug(" updating items..");
+
+	 		try {
+				// sets up our process, the first argument is the command 
+				// and the second holds the arguments passed to the command
+				ProcessStartInfo ps = new ProcessStartInfo ("screen", "-list");
+				ps.UseShellExecute = false;
+				// we need to redirect the standard output so we read it
+				// internally in out program
+				ps.RedirectStandardOutput = true;
+				
+				// starts the process
+				string output = null;
+				using (Process p = Process.Start (ps)) {
+					// we read the output to a string
+					output = p.StandardOutput.ReadToEnd ();
+					// waits for the process to exit
+					p.WaitForExit ();
+				}
+				if (output == null)
+					return;
+
+				StringReader reader = new StringReader (output);
+
+				Regex r = new Regex ("^\\t([^\\t]+)\\t\\((.*)\\)\\t\\((.*)\\)$");
+				Regex pidtty= new Regex ("^([0-9]+)\\.(.*)");
+
+				string s;
+				/* Group matches:
+				 * Groups[0] is the whole string that matched
+				 * and [1] and so on are the () groups one by one
+				 */
+				while ((s = reader.ReadLine ()) != null) {
+					Match m = r.Match (s);
+					if (m.Groups.Count == 4) {
+						string fullname, status, date;
+						fullname = m.Groups[1].ToString();
+						date = m.Groups[2].ToString();
+						status = m.Groups[3].ToString();
+						/* Part the pid.tty.host into pid and tty
+						 */
+						Match ptm = pidtty.Match(fullname);
+						string pid, name;
+						if (ptm.Groups.Count == 3) {
+							pid = ptm.Groups[1].ToString();
+							name = ptm.Groups[2].ToString();
+							items.Add (new ScreenSessionItem (pid, name, status, date));
+						} else {
+							/* Best to report matching error */
+							Log<ScreenSessionItemSource>.Debug("Unable to parse " + s);
+						}
+					}
+				}
+			} catch { }
+		}
+	}
+}
+

=== modified file 'Makefile.am'
--- Makefile.am	2009-01-30 00:17:43 +0000
+++ Makefile.am	2009-02-08 17:28:11 +0000
@@ -30,6 +30,7 @@
 	GoogleDocs \
 	GoogleMaps \
 	GoogleSearch \
+	GNUScreen \
 	ImageShack \
 	JIRA \
 	Launchpad \

=== modified file 'configure.ac'
--- configure.ac	2009-01-29 20:26:24 +0000
+++ configure.ac	2009-02-08 17:28:11 +0000
@@ -135,6 +135,7 @@
 GoogleDocs/Makefile
 GoogleMaps/Makefile
 GoogleSearch/Makefile
+GNUScreen/Makefile
 ImageShack/Makefile
 JIRA/Makefile
 Launchpad/Makefile