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
x11-apps/xdm: Fix -Wincompatible-pointer-types warning from GCC 14 #35916
Conversation
57bfefc
to
72b6544
Compare
Pull Request assignmentSubmitter: @listout x11-apps/xdm: @gentoo/x11 Linked bugsBugs linked: 927712 In order to force reassignment and/or bug reference scan, please append Docs: Code of Conduct ● Copyright policy (expl.) ● Devmanual ● GitHub PRs ● Proxy-maint guide |
Pull request CI reportReport generated at: 2024-03-25 19:05 UTC There are existing issues already. Please look into the report to make sure none of them affect the packages in question: |
The xdm-1.1.15-gcc14.patch patch here claims to be my upstream patch from https://gitlab.freedesktop.org/xorg/app/xdm/-/merge_requests/18/diffs but it's actually the inverse of my patch, undoing my change and restoring the previous code. The actual root cause here is a change in libXaw 1.0.16 that was made earlier this month, as discussed in https://gitlab.freedesktop.org/xorg/app/xdm/-/issues/16. |
@alanc That's right, as I said in the Gitlab issue, I somehow completely missed it, I'll fix it. |
Fixes build when building against libXaw 1.0.16 since xorg/lib/libxaw@d0fcbd97 changed the definition of XawListChange in <X11/Xaw/List.h> from _Xconst char * to String *, where String is defined as const if _CONST_X_STRING is defined. Closes: https://bugs.gentoo.org/927712 Signed-off-by: Brahmajit Das <brahmajit.xyz@gmail.com>
72b6544
to
a16450d
Compare
@alanc should be good now, I backported your patch from master. Thank you! |
Pull request CI reportReport generated at: 2024-04-03 05:31 UTC There are existing issues already. Please look into the report to make sure none of them affect the packages in question: |
This is all fixed in xdm-1.1.16, right? |
Fixes build when building against libXaw 1.0.16 since
xorg/lib/libxaw@d0fcbd97 changed the definition of XawListChange
in <X11/Xaw/List.h> from _Xconst char * to String *, where
String is defined as const if _CONST_X_STRING is defined.
Closes: https://bugs.gentoo.org/927712
Signed-off-by: Brahmajit Das brahmajit.xyz@gmail.com