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.

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

Regression exceptions for cygwin #36548

Closed
gswagere mannequin opened this issue May 3, 2002 · 14 comments
Closed

Regression exceptions for cygwin #36548

gswagere mannequin opened this issue May 3, 2002 · 14 comments
Labels
tests Tests in the Lib/test dir

Comments

@gswagere
Copy link
Mannequin

gswagere mannequin commented May 3, 2002

BPO 551977
Nosy @tim-one, @loewis
Files
  • regrtest.py.patch
  • regrtest.py.diff
  • regrtest.py.diff: skip test_ossaudiodev patch
  • 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 = None
    closed_at = <Date 2003-02-05.16:52:34.000>
    created_at = <Date 2002-05-03.19:19:43.000>
    labels = ['tests']
    title = 'Regression exceptions for cygwin'
    updated_at = <Date 2003-02-05.16:52:34.000>
    user = 'https://bugs.python.org/gswagere'

    bugs.python.org fields:

    activity = <Date 2003-02-05.16:52:34.000>
    actor = 'jlt63'
    assignee = 'jlt63'
    closed = True
    closed_date = None
    closer = None
    components = ['Tests']
    creation = <Date 2002-05-03.19:19:43.000>
    creator = 'gsw_agere'
    dependencies = []
    files = ['4226', '4227', '4228']
    hgrepos = []
    issue_num = 551977
    keywords = ['patch']
    message_count = 14.0
    messages = ['39842', '39843', '39844', '39845', '39846', '39847', '39848', '39849', '39850', '39851', '39852', '39853', '39854', '39855']
    nosy_count = 5.0
    nosy_names = ['tim.peters', 'loewis', 'nnorwitz', 'jlt63', 'gsw_agere']
    pr_nums = []
    priority = 'normal'
    resolution = 'accepted'
    stage = None
    status = 'closed'
    superseder = None
    type = None
    url = 'https://bugs.python.org/issue551977'
    versions = ['Python 2.3']

    @gswagere
    Copy link
    Mannequin Author

    gswagere mannequin commented May 3, 2002

    This patch updates regrtest.py to understand which
    tests are normally skipped under Cygwin. The list of
    tests was verified with the Cygwin Python maintainer.

    @gswagere gswagere mannequin closed this as completed May 3, 2002
    @gswagere gswagere mannequin assigned jlt63 May 3, 2002
    @gswagere gswagere mannequin added the tests Tests in the Lib/test dir label May 3, 2002
    @gswagere gswagere mannequin closed this as completed May 3, 2002
    @gswagere gswagere mannequin assigned jlt63 May 3, 2002
    @gswagere gswagere mannequin added the tests Tests in the Lib/test dir label May 3, 2002
    @loewis
    Copy link
    Mannequin

    loewis mannequin commented May 6, 2002

    Logged In: YES
    user_id=21627

    I'd like to hear the rationale for skipping

    test_email_codecs
    test_largefile
    test_locale
    test_unicode_file
    test_winreg

    All of those *should* work just fine; if they don't, this
    indicates a bug in this port.

    @gswagere
    Copy link
    Mannequin Author

    gswagere mannequin commented May 7, 2002

    Logged In: YES
    user_id=329402

    test_email_codecs is skipped unless the optional Japanese
    codecs are installed (this is also in the exceptions list
    for win32 and linux2).

    Cygwin does not currently support large files, locales, or
    unicode file system semantics (test_unicode_file is also in
    the exceptions list for all but win32, test_locale is in
    most of them, and test_largefile is in *all* of them).

    test_winreg is a stickier one. This is clearly specific to
    Windows, and the goal of Cygwin is to present a UNIX-like
    interface, not a Windows-like interface. There are patches
    that will enable this support under Cygwin Python, although
    they pull in quite a bit of the Windows-specific code and
    would create maintenance issues for Python's codebase. At
    present this is considered unsupported in Cygwin Python.

    There are three additional tests that are in all or almost
    all of the exception lists:
    test_curses
    test_socket_ssl
    test_socketserver

    These are disabled unless "-u" is used to specifically
    enable them. It appears that these should also be added to
    the list.

    @loewis
    Copy link
    Mannequin

    loewis mannequin commented May 7, 2002

    Logged In: YES
    user_id=21627

    I'm quite unhappy with the entire _expectations mechanism.
    E.g. the fact that test_email_codecs is skipped has
    *nothing* to do with the distribution being build on
    Windows, or with Cygwin; I'd rather see the expected fails
    to take into account available resources (e.g. absence of
    the Japanese codecs is a good reason for skipping the tests).

    @loewis
    Copy link
    Mannequin

    loewis mannequin commented Dec 4, 2002

    Logged In: YES
    user_id=21627

    Jason, can you take a look and apply the patch if it looks ok?

    It appears that the official policy is that all tests
    that "normally" can't run should be listed as expected skips.

    @jlt63
    Copy link
    Mannequin

    jlt63 mannequin commented Dec 5, 2002

    Logged In: YES
    user_id=86216

    OK to check in the slightly different, attached patch?

    The differences between Jerry's and mine are the following:

    1. added the following as suggested by Jerry:

    test_curses
    test_socket_ssl
    test_socketserver

    1. removed test_bsddb because it is supported now

    @loewis
    Copy link
    Mannequin

    loewis mannequin commented Dec 5, 2002

    Logged In: YES
    user_id=21627

    Sure. Go ahead!

    @jlt63
    Copy link
    Mannequin

    jlt63 mannequin commented Dec 5, 2002

    Logged In: YES
    user_id=86216

    Checked in as Lib/test/regrtest.py 1.111.

    @jlt63
    Copy link
    Mannequin

    jlt63 mannequin commented Dec 5, 2002

    Logged In: YES
    user_id=86216

    I just updated from CVS and noticed that I missed
    test_bsddb3. Can I add it as an expected skip (like
    test_curses)?

    @tim-one
    Copy link
    Member

    tim-one commented Dec 5, 2002

    Logged In: YES
    user_id=31435

    I added test_bsddb3 to the cygwin skip list, so you don't have
    to.

    @jlt63
    Copy link
    Mannequin

    jlt63 mannequin commented Dec 5, 2002

    Logged In: YES
    user_id=86216

    Thanks Tim!

    @jlt63
    Copy link
    Mannequin

    jlt63 mannequin commented Feb 5, 2003

    Logged In: YES
    user_id=86216

    OK, to apply the skip test_ossaudiodev patch?

    @nnorwitz
    Copy link
    Mannequin

    nnorwitz mannequin commented Feb 5, 2003

    Logged In: YES
    user_id=33168

    Yes.

    @jlt63
    Copy link
    Mannequin

    jlt63 mannequin commented Feb 5, 2003

    Logged In: YES
    user_id=86216

    Checked in as Lib/test/regrtest.py 1.123.

    @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
    Labels
    tests Tests in the Lib/test dir
    Projects
    None yet
    Development

    No branches or pull requests

    1 participant