Marc-Andre Lemburg <[email protected]> added the comment:
Éric Araujo wrote:
>
> Éric Araujo <[email protected]> added the comment:
>
> Is it guaranteed that the -b option will be present in every version of file?
Looking at the only use case of _syscmd_file(), it may not even be worth
the trouble of adding the -b option. Fixing the doc-string may be enough:
# Get data from the 'file' system command
if executable:
output = _syscmd_file(executable, '')
else:
output = ''
...
# Split the output into a list of strings omitting the filename
fileout = _architecture_split(output)[1:]
Note how architecture() already chops off the filename, so it
relies on the existing behavior, rather than the one described
in the doc-string of _syscmd_file().
Sorry, Victor, for not spotting this earlier on.
----------
title: platform.py: use -b option for file command in _syscmd_file() ->
platform.py: use -b option for file command in _syscmd_file()
_______________________________________
Python tracker <[email protected]>
<http://bugs.python.org/issue9560>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe:
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
