Review Request: Add option to control PAM support

Oswald Buddenhagen ossi at kde.org
Wed Feb 1 17:46:29 UTC 2012


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.


More information about the Kde-buildsystem mailing list