[PATCH weston] Add a --shell option to override the default desktop-shell

Kristian Høgsberg hoegsberg at gmail.com
Mon Aug 26 22:08:47 PDT 2013


On Sun, Aug 25, 2013 at 03:42:11PM +0200, Giulio Camuffo wrote:
> Looks good and works good for me.

Thanks for checking Giulio, I wanted this to work for your use case
too.  Patch applied.

Kristian

> 2013/8/23 Jason Ekstrand <jason at jlekstrand.net>
> 
> > Signed-off-by: Jason Ekstrand <jason at jlekstrand.net>
> > ---
> >  src/compositor.c | 12 ++++++++++--
> >  weston.ini       |  3 ++-
> >  2 files changed, 12 insertions(+), 3 deletions(-)
> >
> > diff --git a/src/compositor.c b/src/compositor.c
> > index e9ba0fd..bb66fc7 100644
> > --- a/src/compositor.c
> > +++ b/src/compositor.c
> > @@ -3224,6 +3224,7 @@ usage(int error_code)
> >                 "  -B, --backend=MODULE\tBackend module, one of
> > drm-backend.so,\n"
> >                 "\t\t\t\tfbdev-backend.so, x11-backend.so or\n"
> >                 "\t\t\t\twayland-backend.so\n"
> > +               "  --shell=MODULE\tShell module, defaults to
> > desktop-shell.so\n"
> >                 "  -S, --socket=NAME\tName of socket to listen on\n"
> >                 "  -i, --idle-time=SECS\tIdle time in seconds\n"
> >                 "  --modules\t\tLoad the comma-separated list of modules\n"
> > @@ -3311,6 +3312,7 @@ int main(int argc, char *argv[])
> >                                  struct weston_config *config);
> >         int i, config_fd;
> >         char *backend = NULL;
> > +       char *shell = NULL;
> >         char *modules, *option_modules = NULL;
> >         char *log = NULL;
> >         int32_t idle_time = 300;
> > @@ -3322,6 +3324,7 @@ int main(int argc, char *argv[])
> >
> >         const struct weston_option core_options[] = {
> >                 { WESTON_OPTION_STRING, "backend", 'B', &backend },
> > +               { WESTON_OPTION_STRING, "shell", 0, &shell },
> >                 { WESTON_OPTION_STRING, "socket", 'S', &socket_name },
> >                 { WESTON_OPTION_INTEGER, "idle-time", 'i', &idle_time },
> >                 { WESTON_OPTION_STRING, "modules", 0, &option_modules },
> > @@ -3380,8 +3383,7 @@ int main(int argc, char *argv[])
> >         close(config_fd);
> >
> >         section = weston_config_get_section(config, "core", NULL, NULL);
> > -       weston_config_section_get_string(section, "modules",
> > -                                        &modules, "desktop-shell.so");
> > +       weston_config_section_get_string(section, "modules", &modules, "");
> >
> >         backend_init = load_module(backend, "backend_init");
> >         if (!backend_init)
> > @@ -3400,6 +3402,12 @@ int main(int argc, char *argv[])
> >
> >         setenv("WAYLAND_DISPLAY", socket_name, 1);
> >
> > +       if (!shell)
> > +               weston_config_section_get_string(section, "shell", &shell,
> > +                                                "desktop-shell.so");
> > +       if (load_modules(ec, shell, &argc, argv) < 0)
> > +               goto out;
> > +
> >         if (load_modules(ec, modules, &argc, argv) < 0)
> >                 goto out;
> >         if (load_modules(ec, option_modules, &argc, argv) < 0)
> > diff --git a/weston.ini b/weston.ini
> > index f2abceb..60d10f6 100644
> > --- a/weston.ini
> > +++ b/weston.ini
> > @@ -1,5 +1,6 @@
> >  [core]
> > -#modules=desktop-shell.so,xwayland.so,cms-colord.so
> > +#modules=xwayland.so,cms-colord.so
> > +#shell=desktop-shell.so
> >
> >  [shell]
> >  background-image=/usr/share/backgrounds/gnome/Aqua.jpg
> > --
> > 1.8.3.1
> >
> > _______________________________________________
> > wayland-devel mailing list
> > wayland-devel at lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/wayland-devel
> >

> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel



More information about the wayland-devel mailing list