Skip to content

Conversation

chenrui333
Copy link
Member

@chenrui333 chenrui333 commented Oct 12, 2024

@github-actions github-actions bot added python Python use is a significant feature of the PR or issue rust Rust use is a significant feature of the PR or issue labels Oct 12, 2024
@Homebrew Homebrew deleted a comment from github-actions bot Oct 12, 2024
@chenrui333
Copy link
Member Author

    clang -fno-strict-overflow -Wsign-compare -Wunreachable-code -fno-common -dynamic -DNDEBUG -g -O3 -Wall -isysroot /Library/Developer/CommandLineTools/SDKs/MacOSX15.sdk -I/private/tmp/fastapi--httptools-20241012-12046-r4vfsx/httptools-0.6.1/vendor/llhttp/include -I/private/tmp/fastapi--httptools-20241012-12046-r4vfsx/httptools-0.6.1/vendor/llhttp/src -I/opt/homebrew/Cellar/fastapi/0.115.2/libexec/include -I/opt/homebrew/opt/[email protected]/Frameworks/Python.framework/Versions/3.13/include/python3.13 -c httptools/parser/parser.c -o build/temp.macosx-15.0-arm64-cpython-313/httptools/parser/parser.o -O2
    httptools/parser/parser.c:9940:70: error: too few arguments to function call, expected 6, have 5
     9938 |                 int ret = _PyLong_AsByteArray((PyLongObject *)v,
          |                           ~~~~~~~~~~~~~~~~~~~
     9939 |                                               bytes, sizeof(val),
     9940 |                                               is_little, !is_unsigned);
          |                                                                      ^
    /opt/homebrew/opt/[email protected]/Frameworks/Python.framework/Versions/3.13/include/python3.13/cpython/longobject.h:111:17: note: '_PyLong_AsByteArray' declared here
      111 | PyAPI_FUNC(int) _PyLong_AsByteArray(PyLongObject* v,
          |                 ^                   ~~~~~~~~~~~~~~~~
      112 |     unsigned char* bytes, size_t n,
          |     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
      113 |     int little_endian, int is_signed, int with_exceptions);
          |     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    httptools/parser/parser.c:10136:70: error: too few arguments to function call, expected 6, have 5
     10134 |                 int ret = _PyLong_AsByteArray((PyLongObject *)v,
           |                           ~~~~~~~~~~~~~~~~~~~
     10135 |                                               bytes, sizeof(val),
     10136 |                                               is_little, !is_unsigned);
           |                                                                      ^
    /opt/homebrew/opt/[email protected]/Frameworks/Python.framework/Versions/3.13/include/python3.13/cpython/longobject.h:111:17: note: '_PyLong_AsByteArray' declared here
      111 | PyAPI_FUNC(int) _PyLong_AsByteArray(PyLongObject* v,
          |                 ^                   ~~~~~~~~~~~~~~~~
      112 |     unsigned char* bytes, size_t n,
          |     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
      113 |     int little_endian, int is_signed, int with_exceptions);
          |     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    2 errors generated.
    error: command '/opt/homebrew/Library/Homebrew/shims/mac/super/clang' failed with exit code 1
    error: subprocess-exited-with-error

@chenrui333 chenrui333 added upstream issue An upstream issue report is needed build failure CI fails while building the software labels Oct 12, 2024
Signed-off-by: Rui Chen <[email protected]>

fastapi: add cython patch for httptools

Signed-off-by: Rui Chen <[email protected]>
@chenrui333 chenrui333 added ready to merge PR can be merged once CI is green and removed upstream issue An upstream issue report is needed build failure CI fails while building the software labels Oct 12, 2024
Copy link
Contributor

🤖 An automated task has requested bottles to be published to this PR.

@github-actions github-actions bot added the CI-published-bottle-commits The commits for the built bottles have been pushed to the PR branch. label Oct 13, 2024
@BrewTestBot BrewTestBot enabled auto-merge October 13, 2024 04:14
@BrewTestBot BrewTestBot added this pull request to the merge queue Oct 13, 2024
Merged via the queue into master with commit 581160e Oct 13, 2024
15 checks passed
@BrewTestBot BrewTestBot deleted the fastapi-py3.13 branch October 13, 2024 04:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI-published-bottle-commits The commits for the built bottles have been pushed to the PR branch. python Python use is a significant feature of the PR or issue python-3.13-migration ready to merge PR can be merged once CI is green rust Rust use is a significant feature of the PR or issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants