Merge lp:~bidossessi-sodonon/synapse-project/mpc-plugin into lp:synapse-project

[Bidossessi Sodonon]

Changed MpcEngine's methods to static to avoid unnecessary instantiation.

[Alberto Aldegheri (albyrock87)]

First of all, thank you for your contribution! :)

Some comments:
1)
MpcEngine class is instantiated each time I execute an action.
But MpcEngine doesn't have "attributes" that needs to be instantiated.
So I suggest to turn each method and attribute of MpcEnginge into a static one.
In example:
private static string bin = "mpc"; //this can be also "const"
public static void play_pause ()...

In this way you can call directly:
MpcEngine.play_pause ();

2) what's the meaning of
 >>search the database for a "type" and turn results to uris on the fly<< ?

3) Synapse's configuration capabilities will come asap (mhr3 and I are very busy in this moment)

[Bidossessi Sodonon (bidossessi-sodonon)]

Thanks for your prompt reply.

About 1), blame my lack of programming experience with compiled code. I will update this asap.

Now, for 2), from man mpc:

search <type> <query> [<type> <query>]...
              Searches for songs where all of the given tag <type>s match the given <query>s. Any number of tag type and query combinations can be specified. Possible tag
              types are: artist, album, title, track, name, genre, date, composer, performer, comment, disc, filename, or any (to match any tag).

Here is a sample of the list returned:

[b_sodonon@sysadmin synapse-project]$ mpc search genre world
Ali Farka Toure & Toumani Diabate/In the Heart of the Moon/02. Kala.mp3
Ali Farka Toure & Toumani Diabate/In the Heart of the Moon/03. Mamadou Boutiquier.mp3

The resulting file paths are relative to mpd's "musid_dir" configuration.

what needs to happen is the following (python):

abs_path = music_dir + rel_path
uri = 'file://' + str(urllib.quote(mpd_path, safe='/~'))

I use that code to create uris I feed into zeitgeist.

I have no idea how to do that in vala.
Not to mention that, to enqueue files in mpd, I have to provide relative_path as an argument, which means going from uri to relative path. No idea how I could do that either. So until I become a bit more conversant with vala, I'll stick to the basic functionality.

Once again, thanks for the reply. It feels good be finally giving back to the community. :)

Cheers.

[Bidossessi Sodonon (bidossessi-sodonon)]

I hope this is the right way to go about updating my merge proposal.
If not, I'm sorry.

Cheers

[Alberto Aldegheri (albyrock87)]

:) Do not resubmit the proposal (I made the same mistake the first time I used Launchpad :P).
Just add a comment an I (or mhr3) will come here and look at changes.

I'll look at your merge proposal asap!

[lp:~bidossessi-sodonon/synapse-project/mpc-plugin]

443. By Bidossessi Sodonon on 2011-05-26

Forgot to add the "stop" command

444. By Bidossessi Sodonon on 2011-05-26

Forgot some strings to make Stop appear

445. By Bidossessi Sodonon on 2011-05-26

Fixed unreachable catch issues

446. By Bidossessi Sodonon on 2011-05-27

Feature-complete code, as compared to banshee-plugin

[Bidossessi Sodonon (bidossessi-sodonon)]

Plugin is now feature-complete, using banshee as a feature basis.

It does the following actions:
- start, stop, pause and resume playback
- enqueue a uri in mpd
- play a track, clearing the playlist

I reinstated instantiation in prevision of Synapse's configuration capabilities.
That way, (almost) no refactoring will be needed.

I'm sure the code is not optimized, but, that's my level, for now.

Cheers

[lp:~bidossessi-sodonon/synapse-project/mpc-plugin]

447. By Bidossessi Sodonon on 2011-05-27

Fixed the location for this file.

448. By Bidossessi Sodonon on 2011-05-27

Updated the pot file.

[Michal Hruby (mhr3)]

A few notes:

- please revert the changes to hybrid-search-plugin
- line 210-217 and others - why do you catch the error, if you only re-throw it?

Other than that it looks ok...

review: Needs Fixing

[lp:~bidossessi-sodonon/synapse-project/mpc-plugin]

449. By Bidossessi Sodonon on 2011-05-28

* reverted hybrid-search-plugin to previous code
* stopped catching errors in public methods in mpc-plugin (no real need)
* updated synapse.pot to

