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

python msilib view.fetch is not returning none #76245

Closed
udaykiran mannequin opened this issue Nov 17, 2017 · 4 comments
Closed

python msilib view.fetch is not returning none #76245

udaykiran mannequin opened this issue Nov 17, 2017 · 4 comments
Labels
OS-windows type-bug An unexpected behavior, bug, or error

Comments

@udaykiran
Copy link
Mannequin

udaykiran mannequin commented Nov 17, 2017

BPO 32064
Nosy @pfmoore, @tjguk, @berkerpeksag, @zware, @zooba
Superseder
  • bpo-1102: Add support for _msi.Record.GetString() and _msi.Record.GetInteger()
  • Files
  • MSIFileRead.py: source code where the bug is pointed
  • 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 2017-11-19.06:51:49.072>
    created_at = <Date 2017-11-17.20:37:30.115>
    labels = ['type-bug', 'OS-windows']
    title = 'python msilib view.fetch is not returning none'
    updated_at = <Date 2017-11-19.06:51:48.981>
    user = 'https://bugs.python.org/udaykiran'

    bugs.python.org fields:

    activity = <Date 2017-11-19.06:51:48.981>
    actor = 'berker.peksag'
    assignee = 'none'
    closed = True
    closed_date = <Date 2017-11-19.06:51:49.072>
    closer = 'berker.peksag'
    components = ['Windows']
    creation = <Date 2017-11-17.20:37:30.115>
    creator = 'uday kiran'
    dependencies = []
    files = ['47271']
    hgrepos = []
    issue_num = 32064
    keywords = []
    message_count = 4.0
    messages = ['306456', '306457', '306468', '306497']
    nosy_count = 6.0
    nosy_names = ['paul.moore', 'tim.golden', 'berker.peksag', 'zach.ware', 'steve.dower', 'uday kiran']
    pr_nums = []
    priority = 'normal'
    resolution = 'duplicate'
    stage = 'resolved'
    status = 'closed'
    superseder = '1102'
    type = 'behavior'
    url = 'https://bugs.python.org/issue32064'
    versions = ['Python 3.6']

    @udaykiran
    Copy link
    Mannequin Author

    udaykiran mannequin commented Nov 17, 2017

    Please see my script attached.

    Once the records are completed, the View.Fetch() function should give a none value but intsead giving runtime error: _msi.MSIError: unknown error 103

    It is already mentioned in the bugs.
    https://bugs.python.org/issue29364
    https://bugs.python.org/issue1102

    The comments mentioned as it will be fixed in the coming release.
    But it is a long back comment,but still not fixed.

    Any update on this?

    @udaykiran udaykiran mannequin added type-crash A hard crash of the interpreter, possibly with a core dump OS-windows labels Nov 17, 2017
    @udaykiran
    Copy link
    Mannequin Author

    udaykiran mannequin commented Nov 17, 2017

    It seems you have added some patches as .c files, but how do we apply those on windows machines?
    No documentation related to it.

    https://bugs.python.org/file24795/_msi_fetch.patch.txt
    https://bugs.python.org/file11840/_msi.patch.txt

    Could you please help me how to apply these patches on windows environment.

    @udaykiran
    Copy link
    Mannequin Author

    udaykiran mannequin commented Nov 18, 2017

    I tried adding the bugs to the python source code and compile it by using the steps in https://docs.python.org/devguide/ and the fix in https://bugs.python.org/file24795/_msi_fetch.patch.txt, but ended up with many compilation errors

    @berkerpeksag
    Copy link
    Member

    bpo-1102 is still open so this bug is not fixed yet. I've just opened PR 4459 to fix it.

    @berkerpeksag berkerpeksag added type-bug An unexpected behavior, bug, or error and removed type-crash A hard crash of the interpreter, possibly with a core dump labels Nov 19, 2017
    @ezio-melotti ezio-melotti transferred this issue from another repository Apr 10, 2022
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Labels
    OS-windows type-bug An unexpected behavior, bug, or error
    Projects
    None yet
    Development

    No branches or pull requests

    1 participant