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

zipimport.c is broken on 64-bit SusE AMD, here's a fix #39697

Closed
bensonbasis mannequin opened this issue Dec 11, 2003 · 3 comments
Closed

zipimport.c is broken on 64-bit SusE AMD, here's a fix #39697

bensonbasis mannequin opened this issue Dec 11, 2003 · 3 comments
Labels
extension-modules C modules in the Modules dir

Comments

@bensonbasis
Copy link
Mannequin

bensonbasis mannequin commented Dec 11, 2003

BPO 858317
Nosy @hyeshik
Files
  • zipimport.c.patch: patch to Moduleszipimport.c
  • 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-12-17.05:56:49.000>
    created_at = <Date 2003-12-11.15:27:42.000>
    labels = ['extension-modules']
    title = "zipimport.c is broken on 64-bit SusE AMD, here's a fix"
    updated_at = <Date 2003-12-17.05:56:49.000>
    user = 'https://bugs.python.org/bensonbasis'

    bugs.python.org fields:

    activity = <Date 2003-12-17.05:56:49.000>
    actor = 'hyeshik.chang'
    assignee = 'none'
    closed = True
    closed_date = None
    closer = None
    components = ['Extension Modules']
    creation = <Date 2003-12-11.15:27:42.000>
    creator = 'benson_basis'
    dependencies = []
    files = ['5737']
    hgrepos = []
    issue_num = 858317
    keywords = ['patch']
    message_count = 3.0
    messages = ['45035', '45036', '45037']
    nosy_count = 2.0
    nosy_names = ['hyeshik.chang', 'benson_basis']
    pr_nums = []
    priority = 'normal'
    resolution = 'fixed'
    stage = None
    status = 'closed'
    superseder = None
    type = None
    url = 'https://bugs.python.org/issue858317'
    versions = ['Python 2.4']

    @bensonbasis
    Copy link
    Mannequin Author

    bensonbasis mannequin commented Dec 11, 2003

    The code passed a -15 to a 'l' format, but -15 is not
    promoted to long, it's passed as an int. Casting the -15
    to long fixes the problem.

    @bensonbasis bensonbasis mannequin closed this as completed Dec 11, 2003
    @bensonbasis bensonbasis mannequin added the extension-modules C modules in the Modules dir label Dec 11, 2003
    @bensonbasis bensonbasis mannequin closed this as completed Dec 11, 2003
    @bensonbasis bensonbasis mannequin added the extension-modules C modules in the Modules dir label Dec 11, 2003
    @bensonbasis
    Copy link
    Mannequin Author

    bensonbasis mannequin commented Dec 11, 2003

    Logged In: YES
    user_id=876734

    I've marked this 2.4, but someone else (like someone at
    SuSe) might like to see it sooner.

    @hyeshik
    Copy link
    Contributor

    hyeshik commented Dec 17, 2003

    Logged In: YES
    user_id=55188

    This problem is already fixed in zipimport.c rev 1.17.

    @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
    extension-modules C modules in the Modules dir
    Projects
    None yet
    Development

    No branches or pull requests

    1 participant