[Bidossessi Sodonon (bidossessi-sodonon)]

> A few notes:
>
> - please revert the changes to hybrid-search-plugin
> - line 210-217 and others - why do you catch the error, if you only re-throw
> it?
>
> Other than that it looks ok...
Well, I kept catching until vala stopped giving me unhandled error messages. They were kind of scary. But I admint there's no real need to do that.

I believe that with 449, this plugin is ready.

[Bidossessi Sodonon (bidossessi-sodonon)]

Please check if more needs to be done for this plugin.
Thank you.

review: Needs Resubmitting

[Alberto Aldegheri (albyrock87)]

Please, enter in Synapse's folder, and execute:
bzr revert --revision 440 src/plugins/hybrid-search-plugin.vala

And:

bzr revert --revision 440 po/synapse.pot

(we'll update translations later)

At this point: return path[music_root.length:path.length]; (line 411 of the diff)
Are you sure that music_root is a prefix of path ?

review: Needs Fixing

[lp:~bidossessi-sodonon/synapse-project/mpc-plugin]

450. By Bidossessi Sodonon on 2011-05-31

Don't try to play files that are not in MPD's music root (since it will fail anyway).

451. By Bidossessi Sodonon on 2011-05-31

Reverted pot file and hybrid-search-plugin to rev 440.

[Bidossessi Sodonon (bidossessi-sodonon)]

> Please, enter in Synapse's folder, and execute:
> bzr revert --revision 440 src/plugins/hybrid-search-plugin.vala
>
> And:
>
> bzr revert --revision 440 po/synapse.pot
>
> (we'll update translations later)
>
Done, and sorry about that.
> At this point: return path[music_root.length:path.length]; (line 411 of the
> diff)
> Are you sure that music_root is a prefix of path ?
Got it: files not in mpd's music_root are now handled.

review: Needs Resubmitting

[Antono Vasiljev (antono)]

Any updates here?

Unmerged revisions

451. By Bidossessi Sodonon on 2011-05-31

Reverted pot file and hybrid-search-plugin to rev 440.

450. By Bidossessi Sodonon on 2011-05-31

Don't try to play files that are not in MPD's music root (since it will fail anyway).

449. By Bidossessi Sodonon on 2011-05-28

* reverted hybrid-search-plugin to previous code
* stopped catching errors in public methods in mpc-plugin (no real need)
* updated synapse.pot to

448. By Bidossessi Sodonon on 2011-05-27

Updated the pot file.

447. By Bidossessi Sodonon on 2011-05-27

Fixed the location for this file.

446. By Bidossessi Sodonon on 2011-05-27

Feature-complete code, as compared to banshee-plugin

445. By Bidossessi Sodonon on 2011-05-26

Fixed unreachable catch issues

444. By Bidossessi Sodonon on 2011-05-26

Forgot some strings to make Stop appear

443. By Bidossessi Sodonon on 2011-05-26

Forgot to add the "stop" command

442. By Bidossessi Sodonon on 2011-05-25

Turned MpcEngine instance method to class methods (static)

=== modified file 'src/plugins/Makefile.am'
--- src/plugins/Makefile.am	2011-04-18 14:49:45 +0000
+++ src/plugins/Makefile.am	2011-05-31 10:31:30 +0000
@@ -46,6 +46,7 @@
 	hybrid-search-plugin.vala \
 	launchpad-plugin.vala \
 	locate-plugin.vala \
+	mpc-plugin.vala \
 	opensearch.vala \
 	pastebin-plugin.vala \
 	pidgin-plugin.vala \
 
=== added file 'src/plugins/mpc-plugin.vala'
--- src/plugins/mpc-plugin.vala	1970-01-01 00:00:00 +0000
+++ src/plugins/mpc-plugin.vala	2011-05-31 10:31:30 +0000
@@ -0,0 +1,639 @@
+/*
+ * Copyright (C) 2011 Bidossessi Sodonon <stanislas.sodonon at gmail.com>
+ *
+ * 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 2 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, write to the Free Software
+ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301  USA.
+ *
+ * Authored by Bidossessi Sodonon <stanislas.sodonon at gmail.com>
+ *
+ */
+
+namespace Synapse
+{
+  public class MpcEngine: Object
+  {
+    // our executable string array
+    private const string bin = "mpc";
+    // mpd paths are relative to this directory
+    public string music_root {get;construct;}
+    // if these default values are given, use them
+    public string host {get;construct;} // localhost by defaut
+    public string port {get;construct;}  // 6600 by default
+    public string passwd {get;construct;}// no pasword by default
+
+    // constructor
+    public MpcEngine (string? host = "localhost",
+          string? port = "6600", string? passwd = null,
+          string? music_root = null)
+    {
+      // mpc is called like this: mpc [-h [_passwd@]_host] [-p _port] <command> [arguments]
+      Object(host: host, port: port, passwd: passwd, music_root: music_root);
+    }
+    construct {
+      //make sure we find the music_dir if it wasn't given.
+        music_root = find_music_root(music_root);
+    }
+
+    /* Public operations (pass errors to caller)*/
+    public void stop () throws IOError
+    {
+      run_command("stop");
+    }
+    public void play_pause () throws IOError
+    {
+      run_command("toggle");
+    }
+    public void play_now (string uri) throws IOError
+    {
+      string rel_path = path_from_uri(uri);
+      run_command ("clear");
+      run_command ("add", rel_path);
+      run_command ("play");
+    }
+    public void enqueue (string uri) throws IOError
+    {
+      string rel_path = path_from_uri(uri);
+      run_command("add", rel_path);
+    }
+    public void next () throws IOError
+    {
+      run_command("next");
+    }
+    public void previous () throws IOError
+    {
+      run_command("prev");
+    }
+
+    /* Engine Internals */
+    // convenience function
+    protected string turn_abs(string user_path)
+    {
+      //this shouldn't affect absolute paths
+      try
+      {
+        MatchInfo info;
+        Regex regex = new Regex("^~");
+        string user_dir = Environment.get_home_dir();
+        if (regex.match(user_path, 0, out info))
+        {
+          string abs_path = regex.replace (user_path, -1, 0, user_dir);
+          //make sure it exists
+          var u = File.new_for_path(abs_path);
+          if (u.query_exists ())
+          {
+            return abs_path;
+          }
+        }
+      }
+      // ouch, what happened?
+      catch (Error e)
+      {
+        Utils.Logger.warning (this,"Failed to turn %s to absolute path: %s", user_path, e.message);
+      }
+      return user_path;
+    }
+
+    protected File? get_mpd_conf ()
+    {
+      // try ~/.mpdconf
+      string user_conf = turn_abs("~/.mpdconf");
+      var ufile = File.new_for_path (user_conf);
+      if (ufile.query_exists ())
+      {
+        return ufile;
+      }
+      // try /etc/mpd.conf
+      else
+      {
+        Utils.Logger.warning (this,"File '%s' doesn't exist", user_conf);
+        //not found, trying the other, hoping it's world-readable
+        var sfile = File.new_for_path ("/etc/mpd.conf");
+        if (sfile.query_exists ())
+        {
+          return sfile;
+        }
+      }
+      Utils.Logger.warning (this,"File '/etc/mpd.conf' doesn't exist");
+      return null;
+    }
+
+    protected string? find_music_root (string? given_root)
+    {
+      try
+      {
+        if (given_root != null)
+        {
+          var g = File.new_for_path(given_root);
+          if (g.query_exists ())
+          {
+            return given_root;
+          }
+        }
+        // the given path didn't check out
+        else
+        {
+          File file = get_mpd_conf();
+          if (file != null)
+          {
+            var dis = new DataInputStream (file.read ());
+            string line;
+            while ((line = dis.read_line (null)) != null)
+            {
+              //find the right line
+              Regex tag = new Regex("^music_directory.*$");
+              if (tag.match (line))
+              {
+                //capture the dir between the quotation marks
+                MatchInfo info;
+                Regex dir = new Regex("(?<=\")(.*?)(?=\")");
+                if (dir.match (line, 0, out info))
+                {
+                  string found_root =  turn_abs(info.fetch(0)) + "/";
+                  //make sure it exists
+                  var r = File.new_for_path(found_root);
+                  if (r.query_exists ())
+                  {
+                    return found_root;
+                  }
+                }
+              }
+            }
+          }
+        }
+      }
+      catch (Error e)
+      {
+        Utils.Logger.warning (this,"Music root not found: %s", e.message);
+      }
+      return null;
+    }
+
+    // convert uri to relative path
+    private string? path_from_uri (string uri) throws IOError
+    {
+      try
+      {
+        string path = Filename.from_uri(uri);
+        //make sure it exists
+        var p = File.new_for_path(path);
+        if (p.query_exists ())
+        {
+          if (music_root != null)
+          {
+            if (music_root in path)
+            {
+                //strip music_root
+                return path[music_root.length:path.length];
+            }
+            else
+            {
+                throw new IOError.NOT_FOUND("File not in mpd's music root");
+            }
+          }
+          else
+          {
+            throw new IOError.NOT_FOUND("Music root not found");
+          }
+        }
+        else
+        {
+          throw new IOError.NOT_FOUND("File %s not found", path);
+        }
+      }
+      catch (Error e)
+      {
+        Utils.Logger.warning (this,"Couldn't convert URI to filename: %s", e.message);
+      }
+      return null;
+    }
+
+    // run command
+    protected void run_command (string command,
+    string? qtype = null, string? sarg = null) throws IOError
+    {
+      Pid pid;
+      string complete_output = "";
+      bool error_found = false;
+      string[] argv = {bin};
+      if (host != null)
+      {
+        var s = new StringBuilder();
+        if (passwd != null)
+        {
+          s.append_printf("%s@", passwd);
+        }
+        s.append_printf("%s", host);
+        argv += "-h";
+        argv += s.str;
+      }
+      if (port != null)
+      {
+        argv += "-p";
+        argv += port;
+      }
+      argv += command;
+      if (qtype != null)
+        {
+          argv += qtype;
+          if (sarg != null)
+          {
+            argv += sarg;
+          }
+        }
+
+      try
+      {
+        int read_fd;
+        // FIXME: probably a better way to do this
+        Process.spawn_async_with_pipes (null, argv, null,
+                                      SpawnFlags.SEARCH_PATH,
+                                      null, out pid, null, out read_fd);
+        UnixInputStream read_stream = new UnixInputStream (read_fd, true);
+        DataInputStream mpd_output = new DataInputStream (read_stream);
+
+        string? line = null;
+        do
+        {
+          line = mpd_output.read_line (null);
+          if (line != null)
+          {
+            complete_output += line;
+          }
+        } while (line != null);
+        // try and catch any 'error' line from mpc
+        Regex error = new Regex ("error:.*$");
+        if (error.match (complete_output))
+        {
+          error_found = true;
+        }
+      }
+      catch (Error err)
+      {
+        Utils.Logger.warning (this,"MPD is not available: %s", err.message);
+      }
+      if (error_found == true)
+      {
+        throw new IOError.CONNECTION_REFUSED (complete_output);
+      }
+    }
+  }//end of MpcEngine
+
+
+  /* SYNAPSE side (finally :p) */
+  public class MpcActions: Object, Activatable, ItemProvider, ActionProvider
+  {
+    public bool enabled { get; set; default = true; }
+
+    public void activate ()
+    {
+    }
+
+    public void deactivate ()
+    {
+    }
+
+    static void register_plugin ()
+    {
+      DataSink.PluginRegistry.get_default ().register_plugin (
+        typeof (MpcActions),
+        _ ("Mpc"),
+        _ ("Control the Music Player Daemon."),
+        "mpc",
+        register_plugin,
+        Environment.find_program_in_path ("mpc") != null,
+        _ ("mpc is not installed")
+      );
+    }
+
+    static construct
+    {
+      register_plugin ();
+    }
+
+    private abstract class MpcAction: Object, Match
+    {
+      // from Match interface
+      public string title { get; construct set; }
+      public string description { get; set; }
+      public string icon_name { get; construct set; }
+      public bool has_thumbnail { get; construct set; }
+      public string thumbnail_path { get; construct set; }
+      public MatchType match_type { get; construct set; }
+
+      public int default_relevancy { get; set; }
+
+      public abstract bool valid_for_match (Match match);
+
+      public abstract void execute_internal (Match? match);
+      public void execute (Match? match)
+      {
+        execute_internal (match);
+      }
+      public virtual int get_relevancy ()
+      {
+        return default_relevancy + Match.Score.INCREMENT_LARGE;
+      }
+    }
+
+    private abstract class MpcControlMatch: Object, Match
+    {
+      // for Match interface
+      public string title { get; construct set; }
+      public string description { get; set; default = ""; }
+      public string icon_name { get; construct set; default = ""; }
+      public bool has_thumbnail { get; construct set; default = false; }
+      public string thumbnail_path { get; construct set; }
+      public MatchType match_type { get; construct set; }
+
+      public void execute (Match? match)
+      {
+        this.do_action ();
+      }
+
+      public abstract void do_action ();
+
+    }
+
+    /* MATCHES of Type.ACTION */
+    private class Play: MpcControlMatch
+    {
+      public Play ()
+      {
+        Object (title: _ ("Play"),
+                description: _ ("Start playback in MPD"),
+                icon_name: "media-playback-start", has_thumbnail: false,
+                match_type: MatchType.ACTION);
+      }
+      public override void do_action ()
+      {
+        try
+        {
+          var player = new MpcEngine ();
+          player.play_pause ();
+        }
+        catch (IOError e) {
+          Utils.Logger.warning (this,"MPD is not available: %s", e.message);
+        }
+      }
+    }
+
+    private class Pause: Play
+    {
+      public Pause ()
+      {
+        Object (title: _ ("Pause"),
+                description: _ ("Pause playback in MPD"),
+                icon_name: "media-playback-pause", has_thumbnail: false,
+                match_type: MatchType.ACTION);
+      }
+    }
+
+    private class Next: MpcControlMatch
+    {
+      public Next ()
+      {
+        Object (title: _ ("Next"),
+                description: _ ("Plays the next song in MPD's playlist"),
+                icon_name: "media-skip-forward", has_thumbnail: false,
+                match_type: MatchType.ACTION);
+      }
+
+      public override void do_action ()
+      {
+        try
+        {
+          var player = new MpcEngine ();
+          player.next ();
+        }
+        catch (IOError e) {
+          Utils.Logger.warning (this,"MPD is not available: %s", e.message);
+        }
+      }
+
+    }
+    private class Previous: MpcControlMatch
+    {
+      public Previous ()
+      {
+        Object (title: _ ("Previous"),
+                description: _ ("Plays the previous song in MPD's playlist"),
+                icon_name: "media-skip-backward", has_thumbnail: false,
+                match_type: MatchType.ACTION);
+      }
+
+      public override void do_action ()
+      {
+        try
+        {
+          var player = new MpcEngine ();
+          player.previous ();
+        }
+        catch (IOError e) {
+          Utils.Logger.warning (this,"MPD is not available: %s", e.message);
+        }
+      }
+    }
+
+    private class Stop: MpcControlMatch
+    {
+      public Stop ()
+      {
+        Object (title: _ ("Stop"),
+                description: _ ("Stop playback in MPD"),
+                icon_name: "media-stop", has_thumbnail: false,
+                match_type: MatchType.ACTION);
+      }
+
+      public override void do_action ()
+      {
+        try
+        {
+          var player = new MpcEngine ();
+          player.stop ();
+        }
+        catch (IOError e) {
+          Utils.Logger.warning (this, "MPD is not available: %s", e.message);
+        }
+      }
+    }
+
+    /* ACTIONS FOR Music files */
+    private class AddToPlaylist: MpcAction
+    {
+      public AddToPlaylist ()
+      {
+        Object (title: _ ("Enqueue in MPD"),
+                description: _ ("Add the song to MPD's playlist"),
+                icon_name: "media-playback-start", has_thumbnail: false,
+                match_type: MatchType.ACTION,
+                default_relevancy: Match.Score.AVERAGE);
+      }
+
+      public override void execute_internal (Match? match)
+      {
+        return_if_fail (match.match_type == MatchType.GENERIC_URI);
+        UriMatch uri = match as UriMatch;
+        return_if_fail ((uri.file_type & QueryFlags.AUDIO) != 0);
+
+        try
+        {
+          var player = new MpcEngine ();
+          player.enqueue (uri.uri);
+        }
+        catch (IOError e) {
+          Utils.Logger.warning (this, "MPD is not available: %s", e.message);
+        }
+
+      }
+
+      public override bool valid_for_match (Match match)
+      {
+        switch (match.match_type)
+        {
+          case MatchType.GENERIC_URI:
+            UriMatch uri = match as UriMatch;
+            if ((uri.file_type & QueryFlags.AUDIO) != 0)
+              return true;
+            else
+              return false;
+          default:
+            return false;
+        }
+      }
+    }
+    private class PlayNow: MpcAction
+    {
+      public PlayNow ()
+      {
+        Object (title: _ ("Play in MPD"),
+                description: _ ("Clears the current playlist and plays the song"),
+                icon_name: "media-playback-start", has_thumbnail: false,
+                match_type: MatchType.ACTION,
+                default_relevancy: Match.Score.ABOVE_AVERAGE);
+      }
+
+      public override void execute_internal (Match? match)
+      {
+        return_if_fail (match.match_type == MatchType.GENERIC_URI);
+        UriMatch uri = match as UriMatch;
+        return_if_fail ((uri.file_type & QueryFlags.AUDIO) != 0 );
+
+        try
+        {
+          var player = new MpcEngine ();
+          player.play_now (uri.uri);
+        }
+        catch (IOError e) {
+          Utils.Logger.warning (this, "MPD is not available: %s", e.message);
+        }
+      }
+
+      public override bool valid_for_match (Match match)
+      {
+        switch (match.match_type)
+        {
+          case MatchType.GENERIC_URI:
+            UriMatch uri = match as UriMatch;
+            if ((uri.file_type & QueryFlags.AUDIO) != 0)
+              return true;
+            else
+              return false;
+          default:
+            return false;
+        }
+      }
+    }
+
+    private Gee.List<MpcAction> actions;
+    private Gee.List<MpcControlMatch> matches;
+
+    construct
+    {
+      actions = new Gee.ArrayList<MpcAction> ();
+      matches = new Gee.ArrayList<MpcControlMatch> ();
+
+      actions.add (new PlayNow());
+      actions.add (new AddToPlaylist());
+
+      matches.add (new Play ());
+      matches.add (new Stop ());
+      matches.add (new Pause ());
+      matches.add (new Previous ());
+      matches.add (new Next ());
+    }
+
+    public async ResultSet? search (Query q) throws SearchError
+    {
+      // we only search for actions
+      if (!(QueryFlags.ACTIONS in q.query_type)) return null;
+
+      var result = new ResultSet ();
+
+      var matchers = Query.get_matchers_for_query (q.query_string, 0,
+        RegexCompileFlags.OPTIMIZE | RegexCompileFlags.CASELESS);
+
+      foreach (var action in matches)
+      {
+        foreach (var matcher in matchers)
+        {
+          if (matcher.key.match (action.title))
+          {
+            result.add (action, matcher.value - Match.Score.INCREMENT_SMALL);
+            break;
+          }
+        }
+      }
+
+      q.check_cancellable ();
+
+      return result;
+    }
+
+    public ResultSet? find_for_match (Query query, Match match)
+    {
+      bool query_empty = query.query_string == "";
+      var results = new ResultSet ();
+
+      if (query_empty)
+      {
+        foreach (var action in actions)
+        {
+          if (action.valid_for_match (match))
+          {
+            results.add (action, action.get_relevancy ());
+          }
+        }
+      }
+      else
+      {
+        var matchers = Query.get_matchers_for_query (query.query_string, 0,
+          RegexCompileFlags.OPTIMIZE | RegexCompileFlags.CASELESS);
+        foreach (var action in actions)
+        {
+          if (!action.valid_for_match (match)) continue;
+          foreach (var matcher in matchers)
+          {
+            if (matcher.key.match (action.title))
+            {
+              results.add (action, matcher.value);
+              break;
+            }
+          }
+        }
+      }
+
+      return results;
+    }
+  }
+}
 
=== modified file 'src/ui/synapse-main.vala'
--- src/ui/synapse-main.vala	2011-05-05 00:07:47 +0000
+++ src/ui/synapse-main.vala	2011-05-31 10:31:30 +0000
@@ -178,6 +178,7 @@
         // typeof (FileOpPlugin),
         // typeof (PidginPlugin),
         // typeof (ChatActions),
+        typeof (MpcActions),
 #if HAVE_ZEITGEIST
         typeof (ZeitgeistPlugin),
 #endif