[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#7572: [PATCH] PAM support for su
From: |
Ludwig Nussel |
Subject: |
bug#7572: [PATCH] PAM support for su |
Date: |
Thu, 9 Jun 2011 15:45:29 +0200 |
User-agent: |
KMail/1.13.6 (Linux/2.6.37.1-1.2-default; KDE/4.6.0; x86_64; ; ) |
Jim Meyering wrote:
> Ludwig Nussel wrote:
> > Are there any concerns with the patch? It would be really nice to
> > have this merged upstream to avoid further fragmentation.
>
> The main concern is that by default coreutils doesn't even build su anymore.
Does that mean you intend to drop su from coreutils? If so is there
any suggested alternative? Should we move su to e.g. util-linux
instead?
> However, if this makes it easier on Fedora and Suse packagers, then
> I suppose it's worthwhile.
>
> If you'd like to pursue the matter, there are a few missing pieces:
>
> - Ensure that "make syntax-check" still passes with this patch.
> I see cpp indentation that may fail the test that runs cppi.
> That test is run only when cppi is installed, so you may have
> to install it.
>
> - it will need a ChangeLog entry, including attribution if you can
> dig that up.
Ok, I'll check both.
> - I haven't looked carefully, but considering the size, I'd be
> surprised if there is no need to document changes -- in
> coreutils.texi
Yes. Our package actually has a separate patch that modifies the
docu. For upstream the pam support is optional though so any
addition to coreutils.texi would need to be conditional I suppose.
So we'd need e.g. a coreutils.texi.in that gets rewritten by
configure.
> - include a NEWS entry
ok
> - tests would be most welcome, but I won't insist on those
Hmm, I'm not sure that's feasible. Tests would need to run as
root and they'd likely have to modify /etc/pam.d.
cu
Ludwig
--
(o_ Ludwig Nussel
//\
V_/_ http://www.suse.de/
SUSE LINUX Products GmbH, GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer, HRB
16746 (AG Nürnberg)