<div dir="ltr">Oh thanks, i needed this. I can now stop modifying weston.ini back and forth. :)<div><br></div><div>Looks good to me.<br><div><br></div></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/4/1 Pier Luigi Fiorini <span dir="ltr"><<a href="mailto:pierluigi.fiorini@gmail.com" target="_blank">pierluigi.fiorini@gmail.com</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Let --modules override modules list and load desktop-shell.so as a<br>
fallback if a modules list is not specified neither by passing<br>
--modules nor with weston.ini.<br>
---<br>
 src/compositor.c | 11 +++++++----<br>
 1 file changed, 7 insertions(+), 4 deletions(-)<br>
<br>
diff --git a/src/compositor.c b/src/compositor.c<br>
index 1617d96..e2475c8 100644<br>
--- a/src/compositor.c<br>
+++ b/src/compositor.c<br>
@@ -3611,10 +3611,13 @@ int main(int argc, char *argv[])<br>
<br>
        setenv("WAYLAND_DISPLAY", socket_name, 1);<br>
<br>
-       if (load_modules(ec, modules, &argc, argv, config_file) < 0)<br>
-               goto out;<br>
-       if (load_modules(ec, option_modules, &argc, argv, config_file) < 0)<br>
-               goto out;<br>
+       if (option_modules == NULL) {<br>
+               if (load_modules(ec, modules, &argc, argv, config_file) < 0)<br>
+                       goto out;<br>
+       } else {<br>
+               if (load_modules(ec, option_modules, &argc, argv, config_file) < 0)<br>
+                       goto out;<br>
+       }<br>
<br>
        free(config_file);<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
1.8.2<br>
<br>
_______________________________________________<br>
wayland-devel mailing list<br>
<a href="mailto:wayland-devel@lists.freedesktop.org">wayland-devel@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/wayland-devel" target="_blank">http://lists.freedesktop.org/mailman/listinfo/wayland-devel</a><br>
</font></span></blockquote></div><br></div>