dummy user for testsuites

2007-04-02 Thread Jeremy Huntwork
Hello,

Currently in chapter 6 coreutils we have instructions to append a dummy
user to the /etc/{group,passwd} files so that we can run its test suite
properly. The test suite for bash also calls for an unprivileged user.
See: http://wiki.linuxfromscratch.org/lfs/ticket/1877

Does anyone see a problem with adding that dummy user/group to the
original /etc/{passwd,group} files, perhaps with a note explaining its
purpose? At the end of chapter 6 we would remove the user and group.

I'd like to get opinions before I work on the above ticket.

--
JH
-- 
http://linuxfromscratch.org/mailman/listinfo/lfs-dev
FAQ: http://www.linuxfromscratch.org/faq/
Unsubscribe: See the above information page


Re: dummy user for testsuites

2007-04-02 Thread M.Canales.es
El Lunes, 2 de Abril de 2007 23:10, Jeremy Huntwork escribió:

 Does anyone see a problem with adding that dummy user/group to the
 original /etc/{passwd,group} files, perhaps with a note explaining its
 purpose? At the end of chapter 6 we would remove the user and group.


Why not to use the nobody user for that tests?

There is recent thread about this same issue started by Robert Connolly:

http://linuxfromscratch.org/pipermail/lfs-dev/2007-March/059171.html

-- 
Manuel Canales Esparcia
Usuario de LFS nº2886:   http://www.linuxfromscratch.org
LFS en castellano: http://www.escomposlinux.org/lfs-es http://www.lfs-es.info
TLDP-ES:   http://es.tldp.org
-- 
http://linuxfromscratch.org/mailman/listinfo/lfs-dev
FAQ: http://www.linuxfromscratch.org/faq/
Unsubscribe: See the above information page


Re: dummy user for testsuites

2007-04-02 Thread Jeremy Huntwork
On Mon, Apr 02, 2007 at 11:41:19PM +0200, M.Canales.es wrote:
 Why not to use the nobody user for that tests?
 
 There is recent thread about this same issue started by Robert Connolly:
 
 http://linuxfromscratch.org/pipermail/lfs-dev/2007-March/059171.html

In principle that's fine - I'd rather not add unnecessary users and
groups to the system. But then the nobody user is added with a shell of
/bin/false. I suppose we could change that temporarily. I'm just not
sure what the best answer is yet.

--
JH
-- 
http://linuxfromscratch.org/mailman/listinfo/lfs-dev
FAQ: http://www.linuxfromscratch.org/faq/
Unsubscribe: See the above information page


Re: dummy user for testsuites

2007-04-02 Thread Jeremy Huntwork
On Mon, Apr 02, 2007 at 11:41:19PM +0200, M.Canales.es wrote:
 Why not to use the nobody user for that tests?
 
 There is recent thread about this same issue started by Robert Connolly:
 
 http://linuxfromscratch.org/pipermail/lfs-dev/2007-March/059171.html

Woops. Disregard my last message. Obviously I didn't recall how this
thread started and I didn't look at it again before my last reply.
Looking into it further...

--
JH
-- 
http://linuxfromscratch.org/mailman/listinfo/lfs-dev
FAQ: http://www.linuxfromscratch.org/faq/
Unsubscribe: See the above information page


Re: dummy user for testsuites

2007-04-02 Thread Bruce Dubbs
Jeremy Huntwork wrote:
 Hello,
 
 Currently in chapter 6 coreutils we have instructions to append a dummy
 user to the /etc/{group,passwd} files so that we can run its test suite
 properly. The test suite for bash also calls for an unprivileged user.
 See: http://wiki.linuxfromscratch.org/lfs/ticket/1877
 
 Does anyone see a problem with adding that dummy user/group to the
 original /etc/{passwd,group} files, perhaps with a note explaining its
 purpose? At the end of chapter 6 we would remove the user and group.
 
 I'd like to get opinions before I work on the above ticket.

