[Wayland-bugs] [Bug 87412] Weston now silently quits when the user specifies to load desktop-shell.so

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=87412

Pekka Paalanen <ppaalanen at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |NOTABUG
             Status|NEW                         |RESOLVED

--- Comment #4 from Yong Bakos <junk at humanoriented.com> ---
I'd like to vote for this bug to be closed.

The --modules option is documented, in its man, to be used only for testing. A
developer using the --modules option would know to simply re-run weston without
specifying the particular module.

End-users incorrectly passing an 'invalid' module as a --modules option would
be advised to observe what is documented in man.

The original context of this bug is due to incorrectly specifying a --shell
module as a --modules option.

Otherwise, to honor this use case, we'd have to implement additional logic in
compositor.c that:

1) Checks the module to be sure it is a valid option for --modules, and if it
isn't, prints a helpful message before quitting.
2) Ignores the loading of the valid, specified --modules module and continues,
rather  than causing weston_load_module to return NULL and quitting.

--- Comment #5 from Pekka Paalanen <ppaalanen at gmail.com> ---
We could also make Weston discriminate between module types (backend, shell,
generic) by having a type-specific symbol name instead of just "module_init",
which would help weston quit with a better error message. I still think
quitting is the right thing to do.

Closing.

If you want better errors, that would be a new report. Preferably a new Task in
Phab.fd.o.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/wayland-bugs/attachments/20160425/2b709587/attachment.html>