Merge lp:~lool/ubuntu-app-launch/lp-1227632 into lp:ubuntu-app-launch/13.10

[Loïc Minier]

Fix LP #1227632 by directly joining the components of the Exec line split on % instead of joining them with spaces.

Try to fix LP #1227632 by directly joining the components of the Exec line split on % instead of joining them with spaces.

[Loïc Minier (lool)]

Wanted to add a test case, but there was none setup; happy to copy an existing test setup from another CMake project to apply it here.

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:58
http://jenkins.qa.ubuntu.com/job/upstart-app-launch-ci/74/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/upstart-app-launch-saucy-amd64-ci/76
    SUCCESS: http://jenkins.qa.ubuntu.com/job/upstart-app-launch-saucy-armhf-ci/75
    SUCCESS: http://jenkins.qa.ubuntu.com/job/upstart-app-launch-saucy-i386-ci/74

Click here to trigger a rebuild:
http://s-jenkins:8080/job/upstart-app-launch-ci/74/rebuild

review: Approve (continuous-integration)

[Ted Gould (ted)]

Works here, thanks for the fix!

review: Approve

=== modified file 'helpers.c'
--- helpers.c	2013-09-18 21:13:34 +0000
+++ helpers.c	2013-09-19 13:08:08 +0000
@@ -443,7 +443,7 @@
 		}
 	}
 
-	gchar * output = g_strjoinv(" ", (gchar **)outarray->data);
+	gchar * output = g_strjoinv(NULL, (gchar **)outarray->data);
 	g_array_free(outarray, TRUE);
 
 	g_free(single_uri);