From kde-frameworks-devel Mon Feb 24 15:10:20 2014 From: =?utf-8?q?Martin_Gr=C3=A4=C3=9Flin?= Date: Mon, 24 Feb 2014 15:10:20 +0000 To: kde-frameworks-devel Subject: Re: Review Request 116018: Add a FindWayland module Message-Id: <20140224151020.13755.88137 () probe ! kde ! org> X-MARC-Message: https://marc.info/?l=kde-frameworks-devel&m=139325463624480 MIME-Version: 1 Content-Type: multipart/mixed; boundary="--===============3687029972319467127==" --===============3687029972319467127== Content-Type: multipart/alternative; boundary="===============6313324007712529311==" --===============6313324007712529311== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit > On Feb. 24, 2014, 3:55 p.m., Alex Merry wrote: > > Same comments as for FindEGL.cmake: > > - what projects are using it? I guess KWin is one; any others? > > - it needs to set the DESCRIPTION and URL package properties (from FeatureSummary) > > - I've been aiming to create imported targets in new find modules, because they're easier to use (see FindSharedMimeInfo.cmake for an example, or the modules in kimageformats for better examples). and I can kind of give the same answer. Currently KWin is using it (we have a bad find module in kde-workspace) and for KInfoCenter I created a review request today which adds a dependency. I expect that in future almost every framework which has a FindXCB or FindX11 will also get a FindWayland. - Martin ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116018/#review50706 ----------------------------------------------------------- On Feb. 24, 2014, 2:52 p.m., Martin Gräßlin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/116018/ > ----------------------------------------------------------- > > (Updated Feb. 24, 2014, 2:52 p.m.) > > > Review request for KDE Frameworks and Alex Merry. > > > Repository: extra-cmake-modules > > > Description > ------- > > Add a FindWayland module > > Inspired by the FindXCB module it's using components to simplify > finding just the parts which are really required. > > > Diffs > ----- > > find-modules/FindWayland.cmake PRE-CREATION > > Diff: https://git.reviewboard.kde.org/r/116018/diff/ > > > Testing > ------- > > Changed kde-workspace to use this component based variant. > > > Thanks, > > Martin Gräßlin > > --===============6313324007712529311== Content-Type: text/html; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit

This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116018/

On February 24th, 2014, 3:55 p.m. CET, Alex Merry wrote:

Same comments as for FindEGL.cmake:
- what projects are using it?  I guess KWin is one; any others?
- it needs to set the DESCRIPTION and URL package properties (from FeatureSummary)
- I've been aiming to create imported targets in new find modules, because they're easier to use (see FindSharedMimeInfo.cmake for an example, or the modules in kimageformats for better examples).

and I can kind of give the same answer. Currently KWin is using it (we have a bad find module in kde-workspace) and for KInfoCenter I created a review request today which adds a dependency. I expect that in future almost every framework which has a FindXCB or FindX11 will also get a FindWayland.

- Martin

On February 24th, 2014, 2:52 p.m. CET, Martin Gräßlin wrote:

Review request for KDE Frameworks and Alex Merry.

By Martin Gräßlin.

Updated Feb. 24, 2014, 2:52 p.m.

Repository: extra-cmake-modules

Description

Add a FindWayland module

Inspired by the FindXCB module it's using components to simplify
finding just the parts which are really required.

Testing

Changed kde-workspace to use this component based variant.

Diffs

find-modules/FindWayland.cmake (PRE-CREATION)

--===============6313324007712529311==-- --===============3687029972319467127== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel --===============3687029972319467127==--