Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 880619 - media-libs/sdl-mixer-1.2.12-r5 - /.../dynamic_ogg.c: error: incompatible function pointer types assigning to int ( )(void , OggVor
Summary: media-libs/sdl-mixer-1.2.12-r5 - /.../dynamic_ogg.c: error: incompatible func...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Gentoo Games
URL:
Whiteboard: fixed in 1.2.12_p20221010
Keywords:
Depends on:
Blocks: c99-porting
  Show dependency tree
 
Reported: 2022-11-09 15:43 UTC by Toralf Förster
Modified: 2024-10-28 19:49 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
emerge-info.txt (emerge-info.txt,17.48 KB, text/plain)
2022-11-09 15:43 UTC, Toralf Förster
Details
emerge-history.txt (emerge-history.txt,22.75 KB, text/plain)
2022-11-09 15:43 UTC, Toralf Förster
Details
environment (environment,96.93 KB, text/plain)
2022-11-09 15:43 UTC, Toralf Förster
Details
etc.clang.tar.bz2 (etc.clang.tar.bz2,705 bytes, application/x-bzip)
2022-11-09 15:43 UTC, Toralf Förster
Details
etc.portage.tar.bz2 (etc.portage.tar.bz2,14.44 KB, application/x-bzip)
2022-11-09 15:43 UTC, Toralf Förster
Details
logs.tar.bz2 (logs.tar.bz2,6.38 KB, application/x-bzip)
2022-11-09 15:43 UTC, Toralf Förster
Details
media-libs:sdl-mixer-1.2.12-r5:20221109-152538.log (media-libs:sdl-mixer-1.2.12-r5:20221109-152538.log,23.03 KB, text/plain)
2022-11-09 15:43 UTC, Toralf Förster
Details
temp.tar.bz2 (temp.tar.bz2,29.19 KB, application/x-bzip)
2022-11-09 15:43 UTC, Toralf Förster
Details
var.tmp.clang.tar.bz2 (var.tmp.clang.tar.bz2,1.80 KB, application/x-bzip)
2022-11-09 15:43 UTC, Toralf Förster
Details
sdl-mixer-1.2.12-r6 build.log (build.log,63.98 KB, text/plain)
2022-11-10 09:50 UTC, jospezial
Details
emerge --info (file_880619.txt,12.18 KB, text/plain)
2022-11-10 09:53 UTC, jospezial
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Toralf Förster gentoo-dev 2022-11-09 15:43:25 UTC
too long lines were shrinked:

libtool: compile:  clang -pipe -march=native -fno-diagnostics-color -O2 -Werror=implicit-function-declaration -Werror=implicit-int -D_GNU_SOURCE=1 -I/usr/include/SDL -D_GNU_SOURCE=1 -D_REENTRANT -fvisibility=hidden -DHAVE_FORK -DCMD_MUSIC -DWAV_MUSIC -DMID_MUSIC -DOGG_MUSIC -DFLAC_MUSIC -DHAVE_SIGNA
2 warnings generated.
/bin/sh ./libtool --mode=compile clang -pipe -march=native -fno-diagnostics-color -O2 -Werror=implicit-function-declaration -Werror=implicit-int  -D_GNU_SOURCE=1 -I/usr/include/SDL -D_GNU_SOURCE=1 -D_REENTRANT  -fvisibility=hidden -DHAVE_FORK -DCMD_MUSIC -DWAV_MUSIC -DMID_MUSIC -DOGG_MUSIC -DFLAC_MU
libtool: compile:  clang -pipe -march=native -fno-diagnostics-color -O2 -Werror=implicit-function-declaration -Werror=implicit-int -D_GNU_SOURCE=1 -I/usr/include/SDL -D_GNU_SOURCE=1 -D_REENTRANT -fvisibility=hidden -DHAVE_FORK -DCMD_MUSIC -DWAV_MUSIC -DMID_MUSIC -DOGG_MUSIC -DFLAC_MUSIC -DHAVE_SIGNA
/var/tmp/portage/media-libs/sdl-mixer-1.2.12-r5/work/SDL_mixer-1.2.12/dynamic_ogg.c:111:28: error: incompatible function pointer types assigning to 'int (*)(void *, OggVorbis_File *, char *, long, ov_callbacks)' (aka 'int (*)(void *, struct OggVorbis_File *, char *, long, ov_callbacks)') from 'int (
                vorbis.ov_open_callbacks = ov_open_callbacks;
                                         ^ ~~~~~~~~~~~~~~~~~

  -------------------------------------------------------------------

  This is an unstable amd64 chroot image at a tinderbox (==build bot)
  name: 17.1_desktop_gnome-j4-20221109-090003

  -------------------------------------------------------------------

GNUMAKEFLAGS="$GNUMAKEFLAGS --jobserver-style=pipe"
GNUMAKEFLAGS="$GNUMAKEFLAGS --shuffle"
CC=clang
CXX=clang++
gcc-config -l:
 [1] x86_64-pc-linux-gnu-12 *
clang/llvm (if any):
clang version 15.0.4
Target: x86_64-pc-linux-gnu
Thread model: posix
InstalledDir: /usr/lib/llvm/15/bin
Configuration file: /etc/clang/clang.cfg
/usr/lib/llvm/15
15.0.4
Python 3.10.8
Available Rust versions:
  [1]   rust-bin-1.64.0 *
php cli (if any):

  HEAD of ::gentoo
commit 13b7b1255ae6125608899908b118a34f8a8347f1
Author: Repository mirror & CI <repomirrorci@gentoo.org>
Date:   Wed Nov 9 14:16:53 2022 +0000

    2022-11-09 14:16:52 UTC

emerge -qpvO media-libs/sdl-mixer
[ebuild  N    ] media-libs/sdl-mixer-1.2.12-r5  USE="flac mad mp3 vorbis wav -fluidsynth -midi -mikmod -mod -modplug -playtools -smpeg -static-libs -timidity" ABI_X86="(64) -32 (-x32)"
Comment 1 Toralf Förster gentoo-dev 2022-11-09 15:43:26 UTC
Created attachment 830133 [details]
emerge-info.txt
Comment 2 Toralf Förster gentoo-dev 2022-11-09 15:43:27 UTC
Created attachment 830135 [details]
emerge-history.txt
Comment 3 Toralf Förster gentoo-dev 2022-11-09 15:43:28 UTC
Created attachment 830137 [details]
environment
Comment 4 Toralf Förster gentoo-dev 2022-11-09 15:43:29 UTC
Created attachment 830139 [details]
etc.clang.tar.bz2
Comment 5 Toralf Förster gentoo-dev 2022-11-09 15:43:30 UTC
Created attachment 830141 [details]
etc.portage.tar.bz2
Comment 6 Toralf Förster gentoo-dev 2022-11-09 15:43:31 UTC
Created attachment 830143 [details]
logs.tar.bz2
Comment 7 Toralf Förster gentoo-dev 2022-11-09 15:43:32 UTC
Created attachment 830145 [details]
media-libs:sdl-mixer-1.2.12-r5:20221109-152538.log
Comment 8 Toralf Förster gentoo-dev 2022-11-09 15:43:33 UTC
Created attachment 830147 [details]
temp.tar.bz2
Comment 9 Toralf Förster gentoo-dev 2022-11-09 15:43:34 UTC
Created attachment 830149 [details]
var.tmp.clang.tar.bz2
Comment 10 Larry the Git Cow gentoo-dev 2022-11-10 07:13:36 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=24c9402d23e17f82fd06e074f460da2645698244

commit 24c9402d23e17f82fd06e074f460da2645698244
Author:     Sam James <sam@gentoo.org>
AuthorDate: 2022-11-10 06:45:50 +0000
Commit:     Sam James <sam@gentoo.org>
CommitDate: 2022-11-10 06:45:50 +0000

    media-libs/sdl-mixer: fix Clang 16 compatibility
    
    Closes: https://bugs.gentoo.org/880619
    Signed-off-by: Sam James <sam@gentoo.org>

 ...dl-mixer-1.2.12-incompatible-func-clang16.patch |  64 +++++++++++
 media-libs/sdl-mixer/sdl-mixer-1.2.12-r6.ebuild    | 128 +++++++++++++++++++++
 2 files changed, 192 insertions(+)
Comment 11 jospezial 2022-11-10 09:50:06 UTC
Created attachment 830271 [details]
sdl-mixer-1.2.12-r6 build.log

>>> Completed installing media-libs/sdl-mixer-1.2.12-r6 into /var/tmp/portage/media-libs/sdl-mixer-1.2.12-r6/image

 * Final size of build directory: 16496 KiB (16.1 MiB)
 * Final size of installed tree:    292 KiB

 * 
 * QA Notice: Package triggers severe warnings which indicate that it
 *            may exhibit random runtime failures.
 * 
 * /var/tmp/portage/media-libs/sdl-mixer-1.2.12-r6/work/SDL_mixer-1.2.12/mixer.c:153:60: warning: implicit declaration of function ‘Mix_InitFluidSynth’ [-Wimplicit-function-declaration]
 * /var/tmp/portage/media-libs/sdl-mixer-1.2.12-r6/work/SDL_mixer-1.2.12/mixer.c:205:17: warning: implicit declaration of function ‘Mix_QuitFluidSynth’ [-Wimplicit-function-declaration]
 * /var/tmp/portage/media-libs/sdl-mixer-1.2.12-r6/work/SDL_mixer-1.2.12/dynamic_mod.c:244:36: warning: assignment to ‘BOOL (*)(CHAR *)’ {aka ‘int (*)(char *)’} from incompatible pointer type ‘int (*)(const CHAR *)’ {aka ‘int (*)(const char *)’} [-Wincompatible-pointer-types]
 * /var/tmp/portage/media-libs/sdl-mixer-1.2.12-r6/work/SDL_mixer-1.2.12/dynamic_mod.c:248:40: warning: assignment to ‘char * (*)(int)’ from incompatible pointer type ‘const char * (*)(int)’ [-Wincompatible-pointer-types]
 * /var/tmp/portage/media-libs/sdl-mixer-1.2.12-r6/work/SDL_mixer-1.2.12/dynamic_fluidsynth.c:40:25: warning: assignment to ‘int (*)(fluid_player_t *)’ {aka ‘int (*)(struct _fluid_player_t *)’} from incompatible pointer type ‘void (*)(fluid_player_t *)’ {aka ‘void (*)(struct _fluid_player_t *)’} [-Wincompatible-pointer-types]
 * /var/tmp/portage/media-libs/sdl-mixer-1.2.12-r6/work/SDL_mixer-1.2.12/dynamic_fluidsynth.c:40:25: warning: assignment to ‘int (*)(fluid_synth_t *)’ {aka ‘int (*)(struct _fluid_synth_t *)’} from incompatible pointer type ‘void (*)(fluid_synth_t *)’ {aka ‘void (*)(struct _fluid_synth_t *)’} [-Wincompatible-pointer-types]
 * 
 *  Please file a bug about this at https://bugs.gentoo.org/
 *  with the maintainer of the package.
 * 
 * ERROR: media-libs/sdl-mixer-1.2.12-r6::gentoo failed:
 *   install aborted due to severe warnings shown above
 * 
 * Call stack:
 *   misc-functions.sh, line 670:  Called install_qa_check
 *   misc-functions.sh, line 134:  Called source 'install_symlink_html_docs'
 *      90gcc-warnings, line 197:  Called gcc_warn_check
 *      90gcc-warnings, line 183:  Called die
 * The specific snippet of code:
 *                              die "install aborted due to severe warnings shown above"
Comment 12 Sam James archtester Gentoo Infrastructure gentoo-dev Security 2022-11-10 09:50:58 UTC
thx, looking (might be doing a snapshot sooner than I was thinking, as there's tonnes of patches which touch various bits like this)
Comment 13 jospezial 2022-11-10 09:53:22 UTC
Created attachment 830273 [details]
emerge --info
Comment 15 Sam James archtester Gentoo Infrastructure gentoo-dev Security 2022-11-10 10:04:37 UTC
https://github.com/libsdl-org/SDL_mixer/commit/05b12a3c22c0746c29dc5478f5b7fbd8a51a1303 is enough but there's still incompatible func. ptr warnings, so snapshot it is
Comment 16 Larry the Git Cow gentoo-dev 2022-11-10 10:05:37 UTC
The bug has been referenced in the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=3ece26abf8b9b5a09d17bf166e441cb9746ee4d8

commit 3ece26abf8b9b5a09d17bf166e441cb9746ee4d8
Author:     Sam James <sam@gentoo.org>
AuthorDate: 2022-11-10 10:04:58 +0000
Commit:     Sam James <sam@gentoo.org>
CommitDate: 2022-11-10 10:05:29 +0000

    media-libs/sdl-mixer: backport additional header patch
    
    We're going to do a snapshot but it feels right to fix this bit first
    as it got introduced by one of the previous patches (only shows up
    with some USE flag combos).
    
    Bug: https://bugs.gentoo.org/880619
    Signed-off-by: Sam James <sam@gentoo.org>

 ...dl-mixer-1.2.12-incompatible-func-clang16.patch | 30 ++++++++++++++++++++++
 ...1.2.12-r6.ebuild => sdl-mixer-1.2.12-r7.ebuild} |  0
 2 files changed, 30 insertions(+)
Comment 17 Larry the Git Cow gentoo-dev 2022-11-11 03:50:14 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=209e9244c663238ed56d0ca58c5b2a19e06bf6c8

commit 209e9244c663238ed56d0ca58c5b2a19e06bf6c8
Author:     Sam James <sam@gentoo.org>
AuthorDate: 2022-11-11 03:48:58 +0000
Commit:     Sam James <sam@gentoo.org>
CommitDate: 2022-11-11 03:48:58 +0000

    media-libs/sdl-mixer: add 1.2.12_p20221010
    
    Upstream aren't making releases anymore (since a long time ago!) for the 1.2.x
    branch but are kindly doing backports, so let's make a snapshot.
    
    Fixes a substantial number of warnings, build system quirks, and even
    a security bug. What a bargain!
    
    Bug: https://bugs.gentoo.org/771168
    Closes: https://bugs.gentoo.org/729740
    Closes: https://bugs.gentoo.org/880619
    Signed-off-by: Sam James <sam@gentoo.org>

 media-libs/sdl-mixer/Manifest                      |   1 +
 .../sdl-mixer/sdl-mixer-1.2.12_p20221010.ebuild    | 116 +++++++++++++++++++++
 2 files changed, 117 insertions(+)