Personally, I think adding/deleting the temp users should be
encapsulated in the specific pages that need them.  That was someone
wants to update or evaluate a specific package on a working systems,
they have all the instructions in one place.

After all a cat and a sed to add and delete the users should not be that
big a deal.

  -- Bruce
-- 
http://linuxfromscratch.org/mailman/listinfo/lfs-dev
FAQ: http://www.linuxfromscratch.org/faq/
Unsubscribe: See the above information page


Re: dummy user for testsuites

2007-04-02 Thread Jeremy Huntwork
On Mon, Apr 02, 2007 at 05:42:29PM -0500, Bruce Dubbs wrote:
 Personally, I think adding/deleting the temp users should be
 encapsulated in the specific pages that need them.  That was someone
 wants to update or evaluate a specific package on a working systems,
 they have all the instructions in one place.
 
 After all a cat and a sed to add and delete the users should not be that
 big a deal.

Yes, I thought someone might say as much. And I can see your point.
Still, the cat and sed - in fact the entire addition of another user -
seems unnecessary. After re-reading the thread Manuel linked to where
Robert suggests using the now-present nobody user, I think that's the
way to go.

And the addition of 'su' to /tools/bin in chapter 5 (as was suggested in
ticket #1877) is unnecessary now because coreutils is installed before
bash in chapter 6.

What I think I'm going to do - unless someone has on objection - is add
the commands 'su nobody -s /bin/bash -c ...' to both bash and
coreutils in chapter 6.

Will just do a quick test run here first before I commit.

--
JH
-- 
http://linuxfromscratch.org/mailman/listinfo/lfs-dev
FAQ: http://www.linuxfromscratch.org/faq/
Unsubscribe: See the above information page


Re: dummy user for testsuites

2007-04-02 Thread Robert Connolly
On Monday April 2 2007 22:22, Jeremy Huntwork wrote:
 And the addition of 'su' to /tools/bin in chapter 5 (as was suggested in
 ticket #1877) is unnecessary now because coreutils is installed before
 bash in chapter 6.

 What I think I'm going to do - unless someone has on objection - is add
 the commands 'su nobody -s /bin/bash -c ...' to both bash and
 coreutils in chapter 6.

Chapter 6's Coreutils /bin/su will be overwritten by Shadow's, which isn't a 
big deal, but it's not very elegant. I personally prefer to install Chapter 
5's /tools/bin/tools-su simply so that packages are not overwriting 
eachothers files in Chapter 6... it's a bad precedent. It also adds the 
complication of writing the Coreutils page so that users can reinstall 
Coreutils after Shadow, without losing the PAM-compatible 'su' from Shadow.

robert


pgp7GmdMArC3D.pgp
Description: PGP signature
-- 
http://linuxfromscratch.org/mailman/listinfo/lfs-dev
FAQ: http://www.linuxfromscratch.org/faq/
Unsubscribe: See the above information page


Re: dummy user for testsuites

2007-04-02 Thread Jeremy Huntwork
On Mon, Apr 02, 2007 at 11:47:57PM -0400, Robert Connolly wrote:
 Chapter 6's Coreutils /bin/su will be overwritten by Shadow's, which isn't a 
 big deal, but it's not very elegant. I personally prefer to install Chapter 
 5's /tools/bin/tools-su simply so that packages are not overwriting 
 eachothers files in Chapter 6... it's a bad precedent. It also adds the 
 complication of writing the Coreutils page so that users can reinstall 
 Coreutils after Shadow, without losing the PAM-compatible 'su' from Shadow.

Ah, of course you're right. I forgot about the suppress patch which
keeps the su from coreutils from being installed in chapter 6. We would
still need the temporary su from chapter 5. It would be no problem to
drop that in, though.

--
JH
-- 
http://linuxfromscratch.org/mailman/listinfo/lfs-dev
FAQ: http://www.linuxfromscratch.org/faq/
Unsubscribe: See the above information page