Summary: | dev-python/ruamel-yaml-clib-0.2.7 - _ruamel_yaml.c: error: incompatible function pointer types passing int (void , char , int, int | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Toralf Förster <toralf> |
Component: | Current packages | Assignee: | Marek Szuba <marecki> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | plevine457, proxy-maint, python, sam, zoltan |
Priority: | Normal | Keywords: | PATCH |
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
See Also: | https://bugs.gentoo.org/show_bug.cgi?id=907232 | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Bug Depends on: | |||
Bug Blocks: | 870412 | ||
Attachments: |
emerge-info.txt
dev-python:ruamel-yaml-clib-0.2.7:20221109-160522.log emerge-history.txt environment etc.clang.tar.bz2 etc.portage.tar.bz2 temp.tar.bz2 ruamel-yaml-clib-0.2.7-clang-16.patch ruamel-yaml-clib-0.2.7-clang-16.patch |
Description
Toralf Förster
2022-11-09 19:27:56 UTC
Created attachment 830169 [details]
emerge-info.txt
Created attachment 830171 [details]
dev-python:ruamel-yaml-clib-0.2.7:20221109-160522.log
Created attachment 830173 [details]
emerge-history.txt
Created attachment 830175 [details]
environment
Created attachment 830177 [details]
etc.clang.tar.bz2
Created attachment 830179 [details]
etc.portage.tar.bz2
Created attachment 830181 [details]
temp.tar.bz2
Created attachment 830253 [details, diff] ruamel-yaml-clib-0.2.7-clang-16.patch Upstream PR: https://sourceforge.net/p/ruamel-yaml-clib/code/merge-requests/3/ (In reply to Peter Levine from comment #8) > Created attachment 830253 [details, diff] [details, diff] > ruamel-yaml-clib-0.2.7-clang-16.patch > > Upstream PR: > https://sourceforge.net/p/ruamel-yaml-clib/code/merge-requests/3/ Thanks for both fixing & sending upstream. The bug has been closed via the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=26295868e9b7dfaf330dcc979d9510a24dfe06aa commit 26295868e9b7dfaf330dcc979d9510a24dfe06aa Author: Marek Szuba <marecki@gentoo.org> AuthorDate: 2022-11-10 22:54:58 +0000 Commit: Marek Szuba <marecki@gentoo.org> CommitDate: 2022-11-10 22:54:58 +0000 dev-python/ruamel-yaml-clib: fix clang-16 build error Thanks-to: Peter Levine <plevine457@gmail.com> Closes: https://bugs.gentoo.org/880651 Signed-off-by: Marek Szuba <marecki@gentoo.org> .../files/ruamel-yaml-clib-0.2.7-clang-16.patch | 34 ++++++++++++++++++++++ .../ruamel-yaml-clib/ruamel-yaml-clib-0.2.7.ebuild | 4 +++ 2 files changed, 38 insertions(+) (In reply to Sam James from comment #9) > (In reply to Peter Levine from comment #8) > > Created attachment 830253 [details, diff] [details, diff] [details, diff] > > ruamel-yaml-clib-0.2.7-clang-16.patch > > > > Upstream PR: > > https://sourceforge.net/p/ruamel-yaml-clib/code/merge-requests/3/ > > Thanks for both fixing & sending upstream. The patch here is for the cython-generated sources rather than the real source. The bug has been referenced in the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=8d3d6ac7e5935463fd877b6b31c3605dbd16af96 commit 8d3d6ac7e5935463fd877b6b31c3605dbd16af96 Author: Sam James <sam@gentoo.org> AuthorDate: 2023-05-27 02:43:37 +0000 Commit: Sam James <sam@gentoo.org> CommitDate: 2023-05-27 02:44:28 +0000 dev-python/ruamel-yaml-clib: fix cythonize call to always regenerate Ionen took a look and ended up spotting that it wasn't really regenerating it (and that the Clang 16 patch is touching only the generated sources, so is being overwritten now). Also, while here, I noticed it doesn't respect MAKEOPTS, so fix that. Now Python 3.12 builds again. Bug: https://bugs.gentoo.org/880651 Signed-off-by: Sam James <sam@gentoo.org> .../files/ruamel-yaml-clib-0.2.7-clang-16.patch | 34 ---------------------- .../ruamel-yaml-clib/ruamel-yaml-clib-0.2.7.ebuild | 8 ++--- 2 files changed, 2 insertions(+), 40 deletions(-) (In reply to Sam James from comment #11) > (In reply to Sam James from comment #9) > > (In reply to Peter Levine from comment #8) > > > Created attachment 830253 [details, diff] [details, diff] [details, diff] [details, diff] > > > ruamel-yaml-clib-0.2.7-clang-16.patch > > > > > > Upstream PR: > > > https://sourceforge.net/p/ruamel-yaml-clib/code/merge-requests/3/ > > > > Thanks for both fixing & sending upstream. > > The patch here is for the cython-generated sources rather than the real > source. ... so reopening. Peter, could you fix the .pyx? Created attachment 862957 [details, diff]
ruamel-yaml-clib-0.2.7-clang-16.patch
OK. This only patches _ruamel_yaml.pxd and _ruamel_yaml.pyx. It builds fine with clang-16.0.4 and gcc-12.3.1_p20230519 but I have little experience with cython and bindings so you might want to confirm it works on your end.
updated upstream PR: https://sourceforge.net/p/ruamel-yaml-clib/code/merge-requests/4/ The bug has been closed via the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=0e656f10919ab3442cdbc868b932a1aa37d6d350 commit 0e656f10919ab3442cdbc868b932a1aa37d6d350 Author: Marek Szuba <marecki@gentoo.org> AuthorDate: 2023-08-20 12:50:29 +0000 Commit: Marek Szuba <marecki@gentoo.org> CommitDate: 2023-08-20 12:51:44 +0000 dev-python/ruamel-yaml-clib: support building with -Werror=incompatible-function-pointer-types Closes: https://bugs.gentoo.org/880651 Signed-off-by: Marek Szuba <marecki@gentoo.org> ...amel-yaml-clib-0.2.7_cython_pointer_types.patch | 75 ++++++++++++++++++++++ .../ruamel-yaml-clib/ruamel-yaml-clib-0.2.7.ebuild | 4 ++ 2 files changed, 79 insertions(+) |