Enable pixman-based rendering from config-file option

Pixman-based rendering can be enabled on the command-line using '--use-pixman'. An equivalent option should be available in the configure file (i.e., weston.ini).

A patch for this issue is here:

https://lists.freedesktop.org/archives/wayland-devel/2018-June/038376.html

added libweston API Pixman renderer labels

mentioned in commit tzimmermann/weston@146eeb9b

I wonder if it would make sense to take this one step further and add support for all backend specific command-line options in the config file, using a different "backend-drm/x11/headless/..." section per backend.

Perhaps. Backend specific sections make sense because not all backends have the same options, and depending on your backend you might want a different value, e.g. Pixman with DRM-backend while GL with X11-backend.

OTOH, the pixman option is general enough that it could as well be in [core] like the patch has it.

Then again, if we get yet another renderer at some point, then use-pixman=boolean in weston.ini becomes awkward, the option should be renderer=pixman. This combined with different backends supporting different renderers, it probably should have a backend specific section.

For the proposed patch, I have a style nitpick: please rearrange the code so that you're not adding even more code before the declaration of the options array. We try to not mix code and declarations.

If we go with the proposed patch, it is possible we need to change the weston.ini format later, so in that sense as well I would have slight preference to backend specific renderer option. But only slight, I'd be ok with the proposed approach as well.

Hi,

backend-specific renderers are certainly nicer and cleaner. But how would they interact with the respective command-line option --use-pixman? I guess the command line overrides the config file?

Command line always overrides the config file, yes. There is no reason to change the command line syntax right now, but if we get more renderers we might then.

mentioned in commit tzimmermann/weston@22dd67cc

closed via commit 22dd67cc

mentioned in issue #949 (closed)