Review Request 125375: Move FindAlsa from attic to find-modules

[Boudhayan Gupta]

> On Sept. 25, 2015, 2:37 a.m., Alex Merry wrote:
> > This file doesn't behave at all like find modules usually do. Instead of generating a config file, it should use components, and it shouldn't provide a version macro, but should parse out the version itself. It should be much more in the style of FindWayland.cmake.
> > 
> > In short, this module needs rewriting from scratch. Also, if there is only one project that needs it, I would suggest just shipping it directly in that project. ECM follows the "two or more users" rule in general - there needs to be two or more downstream users to justify shipping it in ECM.

The "two or more users" rule makes sense. I'll just add this directly into my project for the time being.


- Boudhayan


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


On Sept. 24, 2015, 8:51 p.m., Boudhayan Gupta wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125375/
> -----------------------------------------------------------
> 
> (Updated Sept. 24, 2015, 8:51 p.m.)
> 
> 
> Review request for Build System and Alex Merry.
> 
> 
> Repository: extra-cmake-modules
> 
> 
> Description
> -------
> 
> Move FindAlsa from attic to find-modules. libKCompactDisc would like to use it.
> 
> EDIT: For some reason git diff generates an empty file after I git add the files, but here's the git status output:
> 
> $: git status
> On branch master
> Your branch is up-to-date with 'origin/master'.
> Changes to be committed:
>   (use "git reset HEAD <file>..." to unstage)
> 
>         renamed:    attic/modules/FindAlsa.cmake -> find-modules/FindAlsa.cmake
>         renamed:    attic/modules/config-alsa.h.cmake -> find-modules/config-alsa.h.cmake
> 
> 
> Diffs
> -----
> 
>   attic/modules/FindAlsa.cmake 1bad812 
>   attic/modules/config-alsa.h.cmake fb9cb8c 
>   find-modules/FindAlsa.cmake PRE-CREATION 
>   find-modules/config-alsa.h.cmake PRE-CREATION 
> 
> Diff: https://git.reviewboard.kde.org/r/125375/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Boudhayan Gupta
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-buildsystem/attachments/20150924/a7bf0202/attachment-0001.html>