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

Current Android does not have posix_spawn #76886

Closed
yan12125 mannequin opened this issue Jan 29, 2018 · 4 comments
Closed

Current Android does not have posix_spawn #76886

yan12125 mannequin opened this issue Jan 29, 2018 · 4 comments
Labels
3.7 (EOL) end of life build The build process and cross-build

Comments

@yan12125
Copy link
Mannequin

yan12125 mannequin commented Jan 29, 2018

BPO 32705
Nosy @gpshead, @vstinner, @moreati, @yan12125
PRs
  • bpo-32705: Current Android does not have posix_spawn #5413
  • Files
  • build.log
  • 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 2018-01-30.07:40:36.063>
    created_at = <Date 2018-01-29.07:42:18.481>
    labels = ['build', '3.7']
    title = 'Current Android does not have posix_spawn'
    updated_at = <Date 2018-01-30.07:40:36.061>
    user = 'https://github.com/yan12125'

    bugs.python.org fields:

    activity = <Date 2018-01-30.07:40:36.061>
    actor = 'gregory.p.smith'
    assignee = 'none'
    closed = True
    closed_date = <Date 2018-01-30.07:40:36.063>
    closer = 'gregory.p.smith'
    components = ['Cross-Build']
    creation = <Date 2018-01-29.07:42:18.481>
    creator = 'yan12125'
    dependencies = []
    files = ['47416']
    hgrepos = []
    issue_num = 32705
    keywords = ['patch']
    message_count = 4.0
    messages = ['311078', '311085', '311249', '311251']
    nosy_count = 4.0
    nosy_names = ['gregory.p.smith', 'vstinner', 'Alex.Willmer', 'yan12125']
    pr_nums = ['5413']
    priority = 'normal'
    resolution = 'fixed'
    stage = 'resolved'
    status = 'closed'
    superseder = None
    type = 'compile error'
    url = 'https://bugs.python.org/issue32705'
    versions = ['Python 3.7']

    @yan12125
    Copy link
    Mannequin Author

    yan12125 mannequin commented Jan 29, 2018

    Currently Android does not have posix_spawn, thus posixmodule.c does not build. Build log attached. I'll create a patch for this.

    Testing environment: my building scripts [1] + Android NDK r16b
    Building target: ARM64, android-21

    /cc Primary reviewer of bpo-20104 and Android enthusiasts in CPython

    PS. Looks like posix_spawn will be available in Android P (API 28) [2]. For now I propose to disable it in CPython.

    [1] https://github.com/yan12125/python3-android/
    [2] https://android-review.googlesource.com/c/platform/bionic/+/504842

    @yan12125 yan12125 mannequin added 3.7 (EOL) end of life build The build process and cross-build labels Jan 29, 2018
    @vstinner
    Copy link
    Member

    New changeset 8997f9c by Victor Stinner (Chih-Hsuan Yen) in branch 'master':
    bpo-32705: Current Android does not have posix_spawn (bpo-5413)
    8997f9c

    @yan12125
    Copy link
    Mannequin Author

    yan12125 mannequin commented Jan 30, 2018

    With #5418 merged, I guess this issue can be closed?

    @gpshead
    Copy link
    Member

    gpshead commented Jan 30, 2018

    likely. i don't have an android build, please reopen if there are still android posix_spawn related build issues.

    @gpshead gpshead closed this as completed Jan 30, 2018
    @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
    3.7 (EOL) end of life build The build process and cross-build
    Projects
    None yet
    Development

    No branches or pull requests

    2 participants