Wayland userspace library initial 1.0.2
A message that is not in the archive (sent off-list or not yet received): [email protected]
A message that is not in the archive (sent off-list or not yet received): a8fd6c6c5b82bf945b16ac50a63420c918c6acb4.1355214185.git.alexandru.damian@intel.com
Quoted 2 lines · Alex DAMIAN
This patch is specifically based on work by Damien Lespiau for collabora, but updated to build wayland 1.0.2.
Damien Lespiau is an Intel employee, nothing to do with Collabora. Anything mentioning Damien predates Collabora's interest in Yocto.
Ross
A message that is not in the archive (sent off-list or not yet received): a6833e686fb416a1bfa86c1f07c1460b679519e5.1355214185.git.alexandru.damian@intel.com
Re: [PATCH 2/8] Add Wayland image type
The commit summary is wrong, this is a package group not image type.
Review
Quoted 3 lines · Alex DAMIAN
(quoted diff) +3
-0+RPROVIDES_${PN} = "task-wayland" +RREPLACES_${PN} = "task-wayland" +RCONFLICTS_${PN} = "task-wayland"
These are only needed if there was a task-wayland to provide a migration path, as there wasn't these can be removed.
Ross
A message that is not in the archive (sent off-list or not yet received): c59878a0a74222e7f777f8b32907080519c8d15c.1355214185.git.alexandru.damian@intel.com
Re: [PATCH 3/8] Enable Wayland distro feature by default
Quoted 2 lines · Alex DAMIAN
Conflicts:
meta/conf/distro/include/default-distrovars.inc
Remove this section from the message. Also unsure if we should enable it by default right now.
Ross
A message that is not in the archive (sent off-list or not yet received): aa5d85307115afbabead5d3f6e8fd976fe04c315.1355214185.git.alexandru.damian@intel.com
Re: [PATCH 4/8] mesa: merge internal libraries into mesa-dri package
Quoted 5 lines · Alex DAMIAN
From: Ross Burton <[email protected]>
libglapi and libgbm are specific to Mesa and it's unlikely that they'll be used on their own. Mesa 9 also adds another internal library, libdricore, so prepare for that by putting these mesa-specific libraries into the mesa-dri package.
I retracted this patch, drop it from your branch.
(see Saul's latest consolidated pull or master-under-test for the latest patches)
Ross
A message that is not in the archive (sent off-list or not yet received): e2c8ebaeba87f13c6e0e55a825bd397f36239b34.1355214185.git.alexandru.damian@intel.com
Re: [PATCH 5/8] mesa-dri: upgrade -git version to current master
Quoted 1 line · Alex DAMIAN
From: Ross Burton <[email protected]>
Again this is in m-u-t and the latest consolidated pull, so expect to rebase soon and see this disappear.
Ross
A message that is not in the archive (sent off-list or not yet received): 5f3ae296667128f06fab2b1660dcc120d81ad6fd.1355214185.git.alexandru.damian@intel.com
Re: [PATCH 6/8] Fixes for the mesa-dri recipe
Quoted 6 lines · Alex DAMIAN
From: Alexandru DAMIAN <[email protected]>
This set of changes fix packaging and compilation
options for the mesa-dri.It also affects priority so that the git version is preffered.
Why can't "use mesa-dri git" be part of the pre-requisites for now? We may be happy with a well chose git snapshot of Mesa but the current revision is literally just HEAD when I changed mesa-git for the new build system in master.
Review
Quoted 5 lines
DRIDRIVERS = "swrast"
(quoted diff) +2
-2-DRIDRIVERS_append_x86 = ",i915,i965" -DRIDRIVERS_append_x86-64 = ",i915,i965" +#DRIDRIVERS_append_x86 = ",i915,i965" +#DRIDRIVERS_append_x86-64 = ",i915,i965"
That's not a "fix"...
Ross
A message that is not in the archive (sent off-list or not yet received): 690eb2a2b704d5129634f956349f9d8fdc4864d8.1355214185.git.alexandru.damian@intel.com
Re: [PATCH 7/8] Adds wayland/weston recipe
Quoted 1 line · Alex DAMIAN
+EXTRA_OECONF += "--enable-clients --enable-drm-compositor --enable-xwayland"
--enable-drm-compositor is repeated in the (commented out) "kms" packageconfig.
XWayland should be a packageconfig option.
Quoted 1 line
+PACKAGECONFIG ??= "${@base_contains('DISTRO_FEATURES', 'wayland', 'wayland', '', d)} \
What's the point of this?
Review
Quoted 4 lines
(quoted diff) +4
-0+ ${@base_contains('DISTRO_FEATURES', 'x11', 'x11', '', d)} \ + ${@base_contains('DISTRO_FEATURES', 'opengles2', 'gles', '', d)} \ + " +#PACKAGECONFIG[kms] = "--enable-drm-compositor,--disable-drm-compositor,drm virtual/libgbm udev"
Presumably this is the "DRM compositor" that the commit message refers to?
Ross