|Summary:||media-sound/murmur-1.2.4_beta1-r1: ‘MAX’ was not declared in this scope|
|Product:||Gentoo Linux||Reporter:||Martin von Gagern <Martin.vGagern>|
|Component:||Current packages||Assignee:||Timo Gurr (RETIRED) <tgurr>|
|Severity:||normal||CC:||kripton, polynomial-c, voip+disabled|
|Package list:||Runtime testing required:||---|
Description Martin von Gagern 2013-03-06 12:31:37 UTC
Created attachment 341104 [details] build log murmur-1.2.4 fails to (re-)build: Server.cpp: In member function ‘void Server::udpActivated(int)’: Server.cpp:553:21: error: ‘MAX’ was not declared in this scope compilation terminated due to -Wfatal-errors. make: *** [Server.o] Error 1 make: Leaving directory `/var/tmp/portage/media-sound/murmur-1.2.4_beta1-r1/work/mumble-1.2.4~beta1/src/murmur'
Comment 1 Martin von Gagern 2013-03-06 12:32:46 UTC
Created attachment 341106 [details] emerge --info
Comment 2 Martin von Gagern 2013-03-06 12:45:00 UTC
Created attachment 341108 [details, diff] Woraround This patch will avoid the issue by providing a MAX macro unless one has already been defined. This avoids the trouble of identifying where the macro might have come from in previous builds or on upstream build systems. As murmur uses epatch_user, you can simply drop this file into a (newly created) directory called /etc/portage/patches/media-sound/murmur and emerge mumble. This is a quick solution until the issue has been fixed in the main portage tree.
Comment 3 Lars Wendler (Polynomial-C) 2013-03-07 08:29:54 UTC
Would be nice to know if murmur-1.2.4_rc1 is affected as well.
Comment 4 jannis 2013-03-10 19:00:23 UTC
(In reply to comment #3) > Would be nice to know if murmur-1.2.4_rc1 is affected as well. seems to work on my amd64-server
Comment 5 jannis 2013-03-10 19:01:16 UTC
(In reply to comment #4) > (In reply to comment #3) > > Would be nice to know if murmur-1.2.4_rc1 is affected as well. > > seems to work on my amd64-server erm sorry, using "1.2.4_beta1-r1" since the _rc1 isn't in tree yet. seems to be related to installed version of protobuf
Comment 6 Timo Gurr (RETIRED) 2013-03-18 02:26:24 UTC
The latest available snapshot mumble-1.2.4~rc1-8-gb115a29.tar.gz seems to be affected as well. Martin would you please report this upstream to get your fix committed so we can add it to the ebuild as well? Thanks.
Comment 7 Martin von Gagern 2013-03-18 06:44:30 UTC
(In reply to comment #6) > Martin would you please report this upstream […] Reported at https://github.com/mumble-voip/mumble/pull/153
Comment 8 Martin von Gagern 2013-03-18 20:46:01 UTC
Fix committed upstream as https://github.com/mumble-voip/mumble/commit/5aec9c6403d6bc65f957ee1648e6848f2287f670