Re: removing "su" from coreutils [Re: Fwd: [PULL] su

--=-=-=
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable

Hi!

CCing debian-hurd FYI.


On Thu, 24 May 2012 15:21:01 +0200, Jim Meyering <[email protected]> wrote:
> P=C3=A1draig Brady wrote:
> > -------- Original Message --------
> > Subject: [PULL] su
> > Date: Thu, 24 May 2012 14:58:01 +0200
> > From: Ludwig Nussel <[email protected]>
> > To: [email protected]
> >
> > Hi,
> >
> > coreutils isn't really interested in maintaining su anymore.
> > Distributions have to add custom patches to fix bugs and add
> > features. The worst offender is probably the pam patch. Fortunately
> > we managed to sync the patch between openSUSE and Fedora meanwhile.
> > Over time the risk that the patches divert again increases though.
> > Since util-linux now contains a common login program it looks like a
> > good new home for su too.
> >
> > To integrate su into util-linux I've extracted the history of the su
> > program until the last GPLv2 version (they've changed to GPLv3
> > meanwhile) from coreutils' git and merged it with util-linux.
> > Then I've added the patches used in Fedora and openSUSE and fixed the
> > files to actually build.
> >
> > You can find the result in branch su-v1 (ff6b15d) at
> > [email protected]:lnussel/util-linux.git
> >
> > I've also filed a pull request on github:
> > https://github.com/karelzak/util-linux/pull/10
> >
> > Originally I had planned to implement separate tty allocation for
> > the child program on top of that to get rid of the evil setsid
> > patch. It turned out to be more complicated than I thought though. The
> > change would be too massive to backport anyways I guess. So the next
> > step would be to rip out the non-pam legacy stuff and clean up the code
> > to make it readable again.

> I would like to remove su.c, if/when possible from coreutils.
> The last time I proposed that, the Hurd was mentioned as a
> reason not to, since they relied on the su from coreutils.
>=20
> Is that still the case, and if so, can you (the Hurd) switch
> to the one from util-linux?

Yes, that appears to be fine -- despite its name, we're using a lot of
executables provided by util-linux.

I'll test the branch in the lnussel repository and report back.


Gr=C3=BC=C3=9Fe,
 Thomas

--=-=-=
Content-Type: application/pgp-signature

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)

iQEcBAEBAgAGBQJPv1AoAAoJENuKOtuXzphJrrIIAK5PRbUMGj/8MsL/R/Ggn8WQ
NE6E+rDvRgH0HU9UdyD5cgUbR3Alct35O9Ik8K+IGEBOmPsepJJs3hQg1qFiElMe
VVwrgQ7Rkb+Qk2Aq7vnevvPDmzgo7Owu2BwYY6DZL4Q7+awSzqiddRmr2W5PYbyj
Qv2UAPOCW9SS3CuCqLN7XF0Fa4lvqNyebYhjtRWqC4VJxkyDYHfsS6S03TJyuaXn
ywQz6gGW5FuOTYq15nOuG1jvPgdvM0/ivscoQz3KA+GNj0n9O2zh7colYGadwkxV
5EJfn1akZKjWDvtWkzN3Ytj7d59mRW/ceWes9+rupNxtsLd/UdhNj1qMbCwtR2w=
=jwST
-----END PGP SIGNATURE-----
--=-=-=--