Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix to pty.spawn error on Linux #36898

Closed
rasjidw mannequin opened this issue Jul 15, 2002 · 3 comments
Closed

fix to pty.spawn error on Linux #36898

rasjidw mannequin opened this issue Jul 15, 2002 · 3 comments
Assignees
@loewis
Labels
stdlib Python modules in the Lib dir

Comments

@rasjidw
Copy link
Mannequin

rasjidw mannequin commented Jul 15, 2002

BPO 581705
Nosy @loewis
Files
  • patch-pty.py: Patch to fix pty.spawn (on Linux RH 7.3)
  • ptypatch2: Revised patch to pty.py
    		•

    Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.

    Show more details

    GitHub fields:

    assignee = 'https://github.com/loewis'
closed_at = <Date 2002-07-28.09:43:33.000>
created_at = <Date 2002-07-15.14:34:48.000>
labels = ['library']
title = 'fix to pty.spawn error on Linux'
updated_at = <Date 2002-07-28.09:43:33.000>
user = 'https://bugs.python.org/rasjidw'

    bugs.python.org fields:

    activity = <Date 2002-07-28.09:43:33.000>
actor = 'loewis'
assignee = 'loewis'
closed = True
closed_date = None
closer = None
components = ['Library (Lib)']
creation = <Date 2002-07-15.14:34:48.000>
creator = 'rasjidw'
dependencies = []
files = ['4432', '4433']
hgrepos = []
issue_num = 581705
keywords = ['patch']
message_count = 3.0
messages = ['40593', '40594', '40595']
nosy_count = 2.0
nosy_names = ['loewis', 'rasjidw']
pr_nums = []
priority = 'normal'
resolution = 'accepted'
stage = None
status = 'closed'
superseder = None
type = None
url = 'https://bugs.python.org/issue581705'
versions = ['Python 2.2']
    @rasjidw
    Copy link
    Mannequin Author

    rasjidw mannequin commented Jul 15, 2002

    I submitted a bug report, id 581698 called 'pty.spawn -
    wrong error caught'.

    System: RedHat Linux 7.3, using Python2.

    About a year ago, the final 'except' statement was
    changed to catch IOError rather than just error.
    However, at least on my system, the os.read call raises
    an OSError, not an IOError. Therefore, the wrong error
    type is now caught.

    Patch attached.

    Rasjid.

    @rasjidw rasjidw mannequin closed this as completed Jul 15, 2002
    @rasjidw rasjidw mannequin assigned loewis Jul 15, 2002
    @rasjidw rasjidw mannequin added the stdlib Python modules in the Lib dir label Jul 15, 2002
    @rasjidw rasjidw mannequin closed this as completed Jul 15, 2002
    @rasjidw rasjidw mannequin assigned loewis Jul 15, 2002
    @rasjidw rasjidw mannequin added the stdlib Python modules in the Lib dir label Jul 15, 2002
    @rasjidw
    Copy link
    Mannequin Author

    rasjidw mannequin commented Jul 27, 2002

    Logged In: YES
    user_id=39640

    Actually, a bit more testing revealed some more errors when
    the main process had its standard input and output something
    other than a tty.

    I attach my second version of the patch.

    Rasjid.

    @loewis
    Copy link
    Mannequin

    loewis mannequin commented Jul 28, 2002

    Logged In: YES
    user_id=21627

    Thanks for the patch. Committed (in a slightly modified
    form) as pty.py 1.12.

    @ezio-melotti ezio-melotti transferred this issue from another repository Apr 9, 2022
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Assignees

    @loewis loewis

    Labels
    stdlib Python modules in the Lib dir
    Projects
    None yet
    Milestone
    No milestone
    Development

    No branches or pull requests
    0 participants