[issue11279] test_posix and lack of "id -G" support - less noise required?

[Roundup Robot]

Roundup Robot added the comment:

New changeset 4ef517041573 by Jesus Cea in branch '2.7':
Closes #11279: test_posix and lack of "id -G" support - less noise required? 
(Solaris)
http://hg.python.org/cpython/rev/4ef517041573

New changeset 6889fb276d87 by Jesus Cea in branch '3.4':
Closes #11279: test_posix and lack of "id -G" support - less noise required? 
(Solaris)
http://hg.python.org/cpython/rev/6889fb276d87

New changeset 54f94e753269 by Jesus Cea in branch 'default':
MERGE: Closes #11279: test_posix and lack of "id -G" support - less noise 
required? (Solaris)
http://hg.python.org/cpython/rev/54f94e753269

--
nosy: +python-dev
resolution:  -> fixed
stage:  -> resolved
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue11279] test_posix and lack of "id -G" support - less noise required?

[Jesús Cea Avión]

Jesús Cea Avión added the comment:

I never saw this because I use GNU 'id' on my Solaris 10 machines. Taking care 
of this. Thanks for the report and the persistence :-)

--
assignee:  -> jcea
versions: +Python 2.7, Python 3.4, Python 3.5 -Python 3.2

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue11279] test_posix and lack of "id -G" support - less noise required?

[Mark Lawrence]

Mark Lawrence added the comment:

Can our Solaris gurus take this on please.

--
nosy: +BreamoreBoy

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue11279] test_posix and lack of "id -G" support - less noise required?

[Éric Araujo]

Éric Araujo  added the comment:

(Oops, this is for Solaris, not Mac.  I’m always confused about those funny 
OSes.)

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue11279] test_posix and lack of "id -G" support - less noise required?

[Éric Araujo]

Éric Araujo  added the comment:

The correct spelling of that is os.devnull.  platform.DEV_NULL exists only for 
backwards compatibility (not sure platform.py still has standalone releases).  
It is not documented and only used internally.  For a POSIX-only test like this 
one, hard-coding /dev/null is the simplest thing that works.

Thanks for the patch; our Mac experts will get to it.  Please generate unified 
diffs in the future.

--
nosy: +eric.araujo

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue11279] test_posix and lack of "id -G" support - less noise required?

[Antoine Pitrou]

Changes by Antoine Pitrou :


--
nosy: +jcea, ronaldoussoren

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue11279] test_posix and lack of "id -G" support - less noise required?

[Peter Bray]

New submission from Peter Bray :

Solaris 10's id(1M) does not support the "-G" option, so we get the following:

% ./python -m test test_posix 
[1/1] test_posix
id: illegal option -- G
Usage: id [-ap] [user]
1 test OK.

The code already tests that id -G (std) output, and assumes that id -G is not 
supported if the output is empty. 

QUESTION: Does the user compiling the code need to see the stderr output of id? 
After all the test is correctly skipped.

Fix:
% diff ./Lib/test/test_posix.py.FCS ./Lib/test/test_posix.py
378c378
< with os.popen('id -G') as idg:
---
> with os.popen('id -G 2> /dev/null') as idg:

Consideration: Lib/platform.py has introduced DEV_NULL but it does not seem to 
be used in other code or tests (beside platform.py).

Regards,

Peter

--
components: Tests
messages: 129021
nosy: illumino
priority: normal
severity: normal
status: open
title: test_posix and lack of "id -G" support - less noise required?
type: behavior
versions: Python 3.2

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com