From kde-frameworks-devel Mon Feb 24 14:51:49 2014 From: "Alex Merry" Date: Mon, 24 Feb 2014 14:51:49 +0000 To: kde-frameworks-devel Subject: Re: Review Request 116014: Add a FindEgl module Message-Id: <20140224145149.14197.55499 () probe ! kde ! org> X-MARC-Message: https://marc.info/?l=kde-frameworks-devel&m=139325352824043 MIME-Version: 1 Content-Type: multipart/mixed; boundary="--===============3760951488477917827==" --===============3760951488477917827== Content-Type: multipart/alternative; boundary="===============7210758437349091280==" --===============7210758437349091280== Content-Type: text/plain; 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/116014/#review50705 ----------------------------------------------------------- Three things: - 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). - Alex Merry On Feb. 24, 2014, 1:08 p.m., Martin Gräßlin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/116014/ > ----------------------------------------------------------- > > (Updated Feb. 24, 2014, 1:08 p.m.) > > > Review request for KDE Frameworks and Alex Merry. > > > Repository: extra-cmake-modules > > > Description > ------- > > Add a FindEgl module > > This module finds the egl library through pkg-config. > > > Diffs > ----- > > find-modules/FindEgl.cmake PRE-CREATION > > Diff: https://git.reviewboard.kde.org/r/116014/diff/ > > > Testing > ------- > > Adjusted KWin to find egl through this cmake module. Picks up egl, compiles and links. > > > Thanks, > > Martin Gräßlin > > --===============7210758437349091280== 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/116014/

Three things:
- 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).

- Alex Merry

On February 24th, 2014, 1:08 p.m. UTC, Martin Gräßlin wrote:

Review request for KDE Frameworks and Alex Merry.

By Martin Gräßlin.

Updated Feb. 24, 2014, 1:08 p.m.

Repository: extra-cmake-modules

Description

Add a FindEgl module

This module finds the egl library through pkg-config.

Testing

Adjusted KWin to find egl through this cmake module. Picks up egl, compiles and links.

Diffs

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

View Diff

--===============7210758437349091280==-- --===============3760951488477917827== 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 --===============3760951488477917827==--