[prev in list] [next in list] [prev in thread] [next in thread] 

List:       pamldap
Subject:    Re: [pamldap] a few patches to pam_ldap 178
From:       Peter Marschall <peter () adpm ! de>
Date:       2005-04-04 6:29:26
Message-ID: 200504040829.26561.peter () adpm ! de
[Download RAW message or body]

Hi,

On Monday 04 April 2005 02:57, Alejandro Mery wrote:
> > * pam_ldap-178-autotools.patch
> >   This patch changes configure.in and Makefile.am so that ldap.conf gets
> >   installed in the place and with the name that is given to the configure
> >   option --with-ldap-conf-file.
> >   It also changes the names where the two sample programs chfn and chsh
> >   expect the config and the secret file according to the related options
> >   given to configure.
> >   For this 2nd part to work a change to the package is necessary: the
> >   original files chfn and chsh need to be renamed to chfn.sh and chsh.sh.
>
> why .sh? those are perl scripts.
> and considering you do sed replacements inside it i think .in is more
> correct.

Ooops ;-)
You are right.
I have given the code of these scripts only a really coarse glance.
Looking at it again I guess .pl or .in is really better suited.

Peter
-- 
Peter Marschall
eMail: peter@adpm.de
[prev in list] [next in list] [prev in thread] [next in thread]