GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Build failure on wasm-sdk-17 · Issue #100340 · python/cpython

Build failure on wasm-sdk-17 #100340

New issue
Copy link
New issue
Copy link
Closed
Closed
Build failure on wasm-sdk-17#100340
Copy link
Labels
OS-wasibuildThe build process and cross-buildThe build process and cross-buildtype-bugAn unexpected behavior, bug, or errorAn unexpected behavior, bug, or error

Description

@kushaldas
kushaldas
opened on Dec 19, 2022
Issue body actions

Bug report

wasi build is failing on the latest wasm-sdk-17 due to a change in clang itself.

How to verify the failure?

Install latest (version 17) of the wasm-sdk from the link above.

Then:

./Tools/wasm/wasm_build.py wasi

...
<snipped output>

ccache /opt/wasi-sdk/bin/clang  -Wsign-compare -Wunreachable-code -DNDEBUG -g -fwrapv -O3 -Wall    -std=c11 -Wextra -Wno-unused-parameter -Wno-missing-field-initializers -Werror=implicit-function-declaration -fvisibility=hidden  -I../../Include/internal -IObjects -IInclude -IPython -I. -I../../Include    -DPy_BUILD_CORE_BUILTIN -c ../../Modules/_weakref.c -o Modules/_weakref.o
../../Modules/timemodule.c:1972:13: error: incompatible pointer to integer conversion passing 'const struct __clockid *' to parameter of type 'long' [-Wint-conversion]
        if (PyModule_AddIntMacro(module, CLOCK_REALTIME) < 0) {
            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../../Include/modsupport.h:59:69: note: expanded from macro 'PyModule_AddIntMacro'
#define PyModule_AddIntMacro(m, c) PyModule_AddIntConstant((m), #c, (c))
                                                                    ^~~
../../Include/modsupport.h:51:71: note: passing argument to parameter here
PyAPI_FUNC(int) PyModule_AddIntConstant(PyObject *, const char *, long);
                                                                      ^
../../Modules/timemodule.c:1979:13: error: incompatible pointer to integer conversion passing 'const struct __clockid *' to parameter of type 'long' [-Wint-conversion]
        if (PyModule_AddIntMacro(module, CLOCK_MONOTONIC) < 0) {
            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../../Include/modsupport.h:59:69: note: expanded from macro 'PyModule_AddIntMacro'
#define PyModule_AddIntMacro(m, c) PyModule_AddIntConstant((m), #c, (c))
                                                                    ^~~
../../Include/modsupport.h:51:71: note: passing argument to parameter here
PyAPI_FUNC(int) PyModule_AddIntConstant(PyObject *, const char *, long);
                                                                      ^
2 errors generated.
make: *** [Makefile:2872: Modules/timemodule.o] Error 1
make: *** Waiting for unfinished jobs....

Linked PRs

  • gh-100340: Allows -Wno-int-conversion for wasm #100341
  • [3.11] gh-100340: Allows -Wno-int-conversion for wasm (GH-100341) #106066

Activity

  1. kushaldas
    added
    type-bugAn unexpected behavior, bug, or errorAn unexpected behavior, bug, or error
    on Dec 19, 2022
  2. kushaldas
    added 2 commits that reference this issue on Dec 19, 2022

    Fixes python#100340 allows -Wno-int-conversion for wasm

    This issue will close once commit f98222d is merged into the 'master' branch.
    f98222d

    Fixes python#100340 allows -Wno-int-conversion for wasm

    This issue will close once commit cd69467 is merged into the 'master' branch.
    cd69467
  3. AlexWaygood
    added
    buildThe build process and cross-buildThe build process and cross-build
    OS-wasi
    on Dec 19, 2022
  4. kushaldas
    added 2 commits that reference this issue on Dec 20, 2022

    Fixes python#100340 allows -Wno-int-conversion for wasm

    This issue will close once commit 64a65f3 is merged into the 'master' branch.
    64a65f3

    Fixes python#100340 allows -Wno-int-conversion for wasm

    This issue will close once commit 8fe9902 is merged into the 'master' branch.
    8fe9902
  5. kushaldas
    added a commit that references this issue on Jan 17, 2023

    Fixes python#100340 allows -Wno-int-conversion for wasm

    This issue will close once commit 8da5d58 is merged into the 'master' branch.
    8da5d58
  6. kushaldas
    closed this as completedin 75c8133on Jan 18, 2023
  7. brettcannon
    added this to WASI to tier 2on Jun 3, 2023
  8. github-project-automation
    moved this to Todo in WASI to tier 2on Jun 3, 2023
  9. brettcannon
    moved this from Todo to Done in WASI to tier 2on Jun 3, 2023
  10. miss-islington
    added a commit that references this issue on Jun 24, 2023

    pythongh-100340: Allows -Wno-int-conversion for wasm (pythonGH-100341)

    1cd76ef
  11. erlend-aasland
    added a commit that references this issue on Jun 28, 2023

    [3.11] gh-100340: Allows -Wno-int-conversion for wasm (GH-100341) (#1…

    966e0f6
  12. eric-wieser
    added a commit that references this issue on Sep 21, 2024

    pythongh-100340: Allows -Wno-int-conversion for wasm (python#100341)

    75c8133
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    OS-wasibuildThe build process and cross-buildThe build process and cross-buildtype-bugAn unexpected behavior, bug, or errorAn unexpected behavior, bug, or error

    Projects

      • WASI to tier 2
        Closed
        Status
        Done
        Show more project fields

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions

      You are visitor 01402503 — GeezerVison does not store the pages you look up.
      Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.