[Python-bugs-list] [ python-Bugs-775964 ] fix test_grp failing on RedHat 6.2

SourceForge.net noreply@sourceforge.net
Mon, 28 Jul 2003 16:24:33 -0700


Bugs item #775964, was opened at 2003-07-22 18:07
Message generated for change (Comment added) made by jbrouwers
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=105470&aid=775964&group_id=5470

Category: Extension Modules
Group: Python 2.3
Status: Open
Resolution: None
Priority: 6
Submitted By: Neal Norwitz (nnorwitz)
Assigned to: Barry A. Warsaw (bwarsaw)
Summary: fix test_grp failing on RedHat 6.2

Initial Comment:
I didn't really think this should go into 2.3, but I'll
let you make the decision.  This patch fixes the
test_grp failure on RedHat 6.2/Alpha (asmodean) in the
snake-farm.  I thought it was specific to RH 6.2,
apparently it's not.  If you add a + as the last line
in /etc/group the test will fail on RH 9 too.

Walter Doerwald may know more about how best to fix
this.  I'm not certain if it's really a problem in the
extension module or the test.
If you want to fix the test, the patch is included here:

     def check_value(self, value):
         # check that a grp tuple has the entries and
         # attributes promised by the docs
+        if value == ('+', None, 0, []):
+            # some libc's return the last line of +
+            return


----------------------------------------------------------------------

Comment By: Jean M. Brouwers (jbrouwers)
Date: 2003-07-28 18:24

Message:
Logged In: YES 
user_id=832557

The '+ ...' line at the end of the /etc/passwd file tells
the system to also pickup information from the appropriate
map in LDAP, or NIS (yellow pages).  See also section 7.2 on
this page:

<http://www.ibiblio.org/mdw/HOWTO/NIS-HOWTO/settingup_client.html>


----------------------------------------------------------------------

Comment By: Neal Norwitz (nnorwitz)
Date: 2003-07-28 16:56

Message:
Logged In: YES 
user_id=33168

I don't recall what the + is for either.  Does it have
something to do with YP/NIS?  I don't think this is really
critical so I'm droping the priority.  If  pwdmodule.c is
changed, it will be an interface change so it's questionable
for 2.3.1.

This was found on the snake farm, never reported by a user.

----------------------------------------------------------------------

Comment By: Walter Dörwald (doerwalter)
Date: 2003-07-28 13:36

Message:
Logged In: YES 
user_id=89016

pwdmodule.c hasn't changed in 7 months, so I'd say this is a
problem with the test. But ('+', None, 0, []) doesn't seem
to be a valid entry, so (although it's not a bug in Python)
maybe Python should hide this entry.

----------------------------------------------------------------------

Comment By: Barry A. Warsaw (bwarsaw)
Date: 2003-07-27 11:22

Message:
Logged In: YES 
user_id=12800

Remind me what + on a line means in /etc/group.  "man group"
on RH9 gives no clue.  I'm still nervous about it, can we
defer until 2.3.1?


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=105470&aid=775964&group_id=5470