Review Request: Add option to control PAM support

[Theo Chatzimichos]

On Wed, Feb 1, 2012 at 6:46 PM, Oswald Buddenhagen <ossi at kde.org> wrote:
> On Wed, Feb 01, 2012 at 02:24:59PM -0000, Theo Chatzimichos wrote:
>> > On Jan. 29, 2012, 4:42 p.m., Alexander Neundorf wrote:
>> > > The change in FindPAM.cmake is wrong.
>> > > FindPAM.cmake does not care whether WITH_PAM is TRUE or not, when called, it will search PAM.
>> > > If WITH_PAM is disabled, FindPAM.cmake will not be executed, and additionally all PAM_* variables will be set to empty.
>> > > It should work without the change in FindPAM.cmake.
>> >
>> > Michael Palimaka wrote:
>> >     That is intended behaviour. It should be possible to compile kcheckpass and KDM without PAM support, even if PAM is present on the system.
>>
>> Hello Alexander, please have a look at the following gentoo bugs [1] [2]. ossi verified the issue (from #kde-devel):
>> <ossi> tampakrap: seems like it got lost during the cmake migration. too bad
>> <ossi> actually, i was assuming that it is possible to manually set PAM_LIBRARY to NOTFOUND and cmake wouldn't try to find it again
>>
> i didn't verify anything. i said that the code to disable it explicitly
> is gone. if WITH_PAM is supposed to be handled automatically, then so be
> it. that's why i sent you to this list.

I apologize for misinterpreting your words